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

homebrew_bootsnap: fail more gracefully. #10538

Merged
merged 1 commit into from Feb 5, 2021

Conversation

MikeMcQuaid
Copy link
Member

If bootsnap isn't available then print out an error message rather than failing all of Homebrew.

@MikeMcQuaid MikeMcQuaid added the critical Critical change which should be shipped as soon as possible. label Feb 5, 2021
@BrewTestBot
Copy link
Member

Review period skipped due to critical label.

If `bootsnap` isn't available then print out an error message rather
than failing all of Homebrew.
Copy link
Member

@carlocab carlocab left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Seems to work:

❯ brew help
Error: HOMEBREW_BOOTSNAP could not `require "bootsnap"`!

Example usage:
  brew search [TEXT|/REGEX/]
  brew info [FORMULA...]
  brew install FORMULA...
  brew update
  brew upgrade [FORMULA...]
  brew uninstall FORMULA...
  brew list [FORMULA...]

I had to break homebrew_bootsnap.rb to get it to produce an error, though.

@MikeMcQuaid MikeMcQuaid merged commit 15ae53c into Homebrew:master Feb 5, 2021
@MikeMcQuaid MikeMcQuaid deleted the bootsnap_graceful_failure branch February 5, 2021 17:44
@BrewTestBot BrewTestBot added the outdated PR was locked due to age label Mar 8, 2021
@Homebrew Homebrew locked as resolved and limited conversation to collaborators Mar 8, 2021
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
critical Critical change which should be shipped as soon as possible. outdated PR was locked due to age
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

3 participants