Skip to content

Pv/gdal 3.10 #89

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

Draft
wants to merge 8 commits into
base: master
Choose a base branch
from
Draft

Pv/gdal 3.10 #89

wants to merge 8 commits into from

Conversation

philvarner
Copy link
Contributor

@philvarner philvarner commented Mar 4, 2025

Few issues currently:

  1. gdal 3.9 and 3.10 won't build on the public.ecr.aws/lambda/provided:al image -- they need to use al2023 image instead.
  2. al2023 uses dnf instead of yum, but the commands are the same
  3. install package automake instead of automake16
  4. GDAL docs state to build with commands
  && cmake --build . \
  && cmake --build . --target install \

instead of

  && make -j $(nproc) --silent && make install 

@vincentsarago
Copy link
Contributor

thanks @philvarner really appreciate your time on this.

I won't have a lot of time to spend on this but I'll try my best 🙏

@philvarner
Copy link
Contributor Author

thanks @philvarner really appreciate your time on this.

I won't have a lot of time to spend on this but I'll try my best 🙏

@vincentsarago thanks! I'm mostly looking for support for Python 3.13 right now. Can you tag master to trigger the build and push of the new images for the newer Python versions?

GDAL 3.9 and 3.10 are failing to build, so that will require more investigation, and I don't think I'll be able to do that soon.

@philvarner
Copy link
Contributor Author

@vincentsarago after further work, I'm not certain the python 3.11 and 3.12 images are going to work. The docker containers for these use al2023 instead of al2, which is why the builds use dnf instead of yum. I don't know if all of the packages built on al2 will work in al2023, so we should test this thoroughly -- which I doubt either of us has time for right now :(

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