aboutsummaryrefslogtreecommitdiffstats
path: root/Library/Homebrew
diff options
context:
space:
mode:
authorJack Nagel2014-08-09 10:30:43 -0500
committerJack Nagel2014-08-09 10:30:43 -0500
commitd3d2ed27902b3d8374dac283d501b0a19ad5b153 (patch)
tree4ba9536868473ba69f0e007b8f0bd1a7c8e288e3 /Library/Homebrew
parent5dab3ee322a1fe8336d0fe7a7a7ee22467c982c7 (diff)
downloadhomebrew-d3d2ed27902b3d8374dac283d501b0a19ad5b153.tar.bz2
Stop duping options when duping the build object
The options collection cannot be mutated from the build object, so it can be shared among copies safely.
Diffstat (limited to 'Library/Homebrew')
-rw-r--r--Library/Homebrew/build_options.rb2
-rw-r--r--Library/Homebrew/test/test_build_options.rb6
2 files changed, 0 insertions, 8 deletions
diff --git a/Library/Homebrew/build_options.rb b/Library/Homebrew/build_options.rb
index 28fbb959c..b04c9dc1d 100644
--- a/Library/Homebrew/build_options.rb
+++ b/Library/Homebrew/build_options.rb
@@ -7,7 +7,6 @@ class BuildOptions
attr_accessor :args
attr_accessor :universal
- attr_reader :options
def initialize(args, options)
@args = Options.coerce(args)
@@ -16,7 +15,6 @@ class BuildOptions
def initialize_copy(other)
super
- @options = other.options.dup
@args = other.args.dup
end
diff --git a/Library/Homebrew/test/test_build_options.rb b/Library/Homebrew/test/test_build_options.rb
index 11b3145ba..4fa332230 100644
--- a/Library/Homebrew/test/test_build_options.rb
+++ b/Library/Homebrew/test/test_build_options.rb
@@ -37,12 +37,6 @@ class BuildOptionsTests < Homebrew::TestCase
assert_includes @build.unused_options, "--without-baz"
end
- def test_copies_do_not_share_underlying_options
- orig = BuildOptions.new [], Options.new
- copy = orig.dup
- refute_same orig.options, copy.options
- end
-
def test_copies_do_not_share_underlying_args
orig = BuildOptions.new [], Options.new
copy = orig.dup