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

Search Homebrew's prefix on Apple Silicon #51

Merged
merged 1 commit into from
Mar 12, 2021

Conversation

mudge
Copy link
Owner

@mudge mudge commented Mar 12, 2021

GitHub: #50

To make installation easier for users of Homebrew on Apple Silicon, add /opt/homebrew to the default paths searched when trying to find the underlying re2 library. While doing so, add an extra fallback to /usr (instead of only searching /usr/local).

Note we still search /usr/local first to avoid accidentally changing behaviour for existing users (e.g. suddenly compiling against a different version of re2 in /usr).

ext/re2/extconf.rb Outdated Show resolved Hide resolved
ext/re2/extconf.rb Outdated Show resolved Hide resolved
ext/re2/extconf.rb Outdated Show resolved Hide resolved
@mudge mudge force-pushed the add-apple-silicon-homebrew-prefix-to-default-search branch from 5fec7b6 to 56ca0f6 Compare March 12, 2021 13:22
GitHub: #50

To make installation easier for users of Homebrew on Apple Silicon, add
/opt/homebrew to the default paths searched when trying to find the
underlying re2 library.  While doing so, add an extra fallback to /usr
(instead of only searching /usr/local).

Note we still search /usr/local first to avoid accidentally changing
behaviour for existing users (e.g. suddenly compiling against a
different version of re2 in /usr).
@mudge mudge force-pushed the add-apple-silicon-homebrew-prefix-to-default-search branch from 56ca0f6 to d1c4ad2 Compare March 12, 2021 13:27
@mudge mudge merged commit 9d6c9f0 into main Mar 12, 2021
@mudge mudge deleted the add-apple-silicon-homebrew-prefix-to-default-search branch March 12, 2021 13:32
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.

2 participants