don't reinstall already installed casks

[ccutrer]

No description provided.

[passcod]

Given there's not upgrade command, this is actually a non-feature.

[ccutrer]

Not really. If there is a new version of the formula, installed? will return false (because it checks if the directory for the current version exists), and thus install will continue as normal.

[passcod]

True. Mmm, I don't remember how exactly installed? is defined, does it take into account "devel" formulae and installs? If it does, then alright. Otherwise, this would need more thought, as "devel" / HEAD formulae (or whatever we call them) don't have versioning and installing should really just fetch the latest version at the time it's run.

[phinze]

I am 👍 on this idea, but I think @passcod may be right that this kills an upgrade path for Casks that use the latest convention, e.g. rdio. Because the destination path never changes, the guard will always fire.

Maybe this could be acceptable if we added a --force option to push past it.

We'd also need some tests around the behavior before we could pull it in.

Lemme know if this makes sense - I think with some tests and an override option we'd be cool to pull this in.

[mathiasbynens]

In the meantime I use this workaround:

brew tap phinze/homebrew-cask
brew install brew-cask

function installcask() {
    if brew cask info "${@}" | grep "Not installed" > /dev/null; then
        brew cask install "${@}"
    else
        echo "${@} is already installed."
    fi
}

installcask foo
installcask bar
# etc.

[passcod]

Maybe we could have a check to see if the cask is using 'latest' (and hardcode that to start with; more semantic/flexible checking would be available with #142): let it through in this case, and raise otherwise.

[phinze]

Hey this is all done in 17026b6 - install gets a --force option to override the check. Going to close this - feel free to reopen if this doesn't work for you. 🐼