aboutsummaryrefslogtreecommitdiffstats
path: root/Library
diff options
context:
space:
mode:
authorMike McQuaid2016-09-10 10:24:57 +0100
committerMike McQuaid2016-09-11 17:59:01 +0100
commit2c5beb0effd113bd0342f052d735d697e3e5af95 (patch)
tree877ca7b780c33466d9ffbe78278d4b58f0688081 /Library
parentcc526601cfe14861260b76359f6f1a2ec5fa33d2 (diff)
downloadbrew-2c5beb0effd113bd0342f052d735d697e3e5af95.tar.bz2
cmd/upgrade: fix Rubocop warnings.
Diffstat (limited to 'Library')
-rw-r--r--Library/Homebrew/cmd/upgrade.rb14
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