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

Add -fix flag for automatically fixing the pinentry symlink #14

Merged
merged 2 commits into from
Jan 6, 2022

Conversation

jorgelbg
Copy link
Owner

When installing gnupg, homebrew automatically configures the path of the pinentry binary to the opt_bin path of the formula instead of the symlink in /usr/local/bin this means that actual path returned by gpgconf ends up looking like: /usr/local/opt/pinentry/bin/pinentry.

Fixing the symlink in /usr/local/bin doesn't always solve the issue because gpgconf is still reporting the /usr/local/opt path.

The new flag will automatically change this symlink to point to pinentry-mac.

I tried unsuccessfully to include this as part of the automatic Formula installation but it is not possible because during installation the formula cannot modify any external path.

When installing gnupg, homebrew automatically configures the path of the
pinentry binary to the `opt_bin` path of the formula instead of the
symlink in `/usr/local/bin`
https://github.com/Homebrew/homebrew-core/blob/08d0d1cc331d570a31d5ef89e695ad283c75d4f6/Formula/gnupg.rb#L54
this means that actual path returned by `gpgconf` ends up
looking like: /usr/local/opt/pinentry/bin/pinentry.

Fixing the symlink in `/usr/local/bin` doesn't always solve the issue
because `gpgconf` is still reporting the `/usr/local/opt` path.

The new flag changes automatically this symlink to point to
pinentry-mac.

I tried unsuccessfully to include this as part of the automatic Formula
installation but it is not possible because during installation the
formula cannot modify any external path.
@jorgelbg jorgelbg added the enhancement New feature or request label Dec 14, 2021
@jorgelbg jorgelbg merged commit 9f69cea into main Jan 6, 2022
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
enhancement New feature or request size/L
Projects
None yet
Development

Successfully merging this pull request may close these issues.

1 participant