From 75e371684eee49c699bcd192ecc98048855836ca Mon Sep 17 00:00:00 2001 From: Jack Nagel Date: Thu, 16 May 2013 14:06:26 -0500 Subject: 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. --- Library/Homebrew/cmd/fetch.rb | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-) (limited to 'Library/Homebrew/cmd') 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 -- cgit v1.2.3