Merge pull request #17465 from Homebrew/list-fix
cmd/list: new fix for exit code when supplied both a cask and formula
This commit is contained in:
commit
3f08c75e7b
@ -153,7 +153,7 @@ module Homebrew
|
|||||||
system_command! "find", args: casks.map(&:caskroom_path) + find_args, print_stdout: true if casks.present?
|
system_command! "find", args: casks.map(&:caskroom_path) + find_args, print_stdout: true if casks.present?
|
||||||
else
|
else
|
||||||
kegs.each { |keg| PrettyListing.new keg } if kegs.present?
|
kegs.each { |keg| PrettyListing.new keg } if kegs.present?
|
||||||
list_casks if casks.present?
|
Cask::List.list_casks(*casks, one: args.public_send(:"1?")) if casks.present?
|
||||||
end
|
end
|
||||||
end
|
end
|
||||||
end
|
end
|
||||||
@ -165,7 +165,10 @@ module Homebrew
|
|||||||
Formula.racks
|
Formula.racks
|
||||||
else
|
else
|
||||||
racks = args.named.map { |n| Formulary.to_rack(n) }
|
racks = args.named.map { |n| Formulary.to_rack(n) }
|
||||||
racks.select(&:exist?)
|
racks.select do |rack|
|
||||||
|
Homebrew.failed = true unless rack.exist?
|
||||||
|
rack.exist?
|
||||||
|
end
|
||||||
end
|
end
|
||||||
if args.pinned?
|
if args.pinned?
|
||||||
pinned_versions = {}
|
pinned_versions = {}
|
||||||
@ -191,6 +194,7 @@ module Homebrew
|
|||||||
Cask::Caskroom.casks
|
Cask::Caskroom.casks
|
||||||
else
|
else
|
||||||
filtered_args = args.named.dup.delete_if do |n|
|
filtered_args = args.named.dup.delete_if do |n|
|
||||||
|
Homebrew.failed = true unless Cask::Caskroom.path.join(n).exist?
|
||||||
!Cask::Caskroom.path.join(n).exist?
|
!Cask::Caskroom.path.join(n).exist?
|
||||||
end
|
end
|
||||||
# NamedAargs subclasses array
|
# NamedAargs subclasses array
|
||||||
|
Loading…
x
Reference in New Issue
Block a user