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

Support Homebrew libraries on M1 Macs #93583

Closed
wants to merge 1 commit into from

Conversation

sebikeller
Copy link

Homebrew installs to /opt/homebrew/lib on M1 Macs, instead of /usr/local/lib on Intel Macs.
Without disabling SIP (to set DYLD_FALLBACK_LIBRARY_PATH in env) or manually changing the DEFAULT_LIBRARY_FALLBACK before loading any third-party libraries (as proposed here: Kozea/CairoSVG#354 (comment)), libraries installed with homebrew can not be loaded.

Homebrew installs to `/opt/homebrew/lib` on M1 Macs, instead of `/usr/local/lib` on Intel Macs.
Without disabling SIP (to set DYLD_FALLBACK_LIBRARY_PATH in env) or manually changing the `DEFAULT_LIBRARY_FALLBACK` before loading any third-party libraries (as proposed here: Kozea/CairoSVG#354 (comment)), libraries installed with homebrew can not be loaded.
@cpython-cla-bot
Copy link

cpython-cla-bot bot commented Jun 7, 2022

All commit authors signed the Contributor License Agreement.
CLA signed

@bedevere-bot
Copy link

Most changes to Python require a NEWS entry.

Please add it using the blurb_it web app or the blurb command-line tool.

@ned-deily
Copy link
Member

What's the use case? I'm not sure why we should special-case Homebrew here. What about other package managers like MacPorts? And that is only the default installation location: users can install Homebrew from source to arbitrary locations.

@ronaldoussoren
Copy link
Contributor

This needs PR needs an issue and a clear description of the problem it tries to solve. In particular: How can I reproduce the problems hat this PR tries to solve?

Without context I'm -1 on merging because the PR changes a search path that matches the one used by the system. And as @ned-deily noted in an earlier message this special cases one particular package manager.

@hugovk hugovk added the pending The issue will be closed if no feedback is provided label Jan 7, 2023
@kumaraditya303
Copy link
Contributor

Create an issue to discuss the change first as suggested above.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
awaiting review OS-mac pending The issue will be closed if no feedback is provided
Projects
None yet
Development

Successfully merging this pull request may close these issues.

7 participants