diff options
| author | Jack Nagel | 2013-09-05 18:50:42 -0500 |
|---|---|---|
| committer | Jack Nagel | 2013-09-05 18:51:27 -0500 |
| commit | a098f46e10d2b7c0d3c5063cc393a08f7cf6b12a (patch) | |
| tree | 9cbaeef359b0a77046c22d880cca8709ab1cf83f /Library | |
| parent | 37cf251f8f5b7614832ef35aaa05fe4bbdce7f9f (diff) | |
| download | homebrew-a098f46e10d2b7c0d3c5063cc393a08f7cf6b12a.tar.bz2 | |
Fix breakage in option recognition
Fixes #22347.
Diffstat (limited to 'Library')
| -rw-r--r-- | Library/Homebrew/build_options.rb | 2 | ||||
| -rw-r--r-- | Library/Homebrew/test/test_build_options.rb | 8 |
2 files changed, 7 insertions, 3 deletions
diff --git a/Library/Homebrew/build_options.rb b/Library/Homebrew/build_options.rb index 441a81bc2..6400b36ce 100644 --- a/Library/Homebrew/build_options.rb +++ b/Library/Homebrew/build_options.rb @@ -107,7 +107,7 @@ class BuildOptions end def has_opposite_of? option - true if args.include? opposite_of option + @options.include? opposite_of(option) end def opposite_of option diff --git a/Library/Homebrew/test/test_build_options.rb b/Library/Homebrew/test/test_build_options.rb index 9e2a9e06f..38eeae22b 100644 --- a/Library/Homebrew/test/test_build_options.rb +++ b/Library/Homebrew/test/test_build_options.rb @@ -1,9 +1,9 @@ - require 'testing_env' +require 'testing_env' require 'build_options' class BuildOptionsTests < Test::Unit::TestCase def setup - args = %w{--with-foo --with-bar --without-qux} # args fake the command line + args = %w{--with-foo --with-bar --without-qux} @build = BuildOptions.new(args) @build.add("with-foo") @build.add("with-bar") @@ -67,4 +67,8 @@ class BuildOptionsTests < Test::Unit::TestCase assert !@build.has_opposite_of?("--without-qux") assert !@build.has_opposite_of?("--without-nonexisting") end + + def test_actually_recognizes_implicit_options + assert @build.has_opposite_of?("--with-baz") + end end |
