-
-
Notifications
You must be signed in to change notification settings - Fork 2.5k
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
CI: test Docker ubuntu-small and ubuntu-full on linux/arm64 as well #11001
Conversation
f3e9bda
to
93f0e2e
Compare
93f0e2e
to
d5ee743
Compare
I broke libjxl for linux/arm64 with my changes, sorry about that. I don't know anything about multi-platform builds, but you also changed something with the image digest for arm64. Would reverting commit 4a37432 also have fixed that issue? |
I'm not sure why I had to change this part to be honest. I presume this is due to having switched between Docker versions. My current one is "Docker version 24.0.7, build 24.0.7-0ubuntu2~20.04.1"
|
Edit: clarified that I meant no longer support using Docker < 23.0 for building the GDAL images. I get the same with Docker 27.3. The image digest thing works if I revert your change and 4a37432 though:
so I guess that part and a lot of the buildx parts of build.sh can be simplified quite a bit when you decide to not support Docker < 23.0 for building the GDAL images. |
indeed . Docker 23 is still a bit new. I see in https://github.com/docker/cli/releases/tag/v23.0.0 it was released in Feb 2023 |
No description provided.