diff options
| author | Mike McQuaid | 2014-06-17 14:25:43 +0100 |
|---|---|---|
| committer | Mike McQuaid | 2014-06-20 11:14:35 +0100 |
| commit | 1c176348caf6f6d220a1f877433a3ea581ab3030 (patch) | |
| tree | e761ddfbb2b169150f26f8d6d83263cc6663418e /Library/Homebrew/cmd | |
| parent | cca11731ca5842b65764344a2ce955764a0c9c42 (diff) | |
| download | brew-1c176348caf6f6d220a1f877433a3ea581ab3030.tar.bz2 | |
doctor: add caveats to avoid scaring people.
Sporadically people ask to whitelist doctor warnings or file issues when
there's nothing actually wrong. Help reassure such people with a
friendly message.
Closes Homebrew/homebrew#30230.
Signed-off-by: Mike McQuaid <mike@mikemcquaid.com>
Diffstat (limited to 'Library/Homebrew/cmd')
| -rw-r--r-- | Library/Homebrew/cmd/doctor.rb | 10 |
1 files changed, 9 insertions, 1 deletions
diff --git a/Library/Homebrew/cmd/doctor.rb b/Library/Homebrew/cmd/doctor.rb index b00ea583d..eaf380c59 100644 --- a/Library/Homebrew/cmd/doctor.rb +++ b/Library/Homebrew/cmd/doctor.rb @@ -1130,8 +1130,16 @@ module Homebrew methods.each do |method| out = checks.send(method) unless out.nil? or out.empty? + if first_warning + puts <<-EOS.undent + #{Tty.white}Please note that these warnings are just used to help the Homebrew maintainers + with debugging if you file an issue. If everything you use Homebrew for is + working fine: please don't worry and just ignore them. Thanks!#{Tty.reset} + EOS + end + lines = out.to_s.split('\n') - puts unless first_warning + puts opoo lines.shift Homebrew.failed = true puts lines |