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 for issues in importing the tap #6

Open
wants to merge 1 commit into
base: master
Choose a base branch
from

Conversation

gmerlino
Copy link

@gmerlino gmerlino commented Aug 24, 2023

Unless applying this fix for #5, now the tap cannot even be imported..

It looks like Brew has become more strict in checking syntax errors in (imported) formulas, as the following are errors I get when trying to (unsuccessfully) tap this repo (after having untapped it):

$ brew tap sdkman/tap
==> Tapping sdkman/tap
Cloning into '/opt/homebrew/Library/Taps/sdkman/homebrew-tap'...
remote: Enumerating objects: 55, done.
remote: Counting objects: 100% (55/55), done.
remote: Compressing objects: 100% (47/47), done.
remote: Total 55 (delta 16), reused 39 (delta 0), pack-reused 0
Receiving objects: 100% (55/55), 10.36 KiB | 3.45 MiB/s, done.
Resolving deltas: 100% (16/16), done.
Error: Invalid formula (sonoma): /opt/homebrew/Library/Taps/sdkman/homebrew-tap/Formula/sdkmancli.rb
No available formula with the name "sdkmancli". Did you mean sdkman-cli?
In formula file: /opt/homebrew/Library/Taps/sdkman/homebrew-tap/Formula/sdkmancli.rb
Expected to find class Sdkmancli, but only found: SdkmanCli.
Error: Invalid formula (arm64_sonoma): /opt/homebrew/Library/Taps/sdkman/homebrew-tap/Formula/sdkmancli.rb
No available formula with the name "sdkmancli". Did you mean sdkman-cli?
In formula file: /opt/homebrew/Library/Taps/sdkman/homebrew-tap/Formula/sdkmancli.rb
Expected to find class Sdkmancli, but only found: SdkmanCli.
Error: Invalid formula (ventura): /opt/homebrew/Library/Taps/sdkman/homebrew-tap/Formula/sdkmancli.rb
No available formula with the name "sdkmancli". Did you mean sdkman-cli?
In formula file: /opt/homebrew/Library/Taps/sdkman/homebrew-tap/Formula/sdkmancli.rb
Expected to find class Sdkmancli, but only found: SdkmanCli.
Error: Invalid formula (arm64_ventura): /opt/homebrew/Library/Taps/sdkman/homebrew-tap/Formula/sdkmancli.rb
No available formula with the name "sdkmancli". Did you mean sdkman-cli?
In formula file: /opt/homebrew/Library/Taps/sdkman/homebrew-tap/Formula/sdkmancli.rb
Expected to find class Sdkmancli, but only found: SdkmanCli.
Error: Invalid formula (monterey): /opt/homebrew/Library/Taps/sdkman/homebrew-tap/Formula/sdkmancli.rb
No available formula with the name "sdkmancli". Did you mean sdkman-cli?
In formula file: /opt/homebrew/Library/Taps/sdkman/homebrew-tap/Formula/sdkmancli.rb
Expected to find class Sdkmancli, but only found: SdkmanCli.
Error: Invalid formula (arm64_monterey): /opt/homebrew/Library/Taps/sdkman/homebrew-tap/Formula/sdkmancli.rb
No available formula with the name "sdkmancli". Did you mean sdkman-cli?
In formula file: /opt/homebrew/Library/Taps/sdkman/homebrew-tap/Formula/sdkmancli.rb
Expected to find class Sdkmancli, but only found: SdkmanCli.
Error: Invalid formula (big_sur): /opt/homebrew/Library/Taps/sdkman/homebrew-tap/Formula/sdkmancli.rb
No available formula with the name "sdkmancli". Did you mean sdkman-cli?
In formula file: /opt/homebrew/Library/Taps/sdkman/homebrew-tap/Formula/sdkmancli.rb
Expected to find class Sdkmancli, but only found: SdkmanCli.
Error: Invalid formula (arm64_big_sur): /opt/homebrew/Library/Taps/sdkman/homebrew-tap/Formula/sdkmancli.rb
No available formula with the name "sdkmancli". Did you mean sdkman-cli?
In formula file: /opt/homebrew/Library/Taps/sdkman/homebrew-tap/Formula/sdkmancli.rb
Expected to find class Sdkmancli, but only found: SdkmanCli.
Error: Invalid formula (catalina): /opt/homebrew/Library/Taps/sdkman/homebrew-tap/Formula/sdkmancli.rb
No available formula with the name "sdkmancli". Did you mean sdkman-cli?
In formula file: /opt/homebrew/Library/Taps/sdkman/homebrew-tap/Formula/sdkmancli.rb
Expected to find class Sdkmancli, but only found: SdkmanCli.
Error: Invalid formula (mojave): /opt/homebrew/Library/Taps/sdkman/homebrew-tap/Formula/sdkmancli.rb
No available formula with the name "sdkmancli". Did you mean sdkman-cli?
In formula file: /opt/homebrew/Library/Taps/sdkman/homebrew-tap/Formula/sdkmancli.rb
Expected to find class Sdkmancli, but only found: SdkmanCli.
Error: Invalid formula (high_sierra): /opt/homebrew/Library/Taps/sdkman/homebrew-tap/Formula/sdkmancli.rb
No available formula with the name "sdkmancli". Did you mean sdkman-cli?
In formula file: /opt/homebrew/Library/Taps/sdkman/homebrew-tap/Formula/sdkmancli.rb
Expected to find class Sdkmancli, but only found: SdkmanCli.
Error: Invalid formula (sierra): /opt/homebrew/Library/Taps/sdkman/homebrew-tap/Formula/sdkmancli.rb
No available formula with the name "sdkmancli". Did you mean sdkman-cli?
In formula file: /opt/homebrew/Library/Taps/sdkman/homebrew-tap/Formula/sdkmancli.rb
Expected to find class Sdkmancli, but only found: SdkmanCli.
Error: Invalid formula (el_capitan): /opt/homebrew/Library/Taps/sdkman/homebrew-tap/Formula/sdkmancli.rb
No available formula with the name "sdkmancli". Did you mean sdkman-cli?
In formula file: /opt/homebrew/Library/Taps/sdkman/homebrew-tap/Formula/sdkmancli.rb
Expected to find class Sdkmancli, but only found: SdkmanCli.
Error: Invalid formula (x86_64_linux): /opt/homebrew/Library/Taps/sdkman/homebrew-tap/Formula/sdkmancli.rb
No available formula with the name "sdkmancli". Did you mean sdkman-cli?
In formula file: /opt/homebrew/Library/Taps/sdkman/homebrew-tap/Formula/sdkmancli.rb
Expected to find class Sdkmancli, but only found: SdkmanCli.
Error: Cannot tap sdkman/tap: invalid syntax in tap!

@gmerlino gmerlino changed the title Fix for https://github.com/sdkman/homebrew-tap/issues/5 Fix for issues in importing the tap Aug 24, 2023
@colindean
Copy link

This might need to be fixed somehow in https://github.com/sdkman/sdkman-cli/blob/master/src/jreleaser/distributions/sdkman-cli/brew/formula.rb.tpl. I think something is awry with the jreleaser config.

Until then, the sdkmancli formula that installs 5.18.2 won't work. The sdkman-cli formula that installs 5.18.1 works.

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 this pull request may close these issues.

2 participants