-
-
Notifications
You must be signed in to change notification settings - Fork 10.8k
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
music-manager.rb: do not move prefpane #21659
Conversation
See #20405:
|
@tjanson Would a |
@tjanson I was going to suggest what @tsparber just mentioned. As to why I’m removing this, see #21657 (comment). Since we now move instead of linking, this current cask essentially modifies the |
@@ -10,4 +10,6 @@ | |||
# Renamed for consistency: app name is different in the Finder and in a shell. | |||
# Original discussion: https://github.com/caskroom/homebrew-cask/pull/4282 | |||
app 'MusicManager.app', target: 'Music Manager.app' | |||
|
|||
uninstall delete: '~/Library/Preference Panes/MusicManager.prefPane' |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
This should be ~/Library/PreferencePanes/…
(no space), right?
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
I just used the path from man brew-cask
without checking. Turns out this needed some fixing (#21670).
Ah, cool to see that the move to move has been made. 👍 |
* music-manager.rb: do not move prefpane * remove prefpane on uninstall
Editing an existing cask
brew cask audit --download {{cask_file}}
is error-free.brew cask style --fix {{cask_file}}
left no offenses.Why would we be taking it? Presumably the app will take care of it.