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

libgcrypt: temporarily disable #69987

Closed

Conversation

SeekingMeaning
Copy link
Contributor

While #69980 and #69983 are building, temporarily disable libgcrypt so users don't upgrade to 1.9.0. If this PR is merged, #69980/#69983 may have to be manually merged with brew pr-pull

@BrewTestBot BrewTestBot added the formula disabled Formula disabled label Jan 29, 2021
@SMillerDev
Copy link
Member

Let's just merge either one, those couple of minutes aren't going to matter much

@SeekingMeaning
Copy link
Contributor Author

libgcrypt 1.9.0 #69327 took a few hours

@carlocab
Copy link
Member

Won't this break all the formulae that depend on libgcrypt? There are a lot of them.

@fxcoudert
Copy link
Member

Disabling is not the right solution. We'll ship a new version, patch or roll-back ASAP, but disabling would be wrong.

@fxcoudert
Copy link
Member

Many many thanks @SeekingMeaning for spotting this, and exploring our different options!

@BrewTestBot BrewTestBot added the outdated PR was locked due to age label Mar 1, 2021
@Homebrew Homebrew locked as resolved and limited conversation to collaborators Mar 1, 2021
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
formula disabled Formula disabled outdated PR was locked due to age
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

5 participants