FS#69765 - [python-inflect] missing makedepend on python-pip

Attached to Project: Community Packages
Opened by Kevin Mihelich (kmihelich) - Wednesday, 24 February 2021, 20:14 GMT
Last edited by David Runge (dvzrv) - Monday, 01 March 2021, 09:22 GMT
Task Type Bug Report
Category Packages
Status Closed
Assigned To David Runge (dvzrv)
Architecture All
Severity Low
Priority Normal
Reported Version
Due in Version Undecided
Due Date Undecided
Percent Complete 100%
Votes 0
Private No

Details

Description: Checkdepends are being used as makedepends. When building without checks, the package fails to build. Missing a makedepend on python-pip.


Additional info:
* package version: 5.2.0-1
* log:
==> Starting build()...
WARNING: The wheel package is not available.
/usr/bin/python: No module named pip
Traceback (most recent call last):
File "/usr/lib/python3.9/site-packages/setuptools/installer.py", line 75, in fetch_build_egg
subprocess.check_call(cmd)
File "/usr/lib/python3.9/subprocess.py", line 373, in check_call
raise CalledProcessError(retcode, cmd)
subprocess.CalledProcessError: Command '['/usr/bin/python', '-m', 'pip', '--disable-pip-version-check', 'wheel', '--no-deps', '-w', '/tmp/tmpxfar36vp', '--quiet', 'toml']' returned non-zero exit status 1.

Steps to reproduce: Build in a clean chroot passing --nocheck to makepkg
This task depends upon

Closed by  David Runge (dvzrv)
Monday, 01 March 2021, 09:22 GMT
Reason for closing:  Fixed
Additional comments about closing:  Fixed with python-inflect 5.2.0-3
Comment by David Runge (dvzrv) - Thursday, 25 February 2021, 08:34 GMT
@kmihelich: Thanks for the report!

This indeed needs to be added. Good catch!
Comment by Eli Schwartz (eschwartz) - Sunday, 28 February 2021, 22:12 GMT
The missing makedepends is for setuptools_scm[toml] instead of setuptools_scm, and pip here is trying to install the toml module over the network. This is never the solution.

Please remove python-pip and add python-toml instead...
Comment by David Runge (dvzrv) - Monday, 01 March 2021, 09:18 GMT
@eschwartz: Thanks! Will do.

Loading...