From ea4d137e871ace8200743f237c48189e3b930da1 Mon Sep 17 00:00:00 2001 From: Martin Afanasjew Date: Tue, 15 Dec 2015 10:50:38 +0100 Subject: dependency: fix merging tags in 'merge_repeats' While it may suffice to merge string and non-reserved tags by forming a union of all tags of dependencies of the same name, this approach fails to work for the reserved tags. These are now merged such that the most restrictive tag (meaning sometimes an empty tag) is preserved. The previous behavior caused essential dependencies to be omitted and builds to fail in response. E.g., multiple `:fortran` dependencies with tags `[]`, `[:recommended]`, and `[:optional]` would have been expanded and merged to `"gcc"` with tags `[:recommended, :optional]`, causing it to be no longer seen as a required dependency. Closes Homebrew/homebrew#47040. Signed-off-by: Martin Afanasjew --- Library/Homebrew/test/test_dependency.rb | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-) (limited to 'Library/Homebrew/test') diff --git a/Library/Homebrew/test/test_dependency.rb b/Library/Homebrew/test/test_dependency.rb index 4ba7033d9..16bad6507 100644 --- a/Library/Homebrew/test/test_dependency.rb +++ b/Library/Homebrew/test/test_dependency.rb @@ -48,7 +48,7 @@ class DependencyTests < Homebrew::TestCase assert_equal Dependency, merged.first.class foo_named_dep = merged.find {|d| d.name == "foo"} - assert_equal [:build, "bar"], foo_named_dep.tags + assert_equal ["bar"], foo_named_dep.tags assert_includes foo_named_dep.option_names, "foo" assert_includes foo_named_dep.option_names, "foo2" -- cgit v1.2.3