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

Add linux armhf/arm64 support #8

Open
wants to merge 1 commit into
base: main
Choose a base branch
from
Open

Add linux armhf/arm64 support #8

wants to merge 1 commit into from

Conversation

ldgabet
Copy link

@ldgabet ldgabet commented Aug 15, 2023

build box86/box64 to run wine
install wine 7.0.2

build box86/box64 to run wine
install wine 7.0.2
@huan
Copy link
Owner

huan commented Aug 16, 2023

Thanks for the contribution!

Could you please fix the CI and add a section about the ARM version in REAMDE?

@tobiasdiez
Copy link

It seems like https://github.com/ldgabet/docker-box-wine is another version providing arm support. Would be nice to see this merged!

@huan
Copy link
Owner

huan commented Mar 24, 2024

@tobiasdiez, thanks for continuing to discuss this PR!

I'd love to add this support; however, I think it needs:

  1. Make the CI pass
  2. Add an introduction & documentation to the README so that other developers can understand how it works and how to use it

Because I have no experience running wine with ARM, I have very limited knowledge about this PR, which means I need more information to gain enough confidence to merge it, too.

It would be beneficial if you could help to test this PR and make sure everything is correct.

@tobiasdiez
Copy link

As I have only limited experience with wine (and docker in general), I'm not really in the position to help actively. But I'm happy to try out and test a new version.

As for the documentation, I think it follows relatively closely https://github.com/ptitSeb/box64/blob/main/docs/X64WINE.md.

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.

3 participants