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

allow building for Intel architecture #1086

Closed

Conversation

kambala-decapitator
Copy link

Don't see any reason why project can't be built for Intel arch. I understand that D3DM stuff won't work, but not all Windows programs/games need it. Tested the change successfully to run Diablo 2.

When building release, you'd have to pass ARCHS=arm64 to xcodebuild additionally, e.g.:

xcodebuild archive -scheme Whisky ARCHS=arm64

Alternatively, create xcconfig with contents ARCHS = arm64 and pass it to xcodebuild in -xcconfig param. (creating a new build configuration just for that is an overkill)

@IsaacMarovitz
Copy link
Member

We don't support Intel primarily because of differences in GPU limitations between the two architectures and our inability to offer support to regular end users. Technical individuals are expected to understand that they can build for Intel if they want to.

@kambala-decapitator
Copy link
Author

Technical individuals are expected to understand that they can build for Intel if they want to.

This is exactly the goal of the PR - to simplify this process. I'm not asking you to make releases with Intel support which should be clear from xcodebuild examples provided in description.

Alternatively, I can provide xcconfig file that restores Intel support.

@Gcenx
Copy link

Gcenx commented Aug 12, 2024

Building the GUI isn’t enough you’d also need to provide your own custom build of WhiskyWine that doesn’t bundle Apples D3DMetal since that only works on Apple Silicon.

There was someone who forked and made an Intel build but haven’t updated it since last year.

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