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

update-python-resources: show pip install failure when --verbose

Browse Source 
Also use `--disable-pip-version-check` to remove pip update notices
This commit is contained in:
Michael Cho 2023-12-30 11:23:53 -05:00
parent 04d8ae20f3
commit 10659a1d61
No known key found for this signature in database
GPG Key ID: 55E85E28A7CD1E85
2 changed files with 19 additions and 11 deletions
Show Stats Download Patch File Download Diff File Expand all files Collapse all files

1
Library/Homebrew/dev-cmd/update-python-resources.rb
View File

@ -45,6 +45,7 @@ module Homebrew
exclude_packages: args.exclude_packages, exclude_packages: args.exclude_packages,
print_only: args.print_only?, print_only: args.print_only?,
silent: args.silent?, silent: args.silent?,
verbose: args.verbose?,
ignore_non_pypi_packages: args.ignore_non_pypi_packages? ignore_non_pypi_packages: args.ignore_non_pypi_packages?
end end
end end

29
Library/Homebrew/utils/pypi.rb
View File

@ -199,11 +199,12 @@ module PyPI
exclude_packages: T.nilable(T::Array[String]), exclude_packages: T.nilable(T::Array[String]),
print_only: T.nilable(T::Boolean), print_only: T.nilable(T::Boolean),
silent: T.nilable(T::Boolean), silent: T.nilable(T::Boolean),
verbose: T.nilable(T::Boolean),
ignore_non_pypi_packages: T.nilable(T::Boolean), ignore_non_pypi_packages: T.nilable(T::Boolean),
).returns(T.nilable(T::Boolean)) ).returns(T.nilable(T::Boolean))
} }
def self.update_python_resources!(formula, version: nil, package_name: nil, extra_packages: nil, def self.update_python_resources!(formula, version: nil, package_name: nil, extra_packages: nil,
exclude_packages: nil, print_only: false, silent: false, exclude_packages: nil, print_only: false, silent: false, verbose: false,
ignore_non_pypi_packages: false) ignore_non_pypi_packages: false)
auto_update_list = formula.tap&.pypi_formula_mappings auto_update_list = formula.tap&.pypi_formula_mappings
@ -289,21 +290,23 @@ module PyPI
ensure_formula_installed!(python_name) ensure_formula_installed!(python_name)
# Resolve the dependency tree of all input packages # Resolve the dependency tree of all input packages
ohai "Retrieving PyPI dependencies for \"#{input_packages.join(" ")}\"..." if !print_only && !silent show_info = !print_only && !silent
found_packages = pip_report(input_packages, python_name: python_name) ohai "Retrieving PyPI dependencies for \"#{input_packages.join(" ")}\"..." if show_info
found_packages = pip_report(input_packages, python_name: python_name, print_stderr: verbose && show_info)
# Resolve the dependency tree of excluded packages to prune the above # Resolve the dependency tree of excluded packages to prune the above
exclude_packages.delete_if { |package| found_packages.exclude? package } exclude_packages.delete_if { |package| found_packages.exclude? package }
ohai "Retrieving PyPI dependencies for excluded \"#{exclude_packages.join(" ")}\"..." if !print_only && !silent ohai "Retrieving PyPI dependencies for excluded \"#{exclude_packages.join(" ")}\"..." if show_info
exclude_packages = pip_report(exclude_packages, python_name: python_name) + [Package.new(main_package.name)] exclude_packages = pip_report(exclude_packages, python_name: python_name, print_stderr: verbose && show_info)
exclude_packages += [Package.new(main_package.name)]
new_resource_blocks = "" new_resource_blocks = ""
found_packages.sort.each do |package| found_packages.sort.each do |package|
if exclude_packages.include? package if exclude_packages.include? package
ohai "Excluding \"#{package}\"" if !print_only && !silent ohai "Excluding \"#{package}\"" if show_info
next next
end end
ohai "Getting PyPI info for \"#{package}\"" if !print_only && !silent ohai "Getting PyPI info for \"#{package}\"" if show_info
name, url, checksum = package.pypi_info name, url, checksum = package.pypi_info
# Fail if unable to find name, url or checksum for any resource # Fail if unable to find name, url or checksum for any resource
if name.blank? if name.blank?
@ -358,12 +361,16 @@ module PyPI
name.gsub(/[-_.]+/, "-").downcase name.gsub(/[-_.]+/, "-").downcase
end end
def self.pip_report(packages, python_name: "python") def self.pip_report(packages, python_name: "python", print_stderr: false)
return [] if packages.blank? return [] if packages.blank?
command = [Formula[python_name].libexec/"bin/python", "-m", "pip", "install", "-q", "--dry-run", command = [
"--ignore-installed", "--report=/dev/stdout", *packages.map(&:to_s)] Formula[python_name].libexec/"bin/python", "-m", "pip", "install", "-q", "--disable-pip-version-check",
pip_output = Utils.popen_read({ "PIP_REQUIRE_VIRTUALENV" => "false" }, *command) "--dry-run", "--ignore-installed", "--report=/dev/stdout", *packages.map(&:to_s)
]
options = {}
options[:err] = :err if print_stderr
pip_output = Utils.popen_read({ "PIP_REQUIRE_VIRTUALENV" => "false" }, *command, **options)
unless $CHILD_STATUS.success? unless $CHILD_STATUS.success?
odie <<~EOS odie <<~EOS
Unable to determine dependencies for "#{packages.join(" ")}" because of a failure when running Unable to determine dependencies for "#{packages.join(" ")}" because of a failure when running