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

Merge this tool into homebrew-cask #23

Closed
ssreekanth opened this issue Oct 31, 2016 · 4 comments
Closed

Merge this tool into homebrew-cask #23

ssreekanth opened this issue Oct 31, 2016 · 4 comments

Comments

@ssreekanth
Copy link

ssreekanth commented Oct 31, 2016

I believe this tool should be part of homebrew-cask. Any thoughts?

@buo
Copy link
Owner

buo commented Nov 1, 2016

Since 2013, brew cask upgrade has been discussed many times over on the Homebrew Cask issues. Many people have made their own scripts, such as Homebrew/homebrew-cask#23658 and these. They are all bash scripts and fragile. This tool is written in Ruby, but it is likewise fragile and not that stable as mentioned here (now this tool does update :latest things with --all option).

Of course, it would be very nice if this tool were merged into the Homebrew Cask, but I do think we need to make this stable, as well as add more features such as supporting --appdir option.

@jcrben
Copy link

jcrben commented Nov 7, 2016

If someone can explain the problem with :latest succinctly, I'm interested in hearing it... so I guess the issue is that :latest is used when there isn't a published version as noted in the instructions - can't we check against the sha256? as that will be updated as a new cask is uploaded?

I started adding a check for a matching sha (see my branch) it but it seems that the :no_check is pretty common - all of my unversioned packages lack a checksum. Still, it looks like the cask team will be moving towards more sha256 in the future, with a --require-sha option coming down the pipe.

also I'd like to see a quick way to upgrade one at a time through this rather than going through the whole install force thing.

@RandomDSdevel
Copy link

@buo:

     brew cask proper has actually had an upgrade sub-command since Homebrew/brew#3396 was merged into upstream's master branch with commit f50ae44, so I believe this issue can now be closed. What that means for the continued development and maintenance of this project remains up to you locals to decide.

@ondrejfuhrer
Copy link
Collaborator

Closing this, will probably be never merged, already discussed with Homebrew maintainers (#95 (comment))

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