-
-
Notifications
You must be signed in to change notification settings - Fork 25
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
BLD/RLS: update wheels to include GDAL 3.7.2 #298
BLD/RLS: update wheels to include GDAL 3.7.2 #298
Conversation
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Thanks for working on this @jorisvandenbossche !
Probably a good idea to update the GDAL docker version in release.yml
for test-sdist
to ghcr.io/osgeo/gdal:ubuntu-small-3.7.2
to match the GDAL version in the wheels.
Reproducing the linux build failure locally, this is the logs vcpkg produces to open an issue about it: Package: gdal[core,curl,expat,geos,jpeg,lerc,png,qhull,recommended-features,sqlite3]:x64-linux-dynamic -> 3.7.2 Host Environment
To Reproduce
/opt/vcpkg/buildtrees/gdal/install-x64-linux-dynamic-rel-out.log
Additional context vcpkg.json
|
The build error happens in (another option would be to let GDAL use its internal zlib copy, but that is not enabled in vcpkg, so that would require finding a way to override that, and not sure if that's possible without a custom port of gdal where we just override that one option) |
That seems to have worked! The only failures are two windows wheel test builds that regularly fail (segfault at cleanup, #213) |
Thanks @jorisvandenbossche ! 💯 |
Closes #295
Like #248, now updating to GDAL 3.7.2 (microsoft/vcpkg@6298859)