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>
This commit is contained in:
parent
d4dc1bbd15
commit
e9158ca6dc
@ -16,11 +16,11 @@ module Homebrew extend self
|
|||||||
|
|
||||||
Homebrew.perform_preinstall_checks
|
Homebrew.perform_preinstall_checks
|
||||||
|
|
||||||
outdated = if ARGV.named.empty?
|
if ARGV.named.empty?
|
||||||
require 'cmd/outdated'
|
require 'cmd/outdated'
|
||||||
Homebrew.outdated_brews
|
outdated = Homebrew.outdated_brews
|
||||||
else
|
else
|
||||||
ARGV.formulae.select do |f|
|
outdated = ARGV.formulae.select do |f|
|
||||||
if f.installed?
|
if f.installed?
|
||||||
onoe "#{f}-#{f.installed_version} already installed"
|
onoe "#{f}-#{f.installed_version} already installed"
|
||||||
elsif not f.rack.exist? or f.rack.children.empty?
|
elsif not f.rack.exist? or f.rack.children.empty?
|
||||||
@ -29,6 +29,7 @@ module Homebrew extend self
|
|||||||
true
|
true
|
||||||
end
|
end
|
||||||
end
|
end
|
||||||
|
exit 1 if outdated.empty?
|
||||||
end
|
end
|
||||||
|
|
||||||
# Expand the outdated list to include outdated dependencies then sort and
|
# Expand the outdated list to include outdated dependencies then sort and
|
||||||
|
Loading…
x
Reference in New Issue
Block a user