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

Pumping MACOSX_DEPLOYMENT_TARGET #353

Closed
wants to merge 1 commit into from

Conversation

mys721tx
Copy link
Contributor

Pumps MACOSX_DEPLOYMENT_TARGET to 10.9. Xcode 10 on macOS 10.14 warns of libstdc++ deperation with the current target (10.7); the warning can be fixed by pumping the target version.

The last release of Mac OS X 10.7 is 6 years old and according to StatCounter, OS X 10.9 and later makes up for more than 95% of all mac OS machine online. Per OpenMathLib/OpenBLAS#1580, 10.9 should be a good minimal release target.

Pumps MACOSX_DEPLOYMENT_TARGET to 10.9. Xcode 10 on macOS 10.14 warns of
libstdc++ deperation with the current target (10.7); the warning can be
fixed by pumping the target version. The last release of Mac OS X 10.7
is 6 years old and according to
[StatCounter](http://gs.statcounter.com/os-version-market-share/macos/desktop/worldwide),
OS X 10.9 and later makes up for more than 95% of all mac OS machine online. Per OpenMathLib/OpenBLAS#1580, 10.9 should be a good minimal release target.
@torognes
Copy link
Owner

What do we gain by this except from the removal of the warning?

@mys721tx
Copy link
Contributor Author

At some point in the future, mac OS will no longer ship with libstdc++. Switching would allow future compatibility with minimal impact of current user.

libc++ also, among other things, has a more performant standard library hashmap implementations.

@torognes
Copy link
Owner

The MACOSX_DEPLOYMENT_TARGET has been raised to 10.9 in commit da41397 to avoid some problems when building the vsearch bioconda recipe.

@torognes torognes closed this Mar 18, 2019
@mys721tx mys721tx deleted the deployment-target branch March 18, 2019 16:51
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.

None yet

2 participants