aboutsummaryrefslogtreecommitdiffstats
path: root/Library
diff options
context:
space:
mode:
authorJack Nagel2014-12-29 22:52:53 -0500
committerJack Nagel2014-12-29 22:57:45 -0500
commit8b51cf019d56c6ab5ab03c7bb4cf504eb7659c93 (patch)
tree1c70ae289889286d1b0ff58b36e2722a10ff332f /Library
parent912a586d15daaa9daf4c4831dc5cc9022acc2b62 (diff)
downloadbrew-8b51cf019d56c6ab5ab03c7bb4cf504eb7659c93.tar.bz2
Move exception text into exception object
Diffstat (limited to 'Library')
-rw-r--r--Library/Homebrew/download_strategy.rb7
-rw-r--r--Library/Homebrew/exceptions.rb11
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