-
-
Notifications
You must be signed in to change notification settings - Fork 64
Update to GDAL 3.10.0, and add libgdal-avif and libgdal-heif #992
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
Update to GDAL 3.10.0, and add libgdal-avif and libgdal-heif #992
Conversation
Hi! This is the friendly automated conda-forge-linting service. I just wanted to let you know that I linted all conda-recipes in your PR ( |
f22c48c
to
dbcd069
Compare
@conda-forge-admin, please rerender |
17e9db2
to
09329e2
Compare
@conda-forge-admin, please rerender |
5703340
to
02d785c
Compare
@conda-forge-admin, please rerender |
5bd5ddd
to
2d16ac1
Compare
@conda-forge-admin, please rerender |
pull request updated to final 3.10.0 release |
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.
🥳 Thank you!
Oh, we'd likely want to include a |
I suspect this will be somewhat painful as the |
On windows, we don't have the
For the life of me, I can't resolve in the logs what the failure is. Is it a mixed case issue with the test looking for |
For non-windows, I wonder if we should just hack in a |
I don't think so, since the test does check for |
@conda-forge-admin please rerender |
Yeah, you are right. Lets see if we can fix non-win first with my hack. |
…nda-forge-pinning 2024.11.12.14.24.54
Well, my |
I'm crossing my fingers over here as I see these CI runs spinning. We're cheering you on @akrherz - you're fighting the good fight! |
Well, at least the linux build is failing in the same manner as windows, backwards progress! Boggling... |
oOo, we are hitting this due to attempting to upload new packages not previously associated with this repo.... |
I've no idea why we have an issue specifically on the new HEIF and AVIF sub-packages, but they are far from being essential. I'd suggest we remove them for now, if that's enough to get an initial 3.10.0 green build... |
So my Would be nice to have somebody else sign off on this as being OK :) |
My hope is that this turns green now and will take it, if so! Will followup with creating a |
Hi! This is the friendly conda-forge automerge bot! I considered the following status checks when analyzing this PR:
Thus the PR was passing and merged! Have a great day! |
The osx arm64 build, on the build matching this merged PR, failed ,presumably due to some transient error (https://dev.azure.com/conda-forge/feedstock-builds/_build/results?buildId=1079258&view=logs&j=d6b58996-039f-5e48-56bf-c3a016e5cd7f&t=dd6a23d9-2356-5264-64bf-579875a5d8d6) . Could someone restart it ? (and thanks @akrherz for the workaround!) |
Thank you, restarted that job. |
Still failing with
I suspect this is some cross compilation issue. Perhaps restart again.. ? (In GDAL master on github CI, MacOS ARM64 builds work fine: https://github.com/OSGeo/gdal/actions/runs/11816321717/job/32919368044) |
Restarting a second time appears to have been a charm, also build 1 appeared to succeed as well. Thank you @rouault for all the efforts with this repo! |
This dependency was introduced in conda-forge#992.
Checklist
0
(if the version changed)conda-smithy
(Use the phrase@conda-forge-admin, please rerender
in a comment in this PR for automated rerendering)