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

Merge pull request #11269 from MikeMcQuaid/pr_upload_bottle_verbose

Browse Source 
dev-cmd: add more --verbose bottle, pr-upload output.
This commit is contained in:
Mike McQuaid 2021-04-28 13:16:38 +01:00 committed by GitHub
commit 9e6c4f1fc7
No known key found for this signature in database
GPG Key ID: 4AEE18F83AFDEB23
2 changed files with 9 additions and 5 deletions
Show Stats Download Patch File Download Diff File Expand all files Collapse all files

3
Library/Homebrew/dev-cmd/bottle.rb
View File

@ -662,12 +662,15 @@ module Homebrew
all_bottle_hash = { formula_name => all_bottle_formula_hash }
"Copying #{local_filename} to #{all_local_filename}" if args.verbose?
FileUtils.cp local_filename, all_local_filename
all_local_json_path = Pathname(all_local_json_filename)
all_local_json_path.unlink if all_local_json_path.exist?
all_local_json_path.write(JSON.pretty_generate(all_bottle_hash))
end
"Removing #{local_filename} and #{local_json_filename}" if args.verbose?
FileUtils.rm_f [local_filename, local_json_filename]
end
end

11
Library/Homebrew/dev-cmd/pr-upload.rb
View File

@ -84,17 +84,18 @@ module Homebrew
end
end
def json_files_and_bottles_hash_(root_url)
def json_files_and_bottles_hash(args)
json_files = Dir["*.bottle.json"]
odie "No bottle JSON files found in the current working directory" if json_files.blank?
puts "Reading JSON files: #{json_files.join(", ")}" if args.verbose?
bottles_hash = json_files.reduce({}) do |hash, json_file|
hash.deep_merge(JSON.parse(File.read(json_file)))
end
if root_url
if args.root_url
bottles_hash.each_value do |bottle_hash|
bottle_hash["bottle"]["root_url"] = root_url
bottle_hash["bottle"]["root_url"] = args.root_url
end
end
@ -104,7 +105,7 @@ module Homebrew
def pr_upload
args = pr_upload_args.parse
json_files, bottles_hash = json_files_and_bottles_hash_(args.root_url)
json_files, bottles_hash = json_files_and_bottles_hash(args)
bottle_args = ["bottle", "--merge", "--write"]
bottle_args << "--verbose" if args.verbose?
@ -145,7 +146,7 @@ module Homebrew
# Reload the JSON files (in case `brew bottle --merge` generated
# `all: $SHA256` bottles)
_, bottles_hash = json_files_and_bottles_hash_(args.root_url)
_, bottles_hash = json_files_and_bottles_hash(args)
# Check the bottle commits did not break `brew audit`
unless args.no_commit?