From a7946e0088185ff852babc657d5ccf1b57851575 Mon Sep 17 00:00:00 2001
From: Max Howell <mxcl@me.com>
Date: Wed, 29 Aug 2012 15:19:42 -0400
Subject: [PATCH] Warn about removed warnings less shockingly

The problem here now is though that the warnings only appear when compiling verbosely. But they should thus be visible if the build fails. Or if people are hunting for problems.
---
 Library/ENV/4.3/cc        | 11 +++++++----
 Library/Homebrew/build.rb |  8 --------
 2 files changed, 7 insertions(+), 12 deletions(-)

diff --git a/Library/ENV/4.3/cc b/Library/ENV/4.3/cc
index 2342dc51e9..2d7e6ca801 100755
--- a/Library/ENV/4.3/cc
+++ b/Library/ENV/4.3/cc
@@ -106,6 +106,13 @@ class Cmd
         args << arg
       end
     end
+
+    rms = @args - args
+    %w{CPPFLAGS LDFLAGS CXXFLAGS CFLAGS}.each do |flag|
+      unison = ENV[flag].split(' ') & rms
+      puts "Warning! #{unison*' '} removed from #{flag.upcase} by superenv" unless unison.empty?
+    end
+
     args
   end
   def cflags
@@ -135,10 +142,6 @@ end
 ####################################################################### sanity
 abort "The build-tool has reset ENV. --lame-env required." unless ENV['HOMEBREW_BREW_FILE']
 
-%w{CPPFLAGS LDFLAGS CXXFLAGS CFLAGS}.each do |flag|
-  puts "#{flag} set but superenv running! Flags may be removed!" if ENV[flag]
-end
-
 ######################################################################### main
 cmd = Cmd.new($0, ARGV)
 exec "xcrun", cmd.tool, *cmd.args
diff --git a/Library/Homebrew/build.rb b/Library/Homebrew/build.rb
index dc94cf738c..97365a2039 100755
--- a/Library/Homebrew/build.rb
+++ b/Library/Homebrew/build.rb
@@ -85,14 +85,6 @@ def install f
     ENV.deps = keg_only_deps.map(&:to_s)
     ENV.x11 = f.requirements.detect{|rq| rq.class == X11Dependency }
     ENV.setup_build_environment
-    class << ENV
-      def []=(key, value)
-        case key when 'CFLAGS', 'CPPFLAGS', 'LDFAGS'
-          opoo "#{key} set with --env=super! Flags may not take effect!"
-        end
-        store(key, value)
-      end
-    end
   end
 
   f.recursive_requirements.each { |req| req.modify_build_environment }