Spec URL: https://decathorpe.fedorapeople.org/packages/poetry.spec SRPM URL: https://decathorpe.fedorapeople.org/packages/poetry-0.12.17-1.fc30.src.rpm Description: Poetry helps you declare, manage and install dependencies of Python projects, ensuring you have the right stack everywhere. Fedora Account System Username: decathorpe koji scratch build for rawhide: https://koji.fedoraproject.org/koji/taskinfo?taskID=36907719
Taking this review.
> Requires: python3-%{pypi_name} This should be a strictly versioned dependency, since you're splitting it between the Pythom module and the program. Is there any particular value in doing this over just letting it all be in the "poetry" binary package?
(In reply to Neal Gompa from comment #2) > > Requires: python3-%{pypi_name} > > This should be a strictly versioned dependency, since you're splitting it > between the Pythom module and the program. Right. I will fix this. Tomorrow ... > Is there any particular value in doing this over just letting it all be in > the "poetry" binary package? Not particularly. But I know that pyproject.toml actually uses API form the poetry python package and not the poetry binary itself. Now I need to sleep.
I made the dependency from poetry -> python3-poetry explicitly versioned. Also relaxed the dependency on pyrsistent so the package can actually be installed on rawhide as well as f30.
Review notes: [x]: Follows Fedora Packaging Guidelines [x]: Follows Fedora Python Packaging Guidelines [x]: Package includes license file and has correct tag [x]: Package builds and installs properly PACKAGE APPROVED
(fedscm-admin): The Pagure repository was created at https://src.fedoraproject.org/rpms/poetry
Built for rawhide: https://koji.fedoraproject.org/koji/buildinfo?buildID=1354153 Submitted for f30: https://bodhi.fedoraproject.org/updates/FEDORA-2019-458e2b6e9e