Merge pull request #15552 from MikeMcQuaid/info_github_packages_downloads

cmd/info: add (undocumented) --github-packages-downloads option.
This commit is contained in:
Mike McQuaid 2023-06-16 15:58:04 +01:00 committed by GitHub
commit 0606b63136
No known key found for this signature in database
GPG Key ID: 4AEE18F83AFDEB23
2 changed files with 51 additions and 3 deletions

View File

@ -42,6 +42,9 @@ module Homebrew
"The value for <category> must be `install`, `install-on-request` or `build-error`; " \ "The value for <category> must be `install`, `install-on-request` or `build-error`; " \
"`cask-install` or `os-version` may be specified if <formula> is not. " \ "`cask-install` or `os-version` may be specified if <formula> is not. " \
"The default is `install`." "The default is `install`."
switch "--github-packages-downloads",
description: "Scrape GitHub Packages download counts from HTML for a core formula.",
hidden: true
switch "--github", switch "--github",
description: "Open the GitHub source page for <formula> and <cask> in a browser. " \ description: "Open the GitHub source page for <formula> and <cask> in a browser. " \
"To view the history locally: `brew log -p` <formula> or <cask>" "To view the history locally: `brew log -p` <formula> or <cask>"

View File

@ -182,7 +182,7 @@ options: options)
table_output(category, days, results, os_version: os_version, cask_install: cask_install) table_output(category, days, results, os_version: os_version, cask_install: cask_install)
end end
def get_analytics(json, args:) def output_analytics(json, args:)
full_analytics = args.analytics? || verbose? full_analytics = args.analytics? || verbose?
ohai "Analytics" ohai "Analytics"
@ -207,13 +207,58 @@ options: options)
end end
end end
# This method is undocumented because it is not intended for general use.
# It relies on screen scraping some GitHub HTML that's not available as an API.
# This seems very likely to break in the future.
# That said, it's the only way to get the data we want right now.
def output_github_packages_downloads(formula, args:)
return unless args.github_packages_downloads?
return unless formula.core_formula?
escaped_formula_name = GitHubPackages.image_formula_name(formula.name)
formula_url_suffix = "container/core%2F#{escaped_formula_name}/"
formula_url = "https://github.com/Homebrew/homebrew-core/pkgs/#{formula_url_suffix}"
output = Utils::Curl.curl_output("--fail", formula_url)
return unless output.success?
formula_version_urls = output.stdout
.scan(%r{/orgs/Homebrew/packages/#{formula_url_suffix}\d+\?tag=[^"]+})
.map do |url|
url.sub("/orgs/Homebrew/packages/", "/Homebrew/homebrew-core/pkgs/")
end
return if formula_version_urls.empty?
thirty_day_download_count = 0
formula_version_urls.each do |formula_version_url_suffix|
formula_version_url = "https://github.com#{formula_version_url_suffix}"
output = Utils::Curl.curl_output("--fail", formula_version_url)
next unless output.success?
last_thirty_days_match = output.stdout.match(
%r{<span class="[\s\-a-z]*">Last 30 days</span>\s*<span class="[\s\-a-z]*">([\d.M,]+)</span>}m,
)
next if last_thirty_days_match.blank?
last_thirty_days_downloads = last_thirty_days_match.captures.first.tr(",", "")
thirty_day_download_count += if (millions_match = last_thirty_days_downloads.match(/(\d+\.\d+)M/).presence)
millions_match.captures.first.to_i * 1_000_000
else
last_thirty_days_downloads.to_i
end
end
ohai "GitHub Packages Downloads"
puts "#{number_readable(thirty_day_download_count)} (30 days)"
end
def formula_output(formula, args:) def formula_output(formula, args:)
return if Homebrew::EnvConfig.no_analytics? || Homebrew::EnvConfig.no_github_api? return if Homebrew::EnvConfig.no_analytics? || Homebrew::EnvConfig.no_github_api?
json = Homebrew::API::Formula.fetch formula.name json = Homebrew::API::Formula.fetch formula.name
return if json.blank? || json["analytics"].blank? return if json.blank? || json["analytics"].blank?
get_analytics(json, args: args) output_analytics(json, args: args)
output_github_packages_downloads(formula, args: args)
rescue ArgumentError rescue ArgumentError
# Ignore failed API requests # Ignore failed API requests
nil nil
@ -225,7 +270,7 @@ options: options)
json = Homebrew::API::Cask.fetch cask.token json = Homebrew::API::Cask.fetch cask.token
return if json.blank? || json["analytics"].blank? return if json.blank? || json["analytics"].blank?
get_analytics(json, args: args) output_analytics(json, args: args)
rescue ArgumentError rescue ArgumentError
# Ignore failed API requests # Ignore failed API requests
nil nil