diff options
| author | Jack Nagel | 2014-12-29 22:52:53 -0500 | 
|---|---|---|
| committer | Jack Nagel | 2014-12-29 22:57:45 -0500 | 
| commit | 5e191b0a652e23f25e69769c9bc3f8c969d76692 (patch) | |
| tree | 2beb2a823089a234eb0ec605dd049604c12be2b8 /Library | |
| parent | ab64e120d011d110b701bd9fcc1f459444bc3aae (diff) | |
| download | homebrew-5e191b0a652e23f25e69769c9bc3f8c969d76692.tar.bz2 | |
Move exception text into exception object
Diffstat (limited to 'Library')
| -rw-r--r-- | Library/Homebrew/download_strategy.rb | 7 | ||||
| -rw-r--r-- | Library/Homebrew/exceptions.rb | 11 | 
2 files changed, 11 insertions, 7 deletions
| diff --git a/Library/Homebrew/download_strategy.rb b/Library/Homebrew/download_strategy.rb index f474b03a6..55097a92a 100644 --- a/Library/Homebrew/download_strategy.rb +++ b/Library/Homebrew/download_strategy.rb @@ -184,12 +184,7 @@ class CurlDownloadStrategy < AbstractDownloadStrategy            had_incomplete_download = false            retry          else -          if @url =~ %r[^file://] -            msg = "File does not exist: #{@url.sub(%r[^file://], "")}" -          else -            msg = "Download failed: #{@url}" -          end -          raise CurlDownloadStrategyError, msg +          raise CurlDownloadStrategyError.new(@url)          end        end        ignore_interrupts { temporary_path.rename(cached_location) } diff --git a/Library/Homebrew/exceptions.rb b/Library/Homebrew/exceptions.rb index a6a3380bb..66d5c33d2 100644 --- a/Library/Homebrew/exceptions.rb +++ b/Library/Homebrew/exceptions.rb @@ -208,7 +208,16 @@ class DownloadError < RuntimeError  end  # raised in CurlDownloadStrategy.fetch -class CurlDownloadStrategyError < RuntimeError; end +class CurlDownloadStrategyError < RuntimeError +  def initialize(url) +    case url +    when %r[^file://(.+)] +      super "File does not exist: #{$1}" +    else +      super "Download failed: #{url}" +    end +  end +end  # raised by safe_system in utils.rb  class ErrorDuringExecution < RuntimeError | 
