Skip to content
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

Adding support for building on arm64v8 #73

Closed
wants to merge 1 commit into from
Closed

Adding support for building on arm64v8 #73

wants to merge 1 commit into from

Conversation

ppouliot
Copy link

@ppouliot ppouliot commented Jun 1, 2018

This adds a Dockerfile.arm64v8 which successfully builds on a Pine64. Additionally a docker-compose.arm64v8.yaml is also supplied and was successfully built and tested on the same Pine64.

Additionally for successful builds to occur, openssl-dev was replaced with libressl-dev in the Dockerfile. This apparently is a know dependency issue with alpine-linux and openssl-dev.

@calmyournerves
Copy link
Contributor

Hi @ppouliot

Thanks for this! To keep things simple, I'd prefer not to duplicate the whole Dockerfile. I quickly checked, and we'd probably need to change the build scripts quite a bit. Something similar (ARGs) like in the Dockerfile.ldap could be used. Maybe you got another idea?

@calmyournerves calmyournerves added the awaiting answer There is still some open discussion. label Jun 4, 2018
@ppouliot
Copy link
Author

ppouliot commented Jun 4, 2018 via email

@thde
Copy link
Contributor

thde commented Jun 25, 2018

Isn't the official pyhon arm compatible? https://hub.docker.com/_/python/ -> Supported architectures
And the openssl packages should also be available: https://pkgs.alpinelinux.org/packages?name=openssl*

Sadly I have no way to test that here...

@thde
Copy link
Contributor

thde commented Jun 28, 2018

Ok, so it's not that easy... but with docker manifest it will be: https://docs.docker.com/edge/engine/reference/commandline/manifest/

@cimnine
Copy link
Collaborator

cimnine commented Aug 20, 2018

Thank you for sharing this with us. But I'll close this PR as I don't see much demand for this feature right now. There's also not been any progress/comment since almost a month.

@cimnine cimnine closed this Aug 20, 2018
@xvzf xvzf mentioned this pull request Dec 14, 2021
3 tasks
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
awaiting answer There is still some open discussion.
Projects
None yet
Development

Successfully merging this pull request may close these issues.

4 participants