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

Merge pull request #4778 from DomT4/gocache_follow_up

Browse Source 
cleanup: fix go_cache cleanup
This commit is contained in:
Mike McQuaid 2018-09-02 10:16:37 +01:00 committed by GitHub
commit 794fe5d21d
No known key found for this signature in database
GPG Key ID: 4AEE18F83AFDEB23
1 changed files with 7 additions and 0 deletions
Show Stats Download Patch File Download Diff File Expand all files Collapse all files

7
Library/Homebrew/cleanup.rb
View File

@ -39,6 +39,12 @@ module CleanupRefinement
directory? && %w[go_cache glide_home java_cache npm_cache gclient_cache].include?(basename.to_s) directory? && %w[go_cache glide_home java_cache npm_cache gclient_cache].include?(basename.to_s)
end end
def go_cache_directory?
# Go makes its cache contents read-only to ensure cache integrity,
# which makes sense but is something we need to undo for cleanup.
directory? && %w[go_cache].include?(basename.to_s)
end
def prune?(days) def prune?(days)
return false unless days return false unless days
return true if days.zero? return true if days.zero?
@ -228,6 +234,7 @@ module Homebrew
entries ||= [cache, cache/"Cask"].select(&:directory?).flat_map(&:children) entries ||= [cache, cache/"Cask"].select(&:directory?).flat_map(&:children)
entries.each do |path| entries.each do |path|
FileUtils.chmod_R 0755, path if path.go_cache_directory? && !dry_run?
next cleanup_path(path) { path.unlink } if path.incomplete? next cleanup_path(path) { path.unlink } if path.incomplete?
next cleanup_path(path) { FileUtils.rm_rf path } if path.nested_cache? next cleanup_path(path) { FileUtils.rm_rf path } if path.nested_cache?