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.

Sign up for GitHub

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

Jump to bottom

gpg: minor search priority tweaks #3032

Merged
merged 2 commits into from
Aug 9, 2017
Merged

gpg: minor search priority tweaks #3032

merged 2 commits into from
Aug 9, 2017

Conversation

DomT4
Copy link
Member

@DomT4 DomT4 commented Aug 9, 2017 •
edited
Loading

  • Have you followed the guidelines in our Contributing document?
  • Have you checked to ensure there aren't other open Pull Requests for the same change?
  • Have you added an explanation of what your changes do and why you'd like us to include them?
  • Have you successfully run brew tests with your changes locally?

Makes more sense this way around now Homebrew has migrated across to 2.1.x & the latest version of that doesn't install as gpg2 any more.

Doesn't need to be merged before this PR but for the sake of referencing the PR introducing the upstream death of the gpg2 executable is: Homebrew/homebrew-core#16580

@DomT4
gpg: flip search priority order
ca9b8be 
Makes more sense this way around now Homebrew has migrated across
to 2.1.x & the latest version of that doesn't install as `gpg2`
any more.

Doesn't need to be merged before this PR but for the sake of referencing
the PR introducing the upstream death of the `gpg2` symlink is:
Homebrew/homebrew-core#16580
@DomT4
gpg2_requirement: flip search priority
0762cc6
@MikeMcQuaid
Copy link
Member

Cheers again, @DomT4!

@MikeMcQuaid MikeMcQuaid merged commit 01af27f into Homebrew:master Aug 9, 2017
@DomT4 DomT4 deleted the gpg_tweaks branch August 10, 2017 00:30
@DomT4
Copy link
Member Author

DomT4 commented Aug 10, 2017

Thanks for merging here & the other PR. Appreciated as ever ❤️

@Homebrew Homebrew locked and limited conversation to collaborators May 4, 2018
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Reviewers

@MikeMcQuaid MikeMcQuaid MikeMcQuaid approved these changes

Assignees
No one assigned
Labels
None yet
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.
2 participants
@DomT4 @MikeMcQuaid