From cf505fa6ce738983e4ec343ca78c7a5916756b23 Mon Sep 17 00:00:00 2001 From: Mike McQuaid <mike@mikemcquaid.com> Date: Sun, 4 Sep 2016 13:22:08 +0100 Subject: [PATCH] Revert "bottle: fix --keep-old JSON generation." This reverts commit 5e9bb877d54fc1803a866f49997f3e819c3ca650. --- Library/Homebrew/cmd/bottle.rb | 21 +++------------------ 1 file changed, 3 insertions(+), 18 deletions(-) diff --git a/Library/Homebrew/cmd/bottle.rb b/Library/Homebrew/cmd/bottle.rb index 888e8c7789..138e482e0d 100644 --- a/Library/Homebrew/cmd/bottle.rb +++ b/Library/Homebrew/cmd/bottle.rb @@ -279,9 +279,10 @@ module Homebrew end bottle.rebuild rebuild sha256 = bottle_path.sha256 + bottle.sha256 sha256 => Utils::Bottles.tag - if ARGV.include?("--keep-old") && !f.bottle_specification.checksums.empty? - old_spec = f.bottle_specification + old_spec = f.bottle_specification + if ARGV.include?("--keep-old") && !old_spec.checksums.empty? bad_fields = [:root_url, :prefix, :cellar, :rebuild].select do |field| old_spec.send(field) != bottle.send(field) end @@ -290,11 +291,8 @@ module Homebrew bottle_path.unlink if bottle_path.exist? odie "--keep-old is passed but there are changes in: #{bad_fields.join ", "}" end - bottle = old_spec end - bottle.sha256 sha256 => Utils::Bottles.tag - output = bottle_output bottle puts "./#{filename}" @@ -325,19 +323,6 @@ module Homebrew }, }, } - if ARGV.include?("--keep-old") - bottle.checksums.each do |hash_type, checksums| - checksums.each do |checksum_hash| - checksum_hash.each do |checksum, tag| - tag_hash = {} - tag_hash["filename"] ||= Bottle::Filename.create(f, tag, rebuild).to_s - tag_hash[hash_type.to_s] ||= checksum.hexdigest - json[f.full_name]["bottle"]["tags"][tag.to_s] ||= tag_hash - end - end - end - end - File.open("#{filename.prefix}.bottle.json", "w") do |file| file.write Utils::JSON.dump json end -- GitLab