From cac30f618645207d1ad89b33247ba0dd12d1793d Mon Sep 17 00:00:00 2001 From: Luca Favatella Date: Sat, 9 Jun 2018 23:41:44 +0100 Subject: [PATCH] cask doctor: delete unused render_* methods --- Library/Homebrew/cask/lib/hbc/cli/doctor.rb | 55 --------------------- 1 file changed, 55 deletions(-) diff --git a/Library/Homebrew/cask/lib/hbc/cli/doctor.rb b/Library/Homebrew/cask/lib/hbc/cli/doctor.rb index 63edf2d55c..b25894af3c 100644 --- a/Library/Homebrew/cask/lib/hbc/cli/doctor.rb +++ b/Library/Homebrew/cask/lib/hbc/cli/doctor.rb @@ -165,11 +165,6 @@ module Hbc Formatter.error("(#{string})") end - def self.render_with_none(string) - return string if !string.nil? && string.respond_to?(:to_s) && !string.to_s.empty? - none_string - end - def self.alt_taps Tap.select { |t| t.cask_dir.exist? && t != Tap.default_cask_tap } end @@ -181,16 +176,6 @@ module Hbc "0" end - def self.render_taps(*taps) - taps.collect do |tap| - if tap.path.nil? || tap.path.to_s.empty? - none_string - else - "#{tap.path} (#{cask_count_for_tap(tap)})" - end - end - end - def self.render_env_var(var) return unless ENV.key?(var) var = %Q(#{var}="#{ENV[var]}") @@ -201,46 +186,6 @@ module Hbc path.gsub(ENV["HOME"], "~") end - # This could be done by calling into Homebrew, but the situation - # where "doctor" is needed is precisely the situation where such - # things are less dependable. - def self.render_install_location - locations = Dir.glob(HOMEBREW_CELLAR.join("brew-cask", "*")).reverse - if locations.empty? - none_string - else - locations.collect do |l| - "#{l} #{error_string 'error: legacy install. Run "brew uninstall --force brew-cask".'}" - end - end - end - - def self.render_staging_location(path) - path = Pathname.new(user_tilde(path.to_s)) - if !path.exist? - "#{path} #{error_string "error: path does not exist"}" - elsif !path.writable? - "#{path} #{error_string "error: not writable by current user"}" - else - path - end - end - - def self.render_load_path(paths) - paths.map(&method(:user_tilde)) - return "#{none_string} #{error_string}" if [*paths].empty? - paths - end - - def self.render_cached_downloads - cleanup = CLI::Cleanup.new - count = cleanup.cache_files.count - size = cleanup.disk_cleanup_size - msg = user_tilde(Cache.path.to_s) - msg << " (#{number_readable(count)} files, #{disk_usage_readable(size)})" unless count.zero? - msg - end - def self.help "checks for configuration issues" end