Skip to content

Update sqlcl links#249134

Open
bamcgill wants to merge 7 commits intoHomebrew:mainfrom
bamcgill:update-sqlcl-links
Open

Update sqlcl links#249134
bamcgill wants to merge 7 commits intoHomebrew:mainfrom
bamcgill:update-sqlcl-links

Conversation

@bamcgill
Copy link
Contributor

@bamcgill bamcgill commented Feb 12, 2026

Important: Do not tick a checkbox if you haven’t performed its action. Honesty is indispensable for a smooth review process.

Updating sqlcl so that we always reference the latest install so a user does not have to set an environment variable each time.

In the following questions <cask> is the token of the cask you're submitting.

After making any changes to a cask, existing or new, verify:

Additionally, if adding a new cask:

  • Named the cask according to the token reference.
  • Checked the cask was not already refused (add your cask's name to the end of the search field).
  • brew audit --cask --new <cask> worked successfully.
  • HOMEBREW_NO_INSTALL_FROM_API=1 brew install --cask <cask> worked successfully.
  • brew uninstall --cask <cask> worked successfully.

If AI was used to generate or assist with generating the PR:

  • I used AI to generate or assist with generating this PR. Please specify below how you used AI to help you.
  • I have personally reviewed, tested and verified all changes/additions, including zap stanza paths.

… way, once the path is aset, all updates via brew will refresh the link and point to the proper release. @elhanafimohammedamine put this change together for us
… way, once the path is aset, all updates via brew will refresh the link and point to the proper release. @elhanafimohammedamine put this change together for us
… way, once the path is aset, all updates via brew will refresh the link and point to the proper release. @elhanafimohammedamine put this change together for us
Copy link
Contributor Author

@bamcgill bamcgill left a comment

Choose a reason for hiding this comment

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

This change is to set the latest downloaded version and link it to the latest reference in Caskroom/sqlcl/latest/sqlcl/bin so the user does not have to redo the environment variable for PATH each time we update.

@bevanjkay
Copy link
Member

Hi @bamcgill

my feedback on a similar change from December remains here;

I understand that the clash with sql makes this unable to be installed automatically to due to the conflict. But looking at the other package managers that have this available the binary is almost always linked as sqlcl, I think that is going to be our best course of action here, as postflight blocks are not tested, and are therefore more likely to break.

If necessary we could add a caveat making it clear that the binary is renamed.

@bevanjkay bevanjkay added the awaiting user reply Issue needs response from a user. label Feb 13, 2026
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

automerge-skip awaiting user reply Issue needs response from a user.

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants