Skip to content

Conversation

yosifkit
Copy link
Member

replaces/closes #190.

@elbaschid, sorry to replace your PR, I was working on multi-arch images and found out that this fixes the build for ppc64le (https://bugs.python.org/issue22157), so I applied it to all the versions of python 3.

This might fix the build failure on arm64v8, but I haven't tested it there yet.

@yosifkit
Copy link
Member Author

yosifkit commented Jun 16, 2017

Fix works for arm64v8 too! (python 3.3, 3.4, 3.5, and 3.6)

@yosifkit yosifkit requested a review from tianon June 16, 2017 18:55
--enable-loadable-sqlite-extensions \
--enable-shared \
--with-system-ffi \
--with-system-expat \
Copy link
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

e comes before f, doesn't it? 😇

Copy link
Member Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

I apparently sorted everything else but that 😞

@yosifkit
Copy link
Member Author

All green! libexpat1, libffi6, their -devs are already installed in the buildpack-deps image, so we can just use them in the non-slim Debian images. 🎉

@tianon tianon merged commit 6d31b8c into docker-library:master Jun 16, 2017
@tianon tianon deleted the arch-fixes branch June 16, 2017 21:07
tianon added a commit to infosiftr/stackbrew that referenced this pull request Jun 16, 2017
- `php`: multiarch (docker-library/php#454)
- `python`: multiarch (docker-library/python#205, docker-library/python#206)
- `redmine`: passenger 5.1.5
- `wordpress`: multiarch (docker-library/wordpress#223) -- slightly cheated by pointing it at the `library/php` file contents from this PR
JayH5 added a commit to praekeltfoundation/docker-alpine-python that referenced this pull request Jun 19, 2017
tao12345666333 pushed a commit to tao12345666333/python that referenced this pull request Jun 28, 2018
Fix _ctypes test failure on Alpine and build failure on Debian on ppc64le
@blopker blopker mentioned this pull request Dec 5, 2018
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