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

move opt/homebrew/lib ahead of /usr/local/lib #941

Closed

Conversation

davidperched
Copy link

This PR moves /opt/homebrew/lib ahead of /usr/local/lib in the library lookup directories ordering.

The reason is this:
/opt/homebrew/lib is the default path for homebrew on M1 macs
/usr/local/lib is the defualt path for homebrew on Intel Macs

For most users, this change won't do anything, because overwhelmingly only one of those folders will exist.

there is a small intersection of users who have both. the reason is often because they need both m1 and x86 versions of homebrew installed because of some dependency that is only available on x86 (i.e. openvino).

for those users, this would prefer the platform native m1 homebrew ahead of the x86 homebrew path.

so to summarize
intel macs can only have /usr/local/lib
most m1 macs will only have /opt/homebrew/lib
a small number of m1 mac users will have both. this change affects them and it prefers the platform native path to the x86 path

@southwolf
Copy link

Hi @davidperched I believe this PR is duplicated with my previous one #936

@DanielMilstein
Copy link

Hi @davidperched I believe this PR is duplicated with my previous one #936

Ended up here because I came up with the same solution and wanted to know if anyone else had made a pull request about it. Would it help if I made another one in terms of letting the maintainers know it's an issue?

@eregon
Copy link
Collaborator

eregon commented Jun 27, 2022

This is actually slightly better than #941, at least it only moves /opt/homebrew/lib and leaves the rest alone.
#965 seems better by having a darwin-aarch64 handling explicitly.
Also see #880 (comment)

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

5 participants