From 44ae0080e1bcba68f80f70c0f7bd01b4e753c3e1 Mon Sep 17 00:00:00 2001
From: Mike McQuaid
Date: Sun, 4 Sep 2016 13:22:06 +0100
Subject: Revert "bottle: allow --keep-old to ignore same sha256s."

This reverts commit ec1437499a178a09805854469c5162d749270f3e.
---
 Library/Homebrew/cmd/bottle.rb | 10 +++-------
 1 file changed, 3 insertions(+), 7 deletions(-)

(limited to 'Library/Homebrew/cmd')

diff --git a/Library/Homebrew/cmd/bottle.rb b/Library/Homebrew/cmd/bottle.rb
index 12071b907..888e8c778 100644
--- a/Library/Homebrew/cmd/bottle.rb
+++ b/Library/Homebrew/cmd/bottle.rb
@@ -391,14 +391,10 @@ module Homebrew
 
                 if !tag.empty?
                   if !bottle_hash["bottle"]["tags"][tag].to_s.empty?
-                    old_value = bottle_hash["bottle"]["tags"][tag][key].to_s
-                    if value != old_value
-                      mismatches << "#{key} => #{tag}"
-                    end
-                    next
+                    mismatches << "#{key} => #{tag}"
+                  else
+                    bottle.send(key, value => tag.to_sym)
                   end
-
-                  bottle.send(key, value => tag.to_sym)
                   next
                 end
 
-- 
cgit v1.2.3


From cf505fa6ce738983e4ec343ca78c7a5916756b23 Mon Sep 17 00:00:00 2001
From: Mike McQuaid
Date: Sun, 4 Sep 2016 13:22:08 +0100
Subject: Revert "bottle: fix --keep-old JSON generation."

This reverts commit 5e9bb877d54fc1803a866f49997f3e819c3ca650.
---
 Library/Homebrew/cmd/bottle.rb | 21 +++------------------
 1 file changed, 3 insertions(+), 18 deletions(-)

(limited to 'Library/Homebrew/cmd')

diff --git a/Library/Homebrew/cmd/bottle.rb b/Library/Homebrew/cmd/bottle.rb
index 888e8c778..138e482e0 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
-- 
cgit v1.2.3