diff options
| author | Jack Nagel | 2013-05-16 14:06:26 -0500 |
|---|---|---|
| committer | Jack Nagel | 2013-05-16 14:06:26 -0500 |
| commit | 75e371684eee49c699bcd192ecc98048855836ca (patch) | |
| tree | 54e32525f8934b89957f91ab08bc4459a2707cfd /Library/Homebrew/cmd/fetch.rb | |
| parent | 5241271889a465215c4c37cb5f91b6c9762e5f8b (diff) | |
| download | homebrew-75e371684eee49c699bcd192ecc98048855836ca.tar.bz2 | |
Adjust semantics of Formula#fetch
It doesn't really make logical sense that this method returns both the
fetched path (or sometimes nil!) and the downloader, so just return the
path (again, or nil!) and callers that want the downloader can ask for
it separately.
Diffstat (limited to 'Library/Homebrew/cmd/fetch.rb')
| -rw-r--r-- | Library/Homebrew/cmd/fetch.rb | 2 |
1 files changed, 1 insertions, 1 deletions
diff --git a/Library/Homebrew/cmd/fetch.rb b/Library/Homebrew/cmd/fetch.rb index cb0fd4660..24b1b0614 100644 --- a/Library/Homebrew/cmd/fetch.rb +++ b/Library/Homebrew/cmd/fetch.rb @@ -25,7 +25,7 @@ module Homebrew extend self def fetch_formula f f.cached_download.rmtree if already_fetched?(f) && ARGV.force? - download, _ = f.fetch + download = f.fetch # FIXME why are strategies returning different types? return unless download.is_a? Pathname |