diff options
| author | Mike McQuaid | 2016-09-10 19:43:49 +0100 |
|---|---|---|
| committer | Mike McQuaid | 2016-09-10 19:43:49 +0100 |
| commit | d0c33b370528ce57bfc049a3b939cada1481cc96 (patch) | |
| tree | aac7463dd2fd9c011046789cc07e70a6e1eaec4b /Library | |
| parent | 7fb915319d391dfc20367be08005c1f9e154c81e (diff) | |
| download | brew-d0c33b370528ce57bfc049a3b939cada1481cc96.tar.bz2 | |
bottle: use key/value rather than field.
Consistent naming is good.
Diffstat (limited to 'Library')
| -rw-r--r-- | Library/Homebrew/dev-cmd/bottle.rb | 14 |
1 files changed, 7 insertions, 7 deletions
diff --git a/Library/Homebrew/dev-cmd/bottle.rb b/Library/Homebrew/dev-cmd/bottle.rb index f912109ed..9fca091d4 100644 --- a/Library/Homebrew/dev-cmd/bottle.rb +++ b/Library/Homebrew/dev-cmd/bottle.rb @@ -289,17 +289,17 @@ module Homebrew old_spec = f.bottle_specification if ARGV.include?("--keep-old") && !old_spec.checksums.empty? - mismatches = [:root_url, :prefix, :cellar, :rebuild].select do |field| - old_spec.send(field) != bottle.send(field) + mismatches = [:root_url, :prefix, :cellar, :rebuild].select do |key| + old_spec.send(key) != bottle.send(key) end mismatches.delete(:cellar) if old_spec.cellar == :any && bottle.cellar == :any_skip_relocation unless mismatches.empty? bottle_path.unlink if bottle_path.exist? - mismatches.map! do |field| - old_value = old_spec.send(field).inspect - value = bottle.send(field).inspect - "#{field}: old: #{old_value}, new: #{value}" + mismatches.map! do |key| + old_value = old_spec.send(key).inspect + value = bottle.send(key).inspect + "#{key}: old: #{old_value}, new: #{value}" end odie <<-EOS.undent @@ -405,7 +405,7 @@ module Homebrew if old_value.empty? || value != old_value old_value = old_value_original.inspect value = value_original.inspect - mismatches << "#{field}: old: #{old_value}, new: #{value}" + mismatches << "#{key}: old: #{old_value}, new: #{value}" end end |
