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

Ability to specify "linkables" in a Cask #143

Closed
phinze opened this issue Jan 28, 2013 · 2 comments
Closed

Ability to specify "linkables" in a Cask #143

phinze opened this issue Jan 28, 2013 · 2 comments

Comments

@phinze
Copy link
Contributor

phinze commented Jan 28, 2013

I often find when running linkapps that I end up with a bunch of cruft in /Applications - stuff like Uninstall Foo.app and crash_reporter.app.

I'm thinking if there was a way to specify which apps are supposed to be linked that would allow us to clean that up.

Just have to come up with some decent syntax for this.

Here's a first cut to start the conversation

class Steam < Cask
  url '...'
  # ...

  app 'Steam.app'
end

And specifying multiple lines is additive? I welcome thoughts on the method name, not sure I'm sold on this one yet.

@passcod
Copy link
Contributor

passcod commented Jan 28, 2013

I did it in #114 like this (iirc):

link :app, 'Steam.app'

because I then extended it thus:

link :prefpane, 'some.prefpane'

...but I wasn't too pleased with the naming myself :) Naming is hard!


Also, yeah, multiple lines is additive, but allow for multiple args as well? Like:

link :app, 'A.app'
link :app, 'B.app'

# is equivalent to:

link :app, 'A.app', 'B.app'

...now I'm not sure anymore if that's a good idea.

phinze added a commit that referenced this issue Apr 5, 2013
 - abiliity to specify linkables #143 #105
 - use bona fide mac aliases for spotlight compat #99
 - started on specifying proper linkables for a subset of apps
@phinze
Copy link
Contributor Author

phinze commented Apr 12, 2013

Landed! ✈️

@phinze phinze closed this as completed Apr 12, 2013
faun pushed a commit to faun/homebrew-cask that referenced this issue Jun 15, 2014
…ntainlion

sha256 checksum in macvim-mountainlion.rb
@Homebrew Homebrew locked and limited conversation to collaborators May 8, 2018
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
None yet
Projects
None yet
Development

No branches or pull requests

3 participants