admin/brew
1
0
Fork 0
Code Issues Packages Projects Releases 6 Wiki Activity

CurlDownloadStrategy: remove "tarball_path" accessor

Browse Source 
This information is already exposed as "cached_location", which is
common to all download strategies.
This commit is contained in:
Jack Nagel 2013-05-16 15:25:09 -05:00
parent e8997c8a5a
commit 65775e4318
2 changed files with 3 additions and 5 deletions
Show Stats Download Patch File Download Diff File Expand all files Collapse all files

7
Library/Contributions/cmd/brew-mirror-check.rb
View File

@ -9,14 +9,13 @@ class Formula
# Force the downloader to attempt the download by removing the tarball if # Force the downloader to attempt the download by removing the tarball if
# it is allready cached. # it is allready cached.
tarball_path = downloader.tarball_path cached_download.unlink if cached_download.exist?
tarball_path.unlink if tarball_path.exist?
fetched = downloader.fetch downloader.fetch
rescue StandardError rescue StandardError
opoo "Failed to fetch from URL: #{url}" opoo "Failed to fetch from URL: #{url}"
else else
verify_download_integrity fetched if fetched.kind_of? Pathname verify_download_integrity(cached_download) if cached_download.file?
end end
end end

1
Library/Homebrew/download_strategy.rb
View File

@ -31,7 +31,6 @@ class AbstractDownloadStrategy
end end
class CurlDownloadStrategy < AbstractDownloadStrategy class CurlDownloadStrategy < AbstractDownloadStrategy
attr_reader :tarball_path
attr_accessor :local_bottle_path attr_accessor :local_bottle_path
def initialize name, package def initialize name, package