Remove confusing implicit options handling

This code is supposed to allow

  depends_on "foo" => "with-bar"

to work when foo has only a "without-bar" option.

The options system was not designed to support this. Unfortunately, it
was bolted on anyway. The implementation is extremely difficult to
understand, and it only works for certain types of options, which is
confusing from a user's point of view. Luckily, no formulae in core or
the official taps rely on the behavior in order to function.

It is hindering progress in improving this code, so I am removing it.

1 files changed, 0 insertions, 29 deletions

diff --git a/Library/Homebrew/test/test_build_options.rb b/Library/Homebrew/test/test_build_options.rb
index d8327deac..95388a0ea 100644
--- a/Library/Homebrew/test/test_build_options.rb
+++ b/Library/Homebrew/test/test_build_options.rb
@@ -43,35 +43,6 @@ class BuildOptionsTests < Homebrew::TestCase
     assert_includes @build.unused_options, "--without-baz"
   end
 
-  def test_implicit_options
-    # --without-baz is not explicitly specified on the command line (i.e. args)
-    # therefore --with-baz should be implicitly assumed:
-    assert_includes @build.implicit_options, "--with-baz"
-    # But all these should not be in the implict_options:
-    refute_includes @build.implicit_options, "--without-baz"
-    refute_includes @build.implicit_options, "--with-bar"
-    refute_includes @build.implicit_options, "--without-bar"
-    refute_includes @build.implicit_options, "--with-qux"
-  end
-
-  def test_opposite_of
-    assert_equal Option.new("without-foo"), @build.opposite_of(Option.new("with-foo"))
-    assert_equal Option.new("with-foo"), @build.opposite_of("without-foo")
-    assert_equal Option.new("disable-spam"), @build.opposite_of(Option.new("enable-spam"))
-    assert_equal Option.new("enable-beer"), @build.opposite_of("disable-beer")
-  end
-
-  def test_has_opposite_of?
-    assert @build.has_opposite_of?("--without-foo")
-    assert @build.has_opposite_of?(Option.new("--with-qux"))
-    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
-
   def test_copies_do_not_share_underlying_options
     orig = BuildOptions.new []
     copy = orig.dup