diff options
| author | Matt Di Pasquale | 2013-01-19 19:45:18 -0500 |
|---|---|---|
| committer | Mike McQuaid | 2013-01-21 19:55:09 +0000 |
| commit | e9158ca6dc99e3bb7ae475bee98153e78cafa2b0 (patch) | |
| tree | a3f67dfe03a4bd274b65367353b868739ef272a3 | |
| parent | d4dc1bbd1531d557716d4f0da75bf2bb1e021218 (diff) | |
| download | brew-e9158ca6dc99e3bb7ae475bee98153e78cafa2b0.tar.bz2 | |
brew upgrade: improve exit codes.
exit 1 if formulae given but none outdated
Closes Homebrew/homebrew#17199.
Signed-off-by: Mike McQuaid <mike@mikemcquaid.com>
| -rw-r--r-- | Library/Homebrew/cmd/upgrade.rb | 7 |
1 files changed, 4 insertions, 3 deletions
diff --git a/Library/Homebrew/cmd/upgrade.rb b/Library/Homebrew/cmd/upgrade.rb index c3a924ec5..6d477d6a8 100644 --- a/Library/Homebrew/cmd/upgrade.rb +++ b/Library/Homebrew/cmd/upgrade.rb @@ -16,11 +16,11 @@ module Homebrew extend self Homebrew.perform_preinstall_checks - outdated = if ARGV.named.empty? + if ARGV.named.empty? require 'cmd/outdated' - Homebrew.outdated_brews + outdated = Homebrew.outdated_brews else - ARGV.formulae.select do |f| + outdated = ARGV.formulae.select do |f| if f.installed? onoe "#{f}-#{f.installed_version} already installed" elsif not f.rack.exist? or f.rack.children.empty? @@ -29,6 +29,7 @@ module Homebrew extend self true end end + exit 1 if outdated.empty? end # Expand the outdated list to include outdated dependencies then sort and |