Merge pull request #13777 from Homebrew/circular_deps_fail
cmd/deps: return failing exit code on circular dependencies.
This commit is contained in:
commit
22c977fd8d
@ -286,8 +286,14 @@ module Homebrew
|
|||||||
end
|
end
|
||||||
|
|
||||||
display_s = "#{tree_lines} #{dep_display_name(dep, args: args)}"
|
display_s = "#{tree_lines} #{dep_display_name(dep, args: args)}"
|
||||||
|
|
||||||
|
# Detect circular dependencies and consider them a failure if present.
|
||||||
is_circular = dep_stack.include?(dep.name)
|
is_circular = dep_stack.include?(dep.name)
|
||||||
display_s = "#{display_s} (CIRCULAR DEPENDENCY)" if is_circular
|
if is_circular
|
||||||
|
display_s = "#{display_s} (CIRCULAR DEPENDENCY)"
|
||||||
|
Homebrew.failed = true
|
||||||
|
end
|
||||||
|
|
||||||
puts "#{prefix}#{display_s}"
|
puts "#{prefix}#{display_s}"
|
||||||
|
|
||||||
next if !recursive || is_circular
|
next if !recursive || is_circular
|
||||||
|
Loading…
x
Reference in New Issue
Block a user