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

fix issue with "end of file" #18

Merged
merged 1 commit into from
Jan 19, 2022
Merged

fix issue with "end of file" #18

merged 1 commit into from
Jan 19, 2022

Conversation

fredwangwang
Copy link
Contributor

Detailed issue:

can't connect to the PIN entry module '/opt/homebrew/opt/pinentry-touchid/bin/pinentry-touchid': End of file

Reason:

dig into the issue find out when gpg calls out to pinentry-touchi binary, it adds "--display /private/tmp/com.apple.launchd.MGdSfNPjrJ/org.xquartz:0" flag.

Because --display is an unrecognized flag for the binary, it errors out immediately. This then result in an unclear message returned from gpg: "No pinentry".

Fix:

adding the dummy --dispaly flag to the flag parser.

Detailed issue:

can't connect to the PIN entry module '/opt/homebrew/opt/pinentry-touchid/bin/pinentry-touchid': End of file

Reason:

dig into the issue find out when gpg calls out to pinentry-touchi binary, it adds "--display /private/tmp/com.apple.launchd.MGdSfNPjrJ/org.xquartz:0" flag. Because --display is an unrecognized flag for the binary, it errors out immediately. This then result in an unclear message returned from gpg: "No pinentry".

Fix:

adding the dummy --dispaly flag to the flag parser.
SunsetYe66 added a commit to SunsetYe66/pinentry-touchid that referenced this pull request Jan 19, 2022
@jorgelbg jorgelbg merged commit 1d7fbe6 into jorgelbg:main Jan 19, 2022
@plandes
Copy link

plandes commented Apr 11, 2022

This fix appears to have not yet been released (I see only a Dec latest release and the brew version seems even older). @jorgelbg, will you please release it?

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

Successfully merging this pull request may close these issues.

3 participants