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

Link binary for Karabiner-Elements #174245

Merged
merged 1 commit into from
May 19, 2024
Merged

Link binary for Karabiner-Elements #174245

merged 1 commit into from
May 19, 2024

Conversation

tshu-w
Copy link
Contributor

@tshu-w tshu-w commented May 19, 2024

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

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.
  • Checked the cask is submitted to the correct repo.
  • 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.

@@ -137,6 +137,8 @@

auto_updates true

binary "/Library/Application Support/org.pqrs/Karabiner-Elements/bin/karabiner_cli"
Copy link
Member

Choose a reason for hiding this comment

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

Does this work on the legacy (pre Big Sur) version? otherwise this should be scoped within the on big_sur :or_newer block.

Copy link
Contributor Author

@tshu-w tshu-w May 19, 2024

Choose a reason for hiding this comment

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

The karabiner_cli was introduced in this commit (Mar 5, 2017) and I confirmed that the oldest version in this file (11.6.0, released at Feb 20, 2018) also includes karabiner_cli in the same directory:

https://github.com/pqrs-org/Karabiner-Elements/blob/v11.6.0/make-package.sh#L33-L37

Copy link
Member

Choose a reason for hiding this comment

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

Thanks for confirming @tshu-w

@krehel krehel merged commit f47aa3c into Homebrew:master May 19, 2024
15 checks passed
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.

None yet

3 participants