From 9438dc94e3724bcf2456c1d550e3386ae0bc9d94 Mon Sep 17 00:00:00 2001 From: =?UTF-8?q?fn=20=E2=8C=83=20=E2=8C=A5?= <70830482+FnControlOption@users.noreply.github.com> Date: Fri, 5 Nov 2021 11:34:01 -0700 Subject: [PATCH] search: split missing formula help into its own method --- Library/Homebrew/cmd/search.rb | 21 ++++++++++++++------- 1 file changed, 14 insertions(+), 7 deletions(-) diff --git a/Library/Homebrew/cmd/search.rb b/Library/Homebrew/cmd/search.rb index 562959ccb6..0bfd672a66 100644 --- a/Library/Homebrew/cmd/search.rb +++ b/Library/Homebrew/cmd/search.rb @@ -143,14 +143,21 @@ module Homebrew count = all_formulae.count + all_casks.count - if $stdout.tty? && (reason = MissingFormula.reason(query, silent: true)) && local_casks.exclude?(query) - if count.positive? - puts - puts "If you meant #{query.inspect} specifically:" - end - puts reason - end + print_missing_formula_help(query, count.positive?) if local_casks.exclude?(query) odie "No formulae or casks found for #{query.inspect}." if count.zero? end + + def print_missing_formula_help(query, found_matches) + return unless $stdout.tty? + + reason = MissingFormula.reason(query, silent: true) + return if reason.nil? + + if found_matches + puts + puts "If you meant #{query.inspect} specifically:" + end + puts reason + end end