brew/Library/Homebrew/dev-cmd/bump-formula-pr.rb

536 lines
21 KiB
Ruby
Raw Normal View History

# typed: true
# frozen_string_literal: true
require "formula"
2019-04-17 18:25:08 +09:00
require "cli/parser"
require "utils/pypi"
require "utils/tar"
module Homebrew
2016-09-26 01:44:51 +02:00
module_function
2020-10-20 12:03:48 +02:00
sig { returns(CLI::Parser) }
def bump_formula_pr_args
Homebrew::CLI::Parser.new do
description <<~EOS
2019-08-06 14:20:27 -04:00
Create a pull request to update <formula> with a new URL or a new tag.
2018-10-02 19:54:22 +05:30
If a <URL> is specified, the <SHA-256> checksum of the new download should also
be specified. A best effort to determine the <SHA-256> will be made if not supplied
by the user.
2018-10-02 19:54:22 +05:30
If a <tag> is specified, the Git commit <revision> corresponding to that tag
should also be specified. A best effort to determine the <revision> will be made
if the value is not supplied by the user.
If a <version> is specified, a best effort to determine the <URL> and <SHA-256> or
the <tag> and <revision> will be made if both values are not supplied by the user.
2018-09-28 21:39:52 +05:30
*Note:* this command cannot be used to transition a formula from a
URL-and-SHA-256 style specification into a tag-and-revision style specification,
nor vice versa. It must use whichever style specification the formula already uses.
2018-09-28 21:39:52 +05:30
EOS
2018-10-02 19:54:22 +05:30
switch "-n", "--dry-run",
2019-04-30 08:44:35 +01:00
description: "Print what would be done rather than doing it."
switch "--write-only",
description: "Make the expected file modifications without taking any Git actions."
switch "--commit",
depends_on: "--write-only",
2022-06-28 10:09:59 +01:00
description: "When passed with `--write-only`, generate a new commit after writing changes " \
"to the formula file."
2018-10-28 01:13:50 +10:00
switch "--no-audit",
2019-04-30 08:44:35 +01:00
description: "Don't run `brew audit` before opening the PR."
2018-10-02 19:54:22 +05:30
switch "--strict",
2019-04-30 08:44:35 +01:00
description: "Run `brew audit --strict` before opening the PR."
switch "--online",
description: "Run `brew audit --online` before opening the PR."
2018-10-02 19:54:22 +05:30
switch "--no-browse",
2019-04-30 08:44:35 +01:00
description: "Print the pull request URL instead of opening in a browser."
switch "--no-fork",
description: "Don't try to fork the repository."
comma_array "--mirror",
2022-06-28 10:09:59 +01:00
description: "Use the specified <URL> as a mirror URL. If <URL> is a comma-separated list " \
"of URLs, multiple mirrors will be added."
2021-04-15 19:38:10 +02:00
flag "--fork-org=",
description: "Use the specified GitHub organization for forking."
flag "--version=",
2022-06-28 10:09:59 +01:00
description: "Use the specified <version> to override the value parsed from the URL or tag. Note " \
"that `--version=0` can be used to delete an existing version override from a " \
2019-04-30 08:44:35 +01:00
"formula if it has become redundant."
flag "--message=",
description: "Prepend <message> to the default pull request message."
flag "--url=",
2022-06-28 10:09:59 +01:00
description: "Specify the <URL> for the new download. If a <URL> is specified, the <SHA-256> " \
2019-04-30 08:44:35 +01:00
"checksum of the new download should also be specified."
flag "--sha256=",
2019-04-30 08:44:35 +01:00
depends_on: "--url=",
description: "Specify the <SHA-256> checksum of the new download."
flag "--tag=",
2019-04-30 08:44:35 +01:00
description: "Specify the new git commit <tag> for the formula."
flag "--revision=",
2022-06-28 10:09:59 +01:00
description: "Specify the new commit <revision> corresponding to the specified git <tag> " \
"or specified <version>."
switch "-f", "--force",
description: "Remove all mirrors if `--mirror` was not specified."
switch "--install-dependencies",
description: "Install missing dependencies required to update resources."
flag "--python-package-name=",
2022-06-28 10:09:59 +01:00
description: "Use the specified <package-name> when finding Python resources for <formula>. " \
"If no package name is specified, it will be inferred from the formula's stable URL."
comma_array "--python-extra-packages=",
description: "Include these additional Python packages when finding resources."
comma_array "--python-exclude-packages=",
description: "Exclude these Python packages when finding resources."
2020-07-30 18:40:10 +02:00
conflicts "--dry-run", "--write-only"
conflicts "--no-audit", "--strict"
conflicts "--no-audit", "--online"
conflicts "--url", "--tag"
2021-01-10 14:26:40 -05:00
named_args :formula, max: 1, without_api: true
end
end
def bump_formula_pr
2020-07-30 18:40:10 +02:00
args = bump_formula_pr_args.parse
if args.revision.present? && args.tag.nil? && args.version.nil?
raise UsageError, "`--revision` must be passed with either `--tag` or `--version`!"
end
# As this command is simplifying user-run commands then let's just use a
# user path, too.
ENV["PATH"] = PATH.new(ORIGINAL_PATHS).to_s
2017-11-07 07:48:00 +00:00
# Use the user's browser, too.
2020-04-05 15:44:50 +01:00
ENV["BROWSER"] = Homebrew::EnvConfig.browser
2017-11-07 07:48:00 +00:00
formula = args.named.to_formulae.first
new_url = args.url
raise FormulaUnspecifiedError if formula.blank?
odie "This formula is disabled!" if formula.disabled?
2021-01-24 09:34:53 -08:00
odie "This formula is deprecated and does not build!" if formula.deprecation_reason == :does_not_build
2021-01-08 11:42:37 -08:00
odie "This formula is not in a tap!" if formula.tap.blank?
odie "This formula's tap is not a Git repository!" unless formula.tap.git?
odie <<~EOS unless formula.tap.allow_bump?(formula.name)
Whoops, the #{formula.name} formula has its version update
pull requests automatically opened by BrewTestBot!
We'd still love your contributions, though, so try another one
that's not in the autobump list:
#{Formatter.url("#{formula.tap.remote}/blob/master/.github/autobump.txt")}
EOS
2021-01-10 09:47:20 -08:00
formula_spec = formula.stable
odie "#{formula}: no stable specification found!" if formula_spec.blank?
# This will be run by `brew audit` later so run it first to not start
# spamming during normal output.
Homebrew.install_bundler_gems!(groups: ["audit", "style"]) unless args.no_audit?
tap_remote_repo = formula.tap.full_name || formula.tap.remote_repo
remote = "origin"
2023-04-17 09:57:06 -04:00
remote_branch = formula.tap.git_repo.origin_branch_name
previous_branch = "-"
2024-03-07 16:20:20 +00:00
check_open_pull_requests(formula, tap_remote_repo, args:)
new_version = args.version
2024-03-07 16:20:20 +00:00
check_new_version(formula, tap_remote_repo, version: new_version, args:) if new_version.present?
opoo "This formula has patches that may be resolved upstream." if formula.patchlist.present?
if formula.resources.any? { |resource| !resource.name.start_with?("homebrew-") }
opoo "This formula has resources that may need to be updated."
end
old_mirrors = formula_spec.mirrors
new_mirrors ||= args.mirror
new_mirror ||= determine_mirror(new_url)
new_mirrors ||= [new_mirror] if new_mirror.present?
2024-03-07 16:20:20 +00:00
check_for_mirrors(formula, old_mirrors, new_mirrors, args:) if new_url.present?
old_hash = formula_spec.checksum&.hexdigest
new_hash = args.sha256
new_tag = args.tag
new_revision = args.revision
old_url = formula_spec.url
old_tag = formula_spec.specs[:tag]
2021-01-10 09:47:20 -08:00
old_formula_version = formula_version(formula)
old_version = old_formula_version.to_s
forced_version = new_version.present?
new_url_hash = if new_url.present? && new_hash.present?
2024-03-07 16:20:20 +00:00
check_new_version(formula, tap_remote_repo, url: new_url, args:) if new_version.blank?
true
2021-01-10 09:47:20 -08:00
elsif new_tag.present? && new_revision.present?
2024-03-07 16:20:20 +00:00
check_new_version(formula, tap_remote_repo, url: old_url, tag: new_tag, args:) if new_version.blank?
false
elsif old_hash.blank?
if new_tag.blank? && new_version.blank? && new_revision.blank?
2023-02-10 23:15:40 -05:00
raise UsageError, "#{formula}: no `--tag` or `--version` argument specified!"
end
if old_tag.present?
new_tag ||= old_tag.gsub(old_version, new_version)
if new_tag == old_tag
odie <<~EOS
You need to bump this formula manually since the new tag
and old tag are both #{new_tag}.
EOS
end
2024-03-07 16:20:20 +00:00
check_new_version(formula, tap_remote_repo, url: old_url, tag: new_tag, args:) if new_version.blank?
2023-03-06 09:49:53 -08:00
resource_path, forced_version = fetch_resource_and_forced_version(formula, new_version, old_url, tag: new_tag)
2021-03-17 15:34:20 +00:00
new_revision = Utils.popen_read("git", "-C", resource_path.to_s, "rev-parse", "-q", "--verify", "HEAD")
new_revision = new_revision.strip
elsif new_revision.blank?
odie "#{formula}: the current URL requires specifying a `--revision=` argument."
end
false
elsif new_url.blank? && new_version.blank?
2023-02-10 23:15:40 -05:00
raise UsageError, "#{formula}: no `--url` or `--version` argument specified!"
else
new_url ||= PyPI.update_pypi_url(old_url, new_version)
if new_url.blank?
new_url = update_url(old_url, old_version, new_version)
if new_mirrors.blank? && old_mirrors.present?
new_mirrors = old_mirrors.map do |old_mirror|
update_url(old_mirror, old_version, new_version)
end
end
end
if new_url == old_url
odie <<~EOS
2020-07-20 07:44:42 -07:00
You need to bump this formula manually since the new URL
and old URL are both:
#{new_url}
EOS
end
2024-03-07 16:20:20 +00:00
check_new_version(formula, tap_remote_repo, url: new_url, args:) if new_version.blank?
2023-03-06 09:49:53 -08:00
resource_path, forced_version = fetch_resource_and_forced_version(formula, new_version, new_url)
Utils::Tar.validate_file(resource_path)
new_hash = resource_path.sha256
end
replacement_pairs = []
2021-01-10 09:47:20 -08:00
if formula.revision.nonzero?
replacement_pairs << [
/^ revision \d+\n(\n( head "))?/m,
"\\2",
]
end
replacement_pairs += formula_spec.mirrors.map do |mirror|
[
2020-06-02 09:49:23 +01:00
/ +mirror "#{Regexp.escape(mirror)}"\n/m,
"",
]
end
2016-09-04 11:28:02 -07:00
replacement_pairs += if new_url_hash.present?
[
[
/#{Regexp.escape(formula_spec.url)}/,
new_url,
],
[
old_hash,
new_hash,
],
]
elsif new_tag.present?
[
[
/tag:(\s+")#{formula_spec.specs[:tag]}(?=")/,
"tag:\\1#{new_tag}\\2",
],
[
formula_spec.specs[:revision],
new_revision,
],
]
elsif new_url.present?
[
[
/#{Regexp.escape(formula_spec.url)}/,
new_url,
],
[
formula_spec.specs[:revision],
new_revision,
],
]
else
[
[
formula_spec.specs[:revision],
new_revision,
],
]
end
2021-01-10 09:47:20 -08:00
old_contents = formula.path.read
if new_mirrors.present?
replacement_pairs << [
/^( +)(url "#{Regexp.escape(new_url)}"[^\n]*?\n)/m,
"\\1\\2\\1mirror \"#{new_mirrors.join("\"\n\\1mirror \"")}\"\n",
]
2016-09-04 11:28:02 -07:00
end
if forced_version && new_version != "0"
2021-01-10 09:47:20 -08:00
replacement_pairs << if old_contents.include?("version \"#{old_formula_version}\"")
2020-07-20 07:44:42 -07:00
[
"version \"#{old_formula_version}\"",
"version \"#{new_version}\"",
]
elsif new_mirrors.present?
2020-07-20 07:44:42 -07:00
[
/^( +)(mirror "#{Regexp.escape(new_mirrors.last)}"\n)/m,
"\\1\\2\\1version \"#{new_version}\"\n",
]
elsif new_url.present?
2020-07-20 07:44:42 -07:00
[
/^( +)(url "#{Regexp.escape(new_url)}"[^\n]*?\n)/m,
2020-07-20 07:44:42 -07:00
"\\1\\2\\1version \"#{new_version}\"\n",
]
elsif new_revision.present?
[
/^( {2})( +)(:revision => "#{new_revision}"\n)/m,
"\\1\\2\\3\\1version \"#{new_version}\"\n",
]
end
2020-07-20 07:44:42 -07:00
elsif forced_version && new_version == "0"
replacement_pairs << [
/^ version "[\w.\-+]+"\n/m,
"",
]
end
2020-08-16 10:28:26 -07:00
new_contents = Utils::Inreplace.inreplace_pairs(formula.path,
replacement_pairs.uniq.compact,
read_only_run: args.dry_run?,
2020-08-16 10:28:26 -07:00
silent: args.quiet?)
2021-01-10 09:47:20 -08:00
new_formula_version = formula_version(formula, new_contents)
if new_formula_version < old_formula_version
formula.path.atomic_write(old_contents) unless args.dry_run?
2017-10-15 02:28:32 +02:00
odie <<~EOS
You need to bump this formula manually since changing the version
from #{old_formula_version} to #{new_formula_version} would be a downgrade.
EOS
elsif new_formula_version == old_formula_version
formula.path.atomic_write(old_contents) unless args.dry_run?
2017-10-15 02:28:32 +02:00
odie <<~EOS
2020-07-20 07:44:42 -07:00
You need to bump this formula manually since the new version
and old version are both #{new_formula_version}.
EOS
end
alias_rename = alias_update_pair(formula, new_formula_version)
if alias_rename.present?
ohai "Renaming alias #{alias_rename.first} to #{alias_rename.last}"
alias_rename.map! { |a| formula.tap.alias_dir/a }
end
unless args.dry_run?
resources_checked = PyPI.update_python_resources! formula,
version: new_formula_version.to_s,
package_name: args.python_package_name,
extra_packages: args.python_extra_packages,
exclude_packages: args.python_exclude_packages,
install_dependencies: args.install_dependencies?,
silent: args.quiet?,
ignore_non_pypi_packages: true
end
2024-03-07 16:20:20 +00:00
run_audit(formula, alias_rename, old_contents, args:)
pr_message = "Created with `brew bump-formula-pr`."
if resources_checked.nil? && formula.resources.any? { |resource| !resource.name.start_with?("homebrew-") }
pr_message += <<~EOS
- [ ] `resource` blocks have been checked for updates.
EOS
end
if new_url =~ %r{^https://github\.com/([\w-]+)/([\w-]+)/archive/refs/tags/(v?[.0-9]+)\.tar\.}
owner = Regexp.last_match(1)
repo = Regexp.last_match(2)
tag = Regexp.last_match(3)
github_release_data = begin
GitHub::API.open_rest("#{GitHub::API_URL}/repos/#{owner}/#{repo}/releases/tags/#{tag}")
rescue GitHub::API::HTTPNotFoundError
# If this is a 404: we can't do anything.
nil
end
2023-03-08 14:28:34 +01:00
if github_release_data.present?
pre = "pre" if github_release_data["prerelease"].present?
2023-03-08 14:28:34 +01:00
pr_message += <<~XML
<details>
<summary>#{pre}release notes</summary>
<pre>#{github_release_data["body"]}</pre>
</details>
XML
2023-03-08 14:28:34 +01:00
end
end
2020-09-04 06:18:34 -07:00
pr_info = {
sourcefile_path: formula.path,
2024-03-07 16:20:20 +00:00
old_contents:,
2020-09-04 06:18:34 -07:00
additional_files: alias_rename,
2024-03-07 16:20:20 +00:00
remote:,
remote_branch:,
2020-09-04 06:18:34 -07:00
branch_name: "bump-#{formula.name}-#{new_formula_version}",
commit_message: "#{formula.name} #{new_formula_version}",
2024-03-07 16:20:20 +00:00
previous_branch:,
2020-09-04 06:18:34 -07:00
tap: formula.tap,
2024-03-07 16:20:20 +00:00
tap_remote_repo:,
pr_message:,
2020-09-04 06:18:34 -07:00
}
2024-03-07 16:20:20 +00:00
GitHub.create_bump_pr(pr_info, args:)
end
def determine_mirror(url)
case url
when %r{.*ftp\.gnu\.org/gnu.*}
url.sub "ftp.gnu.org/gnu", "ftpmirror.gnu.org"
when %r{.*download\.savannah\.gnu\.org/*}
url.sub "download.savannah.gnu.org", "download-mirror.savannah.gnu.org"
when %r{.*www\.apache\.org/dyn/closer\.lua\?path=.*}
url.sub "www.apache.org/dyn/closer.lua?path=", "archive.apache.org/dist/"
when %r{.*mirrors\.ocf\.berkeley\.edu/debian.*}
url.sub "mirrors.ocf.berkeley.edu/debian", "mirrorservice.org/sites/ftp.debian.org/debian"
end
end
def check_for_mirrors(formula, old_mirrors, new_mirrors, args:)
return if new_mirrors.present? || old_mirrors.empty?
if args.force?
opoo "#{formula}: Removing all mirrors because a `--mirror=` argument was not specified."
else
odie <<~EOS
#{formula}: a `--mirror=` argument for updating the mirror URL(s) was not specified.
Use `--force` to remove all mirrors.
EOS
end
end
sig { params(old_url: String, old_version: String, new_version: String).returns(String) }
def update_url(old_url, old_version, new_version)
new_url = old_url.gsub(old_version, new_version)
return new_url if (old_version_parts = old_version.split(".")).length < 2
return new_url if (new_version_parts = new_version.split(".")).length != old_version_parts.length
partial_old_version = T.must(old_version_parts[0..-2]).join(".")
partial_new_version = T.must(new_version_parts[0..-2]).join(".")
new_url.gsub(%r{/(v?)#{Regexp.escape(partial_old_version)}/}, "/\\1#{partial_new_version}/")
end
2023-03-06 09:49:53 -08:00
def fetch_resource_and_forced_version(formula, new_version, url, **specs)
resource = Resource.new
resource.url(url, **specs)
resource.owner = Resource.new(formula.name)
forced_version = new_version && new_version != resource.version.to_s
resource.version(new_version) if forced_version
odie "Couldn't identify version, specify it using `--version=`." if resource.version.blank?
[resource.fetch, forced_version]
end
2021-01-10 09:47:20 -08:00
def formula_version(formula, contents = nil)
spec = :stable
name = formula.name
path = formula.path
if contents.present?
Formulary.from_contents(name, path, contents, spec).version
else
Formulary::FormulaLoader.new(name, path).get_formula(spec).version
end
end
def check_open_pull_requests(formula, tap_remote_repo, args:)
GitHub.check_for_duplicate_pull_requests(formula.name, tap_remote_repo,
state: "open",
file: formula.path.relative_path_from(formula.tap.path).to_s,
quiet: args.quiet?)
end
def check_new_version(formula, tap_remote_repo, args:, version: nil, url: nil, tag: nil)
if version.nil?
2020-07-20 07:44:42 -07:00
specs = {}
specs[:tag] = tag if tag.present?
version = Version.detect(url, **specs).to_s
return if version.blank?
2020-07-20 07:44:42 -07:00
end
2021-01-24 09:34:53 -08:00
check_throttle(formula, version)
2024-03-07 16:20:20 +00:00
check_closed_pull_requests(formula, tap_remote_repo, args:, version:)
2021-01-24 09:34:53 -08:00
end
def check_throttle(formula, new_version)
throttled_rate = formula.tap.audit_exceptions.dig(:throttled_formulae, formula.name)
return if throttled_rate.blank?
formula_suffix = Version.new(new_version).patch.to_i
return if formula_suffix.modulo(throttled_rate).zero?
odie "#{formula} should only be updated every #{throttled_rate} releases on multiples of #{throttled_rate}"
end
def check_closed_pull_requests(formula, tap_remote_repo, args:, version:)
# if we haven't already found open requests, try for an exact match across closed requests
GitHub.check_for_duplicate_pull_requests(formula.name, tap_remote_repo,
2024-03-07 16:20:20 +00:00
version:,
2021-01-24 09:34:53 -08:00
state: "closed",
file: formula.path.relative_path_from(formula.tap.path).to_s,
quiet: args.quiet?)
end
def alias_update_pair(formula, new_formula_version)
versioned_alias = formula.aliases.grep(/^.*@\d+(\.\d+)?$/).first
return if versioned_alias.nil?
name, old_alias_version = versioned_alias.split("@")
new_alias_regex = (old_alias_version.split(".").length == 1) ? /^\d+/ : /^\d+\.\d+/
new_alias_version, = *new_formula_version.to_s.match(new_alias_regex)
return if Version.new(new_alias_version) <= Version.new(old_alias_version)
[versioned_alias, "#{name}@#{new_alias_version}"]
end
def run_audit(formula, alias_rename, old_contents, args:)
audit_args = ["--formula"]
audit_args << "--strict" if args.strict?
audit_args << "--online" if args.online?
if args.dry_run?
if args.no_audit?
ohai "Skipping `brew audit`"
elsif audit_args.present?
ohai "brew audit #{audit_args.join(" ")} #{formula.path.basename}"
else
ohai "brew audit #{formula.path.basename}"
end
return
end
FileUtils.mv alias_rename.first, alias_rename.last if alias_rename.present?
failed_audit = false
if args.no_audit?
ohai "Skipping `brew audit`"
elsif audit_args.present?
system HOMEBREW_BREW_FILE, "audit", *audit_args, formula.full_name
failed_audit = !$CHILD_STATUS.success?
else
system HOMEBREW_BREW_FILE, "audit", formula.full_name
failed_audit = !$CHILD_STATUS.success?
end
return unless failed_audit
formula.path.atomic_write(old_contents)
FileUtils.mv alias_rename.last, alias_rename.first if alias_rename.present?
odie "`brew audit` failed!"
end
end