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

brew tap fails with invalid formula due to disabled bottle :unneeded #81

Closed
eli-schwartz opened this issue Mar 11, 2022 · 3 comments · Fixed by #82
Closed

brew tap fails with invalid formula due to disabled bottle :unneeded #81

eli-schwartz opened this issue Mar 11, 2022 · 3 comments · Fixed by #82

Comments

@eli-schwartz
Copy link
Contributor

bottle :unneeded

bottle :unneeded

This becomes invalid in Homebrew/brew#12259 and as a result it is no longer possible to tap this brew.

The following error is produced:

==> Tapping cartr/qt4
Cloning into '/usr/local/Homebrew/Library/Taps/cartr/homebrew-qt4'...
Error: Invalid formula: /usr/local/Homebrew/Library/Taps/cartr/homebrew-qt4/treeline.rb
treeline: Calling bottle :unneeded is disabled! There is no replacement.
Please report this issue to the cartr/qt4 tap (not Homebrew/brew or Homebrew/core):
  /usr/local/Homebrew/Library/Taps/cartr/homebrew-qt4/treeline.rb:8

Error: Invalid formula: /usr/local/Homebrew/Library/Taps/cartr/homebrew-qt4/qbzr.rb
qbzr: Calling bottle :unneeded is disabled! There is no replacement.
Please report this issue to the cartr/qt4 tap (not Homebrew/brew or Homebrew/core):
  /usr/local/Homebrew/Library/Taps/cartr/homebrew-qt4/qbzr.rb:8

Error: Cannot tap cartr/qt4: invalid syntax in tap!

Found in https://github.com/mesonbuild/meson/runs/5504514413?check_suite_focus=true#step:6:1

@cartr
Copy link
Owner

cartr commented Mar 11, 2022 via email

@eli-schwartz
Copy link
Contributor Author

Sadly, I don't use macOS except in the sense that a project I work on has macOS CI which someone else set up. So I cannot test it either. :(

(I can do a lot of investigative work and digging to find the cause of the problem, but that's about my limit. I'm not even sure I fully understand what this formula statement does.)

Based on following links from that ticket and Homebrew/brew#12913 to other projects encountering the same issue and seeing how they resolved it, simply removing those lines may be the right solution.

@eli-schwartz
Copy link
Contributor Author

eli-schwartz commented Mar 11, 2022

Alright, I tried to test this out by removing those lines and pointing the Meson CI at my fork, results are here: https://github.com/eli-schwartz/meson/runs/5513240977?check_suite_focus=true

The error is gone, qt4 is successfully installed and the CI that tries to build a program against Qt4 now passes.

My hope is that that means my fix is correct. :D

@cartr cartr closed this as completed in #82 Mar 11, 2022
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging a pull request may close this issue.

2 participants