admin/brew
1
0
Fork 0
Code Issues Packages Projects Releases 6 Wiki Activity

cmd/upgrade, reinstall: don't install on dry-run.

Browse Source 
Fixes #7033.
This commit is contained in:
Mike McQuaid 2020-02-14 12:18:55 +00:00
parent 516d7e8278
commit ac97c8545a
No known key found for this signature in database
GPG Key ID: 48A898132FD8EE70
2 changed files with 4 additions and 0 deletions
Show Stats Download Patch File Download Diff File Expand all files Collapse all files

2
Library/Homebrew/cmd/upgrade.rb
View File

@ -326,6 +326,8 @@ module Homebrew
.join(", ") .join(", ")
end end
return if args.dry_run?
reinstallable_broken_dependents.each do |f| reinstallable_broken_dependents.each do |f|
reinstall_formula(f, build_from_source: true) reinstall_formula(f, build_from_source: true)
rescue FormulaInstallationAlreadyAttemptedError rescue FormulaInstallationAlreadyAttemptedError

2
Library/Homebrew/reinstall.rb
View File

@ -8,6 +8,8 @@ module Homebrew
module_function module_function
def reinstall_formula(f, build_from_source: false) def reinstall_formula(f, build_from_source: false)
return if args.dry_run?
if f.opt_prefix.directory? if f.opt_prefix.directory?
keg = Keg.new(f.opt_prefix.resolved_path) keg = Keg.new(f.opt_prefix.resolved_path)
tab = Tab.for_keg(keg) tab = Tab.for_keg(keg)