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

Homebrew Cask's default Caskroom location has changed #100

Closed
jeroenj opened this issue Jun 15, 2016 · 5 comments
Closed

Homebrew Cask's default Caskroom location has changed #100

jeroenj opened this issue Jun 15, 2016 · 5 comments

Comments

@jeroenj
Copy link
Contributor

jeroenj commented Jun 15, 2016

As per Homebrew/homebrew-cask#21857 the default location for the Caskroom has changed to /usr/local/Caskroom instead of /opt/homebrew-cask/Caskroom.

The cask recipe will always create the old directory: https://github.com/chef-cookbooks/homebrew/blob/ab010defa6d3cc4cbf9c83c4c097f6ac87ef7e1f/recipes/cask.rb#L29-L38

As long as the old directory exists cask will show a warning that you should migrate to the new location but it will keep using the old location. Since this recipe always creates the old location cask will keep on installing in the old location even if you have migrated to the new one.

One option would be to just change the directories in the cask recipe but I'm actually not sure if those directories should be created by that recipe. I'd think that cask would create them if they don't exist yet. But I'd have to investigate that.

seanfisk added a commit to seanfisk/personal-chef-repo that referenced this issue Jun 26, 2016
Disable installation of Casks while these issues are being worked out:

- sous-chefs/homebrew#70
- sous-chefs/homebrew#100
@wendorf
Copy link
Contributor

wendorf commented Aug 18, 2016

This should be fixed by #103

@jobster
Copy link

jobster commented Aug 22, 2016

This is not fixed in the latest version available (2.1.0)! @wendorf can you please release a version that includes this change? 2.1.0 is now 7 months old, and it would be really nice if I didn't have to change my Cheffile so that it referenced tip of master explicitly.

Thanks!

@wendorf
Copy link
Contributor

wendorf commented Aug 23, 2016

Sorry, @jobster. I don't control this repo. I echo the sentiment, though.

@aramprice
Copy link

Is there an ETA on releasing a version of the cookbook with the fix from #103?

@tas50
Copy link
Contributor

tas50 commented Sep 6, 2016

This has been pushed up to Supermarket

@tas50 tas50 closed this as completed Sep 6, 2016
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

No branches or pull requests

5 participants