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

Configure: Do Not Override Default Deployment Target #366

Closed
bdkjones opened this issue Nov 21, 2020 · 3 comments
Closed

Configure: Do Not Override Default Deployment Target #366

bdkjones opened this issue Nov 21, 2020 · 3 comments

Comments

@bdkjones
Copy link
Contributor

The configure script, on lines 351-354, forces -mmacosx-version-min=10.7

This should not be done if the $MACOSX_DEPLOYMENT_TARGET environment variable is set. If it is, THAT value should always take precedence, because it's highly likely that the dependent libraries (libpng, libz, etc.) were built with the deployment target specified globally. The Configure script currently gets into a situation where the dependent libraries might have a deployment target that is higher than the 10.7 value PNGQuant is forcing.

@bdkjones
Copy link
Contributor Author

Apple's docs suggest that the ENV value should override -mmacosx-version-min, but after building I'm getting warnings that libpng and libz were built for newer versions than 10.7.

@kornelski
Copy link
Owner

Ok, I've made it unset if MACOSX_DEPLOYMENT_TARGET is set. I've also bumped minimum sdk to 10.9.

@bdkjones
Copy link
Contributor Author

bdkjones commented Nov 22, 2020 via email

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

2 participants