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

Support x86_64 and i686 architectures #94

Open
wants to merge 3 commits into
base: master
Choose a base branch
from

Conversation

lalten
Copy link

@lalten lalten commented Oct 1, 2022

Actions using the amd64 and i686 arch will not actually use Qemu to run things.
But it makes user's lifes easier by removing the need for special-casing no emulation builds.

The Dockerfiles use the https://hub.docker.com/u/amd64/ organization's containers as base.

Closes #93.

@lalten
Copy link
Author

lalten commented Oct 1, 2022

Ah, I just found out about #58... @uraimo feel free to pick this or the other one I guess.

@lalten lalten changed the title Support x86_64 aka amd64 arch Support x86_64 and i686 architectures Oct 1, 2022
@lalten
Copy link
Author

lalten commented Oct 1, 2022

While I was at it I added i686 support as well now. Maybe you'll find this PR easier to merge than #58 as it has fewer changes on top of latest master and supports more distros.

@lalten lalten force-pushed the amd64-support branch 2 times, most recently from 6324b47 to e19aade Compare October 1, 2022 20:39
@lalten
Copy link
Author

lalten commented Oct 8, 2022

@uraimo friendly ping 🔔

@memark
Copy link

memark commented Oct 21, 2022

Would love to see this!

@thorrak
Copy link

thorrak commented Mar 23, 2023

I'd love to see this as well. I'm building for armv7 and aarch64 at the moment - being able to use a single action for x86_64 (and while I still support it, i686) would be great!

@lalten
Copy link
Author

lalten commented Aug 29, 2023

@uraimo I rebased this PR branch onto latest master and resolved all conflicts. Would be happy to hear your thoughts on this PR.

@maxexcloo
Copy link

Would love to use this also!

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.

Support amd64 as a no-op
4 participants