diff options
| author | Mike McQuaid | 2016-09-10 10:24:57 +0100 |
|---|---|---|
| committer | Mike McQuaid | 2016-09-11 17:59:01 +0100 |
| commit | 2c5beb0effd113bd0342f052d735d697e3e5af95 (patch) | |
| tree | 877ca7b780c33466d9ffbe78278d4b58f0688081 /Library | |
| parent | cc526601cfe14861260b76359f6f1a2ec5fa33d2 (diff) | |
| download | brew-2c5beb0effd113bd0342f052d735d697e3e5af95.tar.bz2 | |
cmd/upgrade: fix Rubocop warnings.
Diffstat (limited to 'Library')
| -rw-r--r-- | Library/Homebrew/cmd/upgrade.rb | 14 |
1 files changed, 9 insertions, 5 deletions
diff --git a/Library/Homebrew/cmd/upgrade.rb b/Library/Homebrew/cmd/upgrade.rb index d0fa180ce..1053319bf 100644 --- a/Library/Homebrew/cmd/upgrade.rb +++ b/Library/Homebrew/cmd/upgrade.rb @@ -35,7 +35,7 @@ module Homebrew end (ARGV.resolved_formulae - outdated).each do |f| - versions = f.installed_kegs.map { |keg| keg.version } + versions = f.installed_kegs.map(&:version) if versions.empty? onoe "#{f.full_name} not installed" else @@ -51,11 +51,11 @@ module Homebrew outdated -= pinned end - unless outdated.empty? + if outdated.empty? + oh1 "No packages to upgrade" + else oh1 "Upgrading #{outdated.length} outdated package#{plural(outdated.length)}, with result:" puts outdated.map { |f| "#{f.full_name} #{f.pkg_version}" } * ", " - else - oh1 "No packages to upgrade" end unless upgrade_pinned? || pinned.empty? @@ -116,6 +116,10 @@ module Homebrew ofail e ensure # restore previous installation state if build failed - outdated_keg.link if outdated_keg && !f.installed? rescue nil + begin + outdated_keg.link if outdated_keg && !f.installed? + rescue + nil + end end end |
