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

Prefer command line tools SDK on macOS #5624

Merged
merged 1 commit into from Jul 29, 2021

Conversation

radarhere
Copy link
Member

Resolves #5622

Copy link
Member

@hugovk hugovk left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

I'm still on macOS Mojave 10.14.6, and this builds fine and passes tests.

(I do have /Library/Developer/CommandLineTools/SDKs/MacOSX.sdk)

@hugovk hugovk merged commit e276655 into python-pillow:master Jul 29, 2021
@radarhere radarhere deleted the macos_zlib branch July 29, 2021 06:52
@h-vetinari
Copy link

Hey all

Just wanted to let you know that this broke the build process in conda-forge. There, we rely on the fact that we can control the MacOS SDK being used (which is explicitly set up by conda-build), but with this patch, the sdk from the system gets preference and ends up breaking (mismatched compiler vs. lib).

I reverted it to get conda-forge/pillow-feedstock#101 out the door, but carrying patches indefinitely is not great.

Would you like me to open another issue to discuss this?

@radarhere
Copy link
Member Author

A new issue would be good, thanks.

@h-vetinari
Copy link

Done: #5827

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

Building with Homebrew llvm/clang fails with error: 'TARGET_OS_IPHONE' is not defined
3 participants