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

Mac: Remove universal binary support. #39

Open
rryan opened this issue Apr 20, 2018 · 9 comments
Open

Mac: Remove universal binary support. #39

rryan opened this issue Apr 20, 2018 · 9 comments

Comments

@rryan
Copy link
Member

rryan commented Apr 20, 2018

Now that macOS is pretty much all amd64, we don't need to make universal PPC/i386/amd64 builds, so we can remove a bunch of code that supports that.

@Pegasus-RPG
Copy link
Member

Up to what age of machines do we want to support?

@Pegasus-RPG
Copy link
Member

Can we check the Web logs to get an idea of how many people on non-amd64 Macs visit the site at least? Or do we have metrics of how many non-amd64 ones click the download link?

@Be-ing
Copy link
Contributor

Be-ing commented Apr 20, 2018

What is the cost of continuing to build universal binaries?

@rryan
Copy link
Member Author

rryan commented Apr 20, 2018

We dropped x86 and PPC support years ago (1.11?), so this is just removing the vestigial code that supported building a single binary for all 3.

@rryan
Copy link
Member Author

rryan commented Apr 20, 2018

We require macOS 10.8, which doesn't support x86 or PPC.

@Be-ing
Copy link
Contributor

Be-ing commented Apr 20, 2018

Okay, then let's remove the vestigial code 👍

@Pegasus-RPG
Copy link
Member

We can always pull it out of the history if we ever need it again, right? If so, go ahead.

@ywwg
Copy link
Member

ywwg commented Apr 20, 2018

the most likely future scenarios are mac remains 100% x64, or they come out with a line of ARM macs and we need a universal binary that's x64 and ARM

@ywwg
Copy link
Member

ywwg commented Apr 20, 2018

(further prediction: current macOS libs (carbon/cocoa whatever) will be compatibility mode, and the main UI will move toward iOS-on-desktop)

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

No branches or pull requests

4 participants