Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

[geotiff] build with libtiff 4.5 #6929

Merged
merged 3 commits into from
Jun 23, 2023
Merged

Conversation

visr
Copy link
Contributor

@visr visr commented Jun 22, 2023

Looks like the upgrade from libtiff 4.4 to 4.5 was breaking at least on Windows.

Reported in JuliaGeo/GDAL.jl#158
Also addresses one case of #6857 in an effort to get nightly support working again (JuliaGeo/GDAL.jl#152).

@visr visr marked this pull request as draft June 22, 2023 20:10
@visr
Copy link
Contributor Author

visr commented Jun 22, 2023

I need to do PROJ_jll first (edit: see #6930), since that also has a libtiff dependency, as well as a geotiff dependency.
Then, this should be used:

dependencies = [
    Dependency("PROJ_jll"; compat="~900.200.100"),
    Dependency("Libtiff_jll"; compat="4.5.1"),
    Dependency("LibCURL_jll"; compat="7.73,8"),
]

EDIT: done

visr added a commit to visr/General that referenced this pull request Jun 22, 2023
Looks like the [upgrade](JuliaPackaging/Yggdrasil#6924) from libtiff 4.4 to 4.5 was [breaking](JuliaBinaryWrappers/Libtiff_jll.jl@fddff20#diff-248757425c80d519c91e1be718766832fdd989ae7837c5c4b6e64f95931a4757) at least on Windows.
Is this the right way to avoid breaking JLLs that depend directly on Libtiff_jll?

Ref JuliaPackaging/Yggdrasil#6929
@visr visr marked this pull request as ready for review June 23, 2023 12:49
@giordano giordano merged commit d6b7bfe into JuliaPackaging:master Jun 23, 2023
@visr visr deleted the geo-tiff branch June 23, 2023 14:08
@visr visr mentioned this pull request Sep 18, 2023
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants