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

Sparkle's finish_installation.app gets linked sometimes #257

Closed
Crazor opened this issue May 3, 2013 · 3 comments
Closed

Sparkle's finish_installation.app gets linked sometimes #257

Crazor opened this issue May 3, 2013 · 3 comments

Comments

@Crazor
Copy link
Contributor

Crazor commented May 3, 2013

For some apps, e.g. Bartender, the finish_installation.app gets linked into ~/Applications. For others, e.g. Adium, this does not happen.

There is a line in Adium's cask to prevent this, but I feel this should be blacklisted. Sparkle is a pretty common framework, so this will happen with many apps. Also, linking .app bundles from inside an application's bundle doesn't sound that useful, and linking .app bundles from inside an application bundle's frameworks sounds completely useless to me.

So maybe .app bundles shouldn't be traversed at all.

@phinze
Copy link
Contributor

phinze commented May 3, 2013

Good point. brew-cask does a pretty dumb traversal when specifically linked apps are not mentioned. I like the idea to prune the walk at .app bundles.

@shrx
Copy link

shrx commented May 7, 2013

Also happens with BetterTouchTool:

==> Success! bettertouchtool installed to /opt/homebrew-cask/Caskroom/bettertouchtool/latest
==> Linking BetterTouchTool.app to /Applications/BetterTouchTool.app
==> Linking finish_installation.app to /Applications/finish_installation.app
==> Linking BTTHelper.app to /Applications/BTTHelper.app

Edit: I'm not sure BTTHelper.app should be linked here either, it wasn't here before.

phinze added a commit that referenced this issue May 12, 2013
@phinze
Copy link
Contributor

phinze commented May 12, 2013

This will get taken care of with explicit app linking which is the path we've decided to take, so I'm going to close this issue.

@phinze phinze closed this as completed May 12, 2013
faun pushed a commit to faun/homebrew-cask that referenced this issue Jun 15, 2014
@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