Overview
Two points, one is just a cosmetic, other not so:
The nitpick is
%{python_sitelib}/cloup-*.egg-info
. We prefer%{python_sitelib}/cloup-%{version}*-info
. First, when there is a problem with generated metadata, they often end up with directory%{python_sitelib}/cloup-0.0.0*-info
, so checking the version is a good Canary in a Coal Mine for that. Second, packages with%pyproject_wheel
(which is now preferred way of building packages) have the directory ending with.dist-info
, so I usually put that difference into asterisk.More important question is whether this package doesn’t belong more to
devel:languages:python:numeric
which is the home of our scientific/numerical packages. Do you expect any package outside of that project will ever use this one?
another one: you have twice BuildRequires: %{python_module pytest}
Thank you for you time.
- About the first one, sure!
- About the second one, cloup is a package to enrich python-click, which is in this project: https://build.opensuse.org/package/show/devel:languages:python/python-click