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

Distinguish between apps that auto-update and those marked as latest #53

Merged
merged 2 commits into from
Apr 16, 2017

Conversation

F30
Copy link
Contributor

@F30 F30 commented Mar 26, 2017

As a follow-up to #49, this adds a separation between apps that auto-update and those that are marked as latest through new command line options: --all now includes apps that auto-update, while --force is (additionally or exclusively) required for apps marked as latest.

The rationale behind this is to allow upgrades of apps that auto-update without always reinstalling such marked as latest. The wish for that has also been brought up by @leipert in the discussion on Homebrew/homebrew-cask#29301.
I can see three major reasons why one would want this:

  • Bringing all apps (or at least as many as possible) to a current version without having to launch each of them
  • Disallowing apps to "call home" through update checks for privacy concerns
  • Using a non-admin user account for daily work, such that apps cannot necessarily write themselves

As for the command line options, the semantics of --all are of course changed yet again, but I consider this one the most intuitive variant. However, I care much less for the names than I do for the feature in general 🙂.

F30 added 2 commits March 26, 2017 18:09
Remove duplicate code for outputting tables of apps and identification
of an app's state.
As a follow-up to 40baf90, this adds a separation between apps that
auto-update and those that are marked as latest through new command line
options: `--all` now includes apps that auto-update, while `--force` is
(additionally or exclusively) required for apps marked as latest.

The rationale behind this is to allow upgrades of apps that auto-update
without always reinstalling such marked as latest.
@F30 F30 force-pushed the all-force-separation branch from 15ccfa6 to aec085a Compare March 26, 2017 16:10
@leipert
Copy link
Contributor

leipert commented Mar 26, 2017

Maybe you can leave --all as it is right now, but simply add --auto and --latest

--all: shorthand for --latest and --auto
--auto: also include as auto-update marked casks
--latest: also include as latest marked casks

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

Successfully merging this pull request may close these issues.

3 participants