-
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
Docker upload CI is broken #2568
Comments
The detailed error is actually
It seems that regex did have some prebuilt wheels for arm64 in previous versions, but the latest doesn't have any aarch64 builds yet. regex's CI might be broken: they have a travis-ci but I don't find any public interface where we could see the build status. I see 3 possible solutions:
|
Thanks for debugging further! Lets try the following:
|
I opened this issue on regex project: https://bitbucket.org/mrabarnett/mrab-regex/issues/434/aarch64-build-is-failing I saw that you pinned https://github.com/vbarbaresi/black/runs/4061992219?check_suite_focus=true |
Thanks for trying this. I think long term we should definitely support building from source but will probably need a multi stage build so we can keep the image size around the current ~50MB ... If you/anyone have cycles I'd totally take a PR making us able to build from source via a multi stage build to keep our image size low. |
- Install build-essential to avoid build issues like psf#2568 when dependencies don't have prebuilt wheels available - Use multi-stage build instead of trying to purge packages and cache from the image Copying `/root/.local/` installs only black's built Python dependencies (< 20 MB). So the image is barely larger than python:3-slim base image
I took a stab at it #2582 |
…build (#2582) - Install build-essential to avoid build issues like #2568 when dependencies don't have prebuilt wheels available - Use multi-stage build instead of trying to purge packages and cache from the image Copying `/root/.local/` installs only black's built Python dependencies (< 20 MB). So the image is barely larger than python:3-slim base image
This is all fixed and was used to release 20.10b0 - Thanks for your help! |
…build (#2582) - Install build-essential to avoid build issues like #2568 when dependencies don't have prebuilt wheels available - Use multi-stage build instead of trying to purge packages and cache from the image Copying `/root/.local/` installs only black's built Python dependencies (< 20 MB). So the image is barely larger than python:3-slim base image
Haven't debugged yet but seems a build level bug inside the container. Need to see if it can be reproduced locally.
Example fail: https://github.com/psf/black/actions/runs/1391464711
The text was updated successfully, but these errors were encountered: