GeoTIFF / georaster

Wrapper around Georeferenced Rasters like GeoTIFF and soon JPG and PNG that provides a standard interface
Apache License 2.0
86 stars 32 forks source link

Broken dependency on geotiff.js #21

Closed asaed closed 6 years ago

asaed commented 6 years ago

When using yarn@1.10.1 to install dependencies of our app that depends on geoblaze@^0.1.8, we get an error:

warning Pattern ["geotiff@github:danieljdufour/geotiff.js#877f064"] is trying to unpack in the same destination "/w/workspace/imagery_ui-merge-master-none/.yarn_cache/v2/npm-geotiff-0.4.1" as pattern ["geotiff@^0.4.1"]. This could result in non-deterministic behavior, skipping.
error https://codeload.github.com/danieljdufour/geotiff.js/tar.gz/877f064846d6446da1d30f727ee144ffa590ea77: Integrity check failed for "geotiff" (computed integrity doesn't match our records, got "sha1-a7VkDR5y+0Xn3cKyqsSqKtfSCXU=")

We traced the root cause to georaster (hence creating this issue). Here is the dependency subtree

geoblaze@0.1.8
| _ georaster@0.3.3
    |_ geotiff@github:danieljdufour/geotiff.js#877f064
|_ geotiff@0.4.1,

As you can see geotiff@github:danieljdufour/geotiff.js#877f064 conflicts with geotiff@0.4.1 because both of them are versioned as 0.4.1 in their respective package.json file.

Can georaster stop using geotiff@github:danieljdufour/geotiff.js#877f064 and replace it with the latest version geotiff (i.e. geotiff@1.0.0-beta.6)?

DanielJDufour commented 6 years ago

Thank you for submitting the issue @asaed ! The Github version was being used as a temporary workaround to an issue that has now been resolved. We will fix this in georaster and geoblaze and update this issue when this happens. Expect a fix within 24 hrs. Thanks again! :-)

DanielJDufour commented 6 years ago

This issue has been fixed in the source code on GitHub. We'll deploy a new version to NPM and update this ticket when that happens.

DanielJDufour commented 6 years ago

@asaed this is now fixed on NPMJS and Yarn. Please try and let me know if the fix works for you. Thanks so much! :-)

asaed commented 6 years ago

Thank you so much, @DanielJDufour . Now it works flawlessly in Yarn.

DanielJDufour commented 6 years ago

Awesome. Closing this issue. Please let us know if you find any other issues! :-)