From f96afdbf413635ff51e0563e46bfaa4f12eb3b7b Mon Sep 17 00:00:00 2001 From: "dependabot-preview[bot]" <27856297+dependabot-preview[bot]@users.noreply.github.com> Date: Tue, 16 Jul 2019 09:46:36 +0000 Subject: [PATCH 1/3] build: bump rubocop from 0.72.0 to 0.73.0 in /Library/Homebrew Bumps [rubocop](https://github.com/rubocop-hq/rubocop) from 0.72.0 to 0.73.0. - [Release notes](https://github.com/rubocop-hq/rubocop/releases) - [Changelog](https://github.com/rubocop-hq/rubocop/blob/master/CHANGELOG.md) - [Commits](https://github.com/rubocop-hq/rubocop/compare/v0.72.0...v0.73.0) Signed-off-by: dependabot-preview[bot] --- Library/Homebrew/Gemfile.lock | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-) diff --git a/Library/Homebrew/Gemfile.lock b/Library/Homebrew/Gemfile.lock index c46299e0bd..230c6e45b0 100644 --- a/Library/Homebrew/Gemfile.lock +++ b/Library/Homebrew/Gemfile.lock @@ -80,7 +80,7 @@ GEM rspec-support (3.8.2) rspec-wait (0.0.9) rspec (>= 3, < 4) - rubocop (0.72.0) + rubocop (0.73.0) jaro_winkler (~> 1.5.1) parallel (~> 1.10) parser (>= 2.6) From 12a0c281ea6dca7e2e0f7e10be2b2900c3d086e9 Mon Sep 17 00:00:00 2001 From: Mike McQuaid Date: Tue, 16 Jul 2019 21:10:21 +0100 Subject: [PATCH 2/3] cask/cmd/info: fix brew style, refactor. --- Library/Homebrew/cask/cmd/info.rb | 24 ++++++++++-------------- 1 file changed, 10 insertions(+), 14 deletions(-) diff --git a/Library/Homebrew/cask/cmd/info.rb b/Library/Homebrew/cask/cmd/info.rb index bfe4840703..a09291bce0 100644 --- a/Library/Homebrew/cask/cmd/info.rb +++ b/Library/Homebrew/cask/cmd/info.rb @@ -59,23 +59,19 @@ module Cask end def self.installation_info(cask) + return "Not installed\n" unless cask.installed? + install_info = +"" - if cask.installed? - cask.versions.each do |version| - versioned_staged_path = cask.caskroom_path.join(version) - install_info << versioned_staged_path.to_s - .concat(" (") - .concat( - if versioned_staged_path.exist? - then versioned_staged_path.abv - else Formatter.error("does not exist") - end, - ).concat(")\n") + cask.versions.each do |version| + versioned_staged_path = cask.caskroom_path.join(version) + path_details = if versioned_staged_path.exist? + versioned_staged_path.abv + else + Formatter.error("does not exist") end - install_info.freeze - else - "Not installed\n" + install_info << "#{versioned_staged_path} (#{path_details})\n" end + install_info.freeze end def self.name_info(cask) From dca43851afb2c95c0c514cf80f99f6fa8a0510b5 Mon Sep 17 00:00:00 2001 From: Mike McQuaid Date: Tue, 16 Jul 2019 21:10:36 +0100 Subject: [PATCH 3/3] download_strategy: fix brew style, refactor. --- Library/Homebrew/download_strategy.rb | 14 +++++++------- 1 file changed, 7 insertions(+), 7 deletions(-) diff --git a/Library/Homebrew/download_strategy.rb b/Library/Homebrew/download_strategy.rb index 467959f213..ac8225741f 100644 --- a/Library/Homebrew/download_strategy.rb +++ b/Library/Homebrew/download_strategy.rb @@ -66,13 +66,13 @@ class AbstractDownloadStrategy def chdir entries = Dir["*"] - case entries.length - when 0 then raise "Empty archive" - when 1 then begin - Dir.chdir entries.first - rescue - nil - end + raise "Empty archive" if entries.length.zero? + return if entries.length != 1 + + begin + Dir.chdir entries.first + rescue + nil end end private :chdir