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

bottle: fix old_value/value ordering for --merge.

Browse Source 
Previously this was the wrong way around causing it to not be accepting
enough on e.g. a bump from :any to :any_no_relocation.
This commit is contained in:
Mike McQuaid 2016-09-10 21:15:28 +01:00
parent 7fb915319d
commit be6876c79a
1 changed files with 5 additions and 5 deletions
Show Stats Download Patch File Download Diff File Expand all files Collapse all files

10
Library/Homebrew/dev-cmd/bottle.rb
View File

@ -383,24 +383,24 @@ module Homebrew
bottle_block_contents.lines.each do |line|
line = line.strip
next if line.empty?
key, value_original, _, tag = line.split " ", 4
key, old_value_original, _, tag = line.split " ", 4
valid_key = %w[root_url prefix cellar rebuild sha1 sha256].include? key
next unless valid_key
value = value_original.to_s.delete ":'\""
old_value = old_value_original.to_s.delete ":'\""
tag = tag.to_s.delete ":"
if !tag.empty?
if !bottle_hash["bottle"]["tags"][tag].to_s.empty?
mismatches << "#{key} => #{tag}"
else
bottle.send(key, value => tag.to_sym)
bottle.send(key, old_value => tag.to_sym)
end
next
end
old_value_original = bottle_hash["bottle"][key]
old_value = old_value_original.to_s
value_original = bottle_hash["bottle"][key]
value = value_original.to_s
next if key == "cellar" && old_value == "any" && value == "any_skip_relocation"
if old_value.empty? || value != old_value
old_value = old_value_original.inspect