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 | |
| parent | ab64e120d011d110b701bd9fcc1f459444bc3aae (diff) | |
| download | homebrew-5e191b0a652e23f25e69769c9bc3f8c969d76692.tar.bz2 | |
Move exception text into exception object
| -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 |
