Don't warn about unused `brew install` arguments.

This is a step closer to better argument handling but for now just
fixes the issue in #1217 where it starts complaining about options like
`--build-from-source` being used.

2 files changed, 32 insertions, 1 deletions

diff --git a/Library/Homebrew/build_options.rb b/Library/Homebrew/build_options.rb
index edcae01f5..44b6440dc 100644
--- a/Library/Homebrew/build_options.rb
+++ b/Library/Homebrew/build_options.rb
@@ -103,7 +103,7 @@ class BuildOptions
 
   # @private
   def invalid_options
-    @args - @options
+    @args - @options - BuildOptions.formula_install_options
   end
 
   # @private
@@ -111,6 +111,12 @@ class BuildOptions
     invalid_options.map(&:flag).sort
   end
 
+  def self.formula_install_options
+    @formula_install_options ||= ARGV.formula_install_option_names.map do |option_name|
+      Option.new option_name[2..-1]
+    end
+  end
+
   private
 
   def option_defined?(name)
diff --git a/Library/Homebrew/extend/ARGV.rb b/Library/Homebrew/extend/ARGV.rb
index bd60cbecc..3e4e7f5c7 100644
--- a/Library/Homebrew/extend/ARGV.rb
+++ b/Library/Homebrew/extend/ARGV.rb
@@ -1,4 +1,29 @@
 module HomebrewArgvExtension
+  def formula_install_option_names
+    %w[
+      --debug
+      --env=
+      --ignore-dependencies
+      --cc=
+      --build-from-source
+      --devel
+      --HEAD
+      --keep-tmp
+      --interactive
+      --git
+      --sandbox
+      --no-sandbox
+      --build-bottle
+      --force-bottle
+      --verbose
+      -i
+      -v
+      -d
+      -g
+      -s
+    ].freeze
+  end
+
   def named
     @named ||= self - options_only
   end