I agree with the plan, but I wanted to note that deprecating a package
that other packages depend on requires a FESCo-approved Change[1].
(It is weird that the deprecation policy is so much more bureaucratic
than the retirement policy. By comparison, I don’t know of any policy
that would forbid summarily retiring a package that others depend on.
It’s a bad practice, and one might be asked to allow someone else to
unretire and maintain the package if the consequences are bad enough,
but I’ve never seen a blanket policy against it.)
[1]
https://docs.fedoraproject.org/en-US/packaging-guidelines/deprecating-packages/#_prerequisites_for_deprecation
-----
I opened PR’s (or in some cases committed directly) to remove the
pytest-runner dependency from the following packages for which I am a
direct or group/SIG co-maintainer, sending PR’s upstream where applicable.
python-astral:
https://src.fedoraproject.org/rpms/python-astral/pull-request/2
python-datrie:
https://src.fedoraproject.org/rpms/python-datrie/c/fac1efd277e73267234769353dcabc81d6274c5c?branch=rawhide
python-decopatch:
https://src.fedoraproject.org/rpms/python-decopatch/pull-request/1
python-ncclient:
https://src.fedoraproject.org/rpms/python-ncclient/c/7f19d76a0461ccc4c18eafd85a6e41f08c30aa83?branch=rawhide
I might take a look at a few others later if time allows. I doubt there
is any package from which the pytest-runner dependency can’t be removed
with a modest amount of effort.
On 12/23/24 6:52 PM, Michel Lind wrote:
Dear all,
pytest-runner has been considered deprecated upstream since 2019:
https://github.com/pytest-dev/pytest-runner/commit/78a492cb9f611a4fccaf6556c966b848885833ba
and the repo upstream was archived over a year ago (December 3, 2023).
As such, and given the test suite never worked (it ran 0 tests, see
https://kojipkgs.fedoraproject.org//packages/python-pytest-runner/4.0/24.fc41/data/logs/noarch/build.log)
prior to `setup.py test` being removed in setuptools 74
(https://bugzilla.redhat.com/show_bug.cgi?id=2319709) and that the only
test defined requires Internet access anyway - I've now disabled tests
and added import checks instead, and marked the package as
`deprecated()` - so this should not be used for new packages.
The upstream deprecation notice provided a guidance for switching away
from `pytest-runner` - you should always be able to just invoke `%pytest`
directly anyway
Affected packages (maintainers bcc-ed)
bst-external : python3dist(pytest-runner)
cozy : python3dist(pytest-runner)
pyp2rpm : python3dist(pytest-runner)
python-accuweather : python3dist(pytest-runner)
python-aioasuswrt : python3dist(pytest-runner)
python-aiohttp-sse-client : python3dist(pytest-runner)
python-arpeggio : python3dist(pytest-runner)
python-astral : python3dist(pytest-runner)
python-cid : python3dist(pytest-runner)
python-cs : python3dist(pytest-runner)
python-datrie : python3dist(pytest-runner)
python-decopatch : python3dist(pytest-runner)
python-devolo-home-control-api : python3dist(pytest-runner)
python-django-prometheus : python3dist(pytest-runner)
python-flask-caching : python3dist(pytest-runner)
python-flask-gravatar : python3dist(pytest-runner) >= 2.6.2
python-gios : python3dist(pytest-runner)
python-healpy : python3dist(pytest-runner)
python-inject : python3dist(pytest-runner)
python-makefun : python3dist(pytest-runner)
python-multiaddr : python3dist(pytest-runner)
python-multibase : python3dist(pytest-runner)
python-multicodec : python3dist(pytest-runner)
python-multihash : python3dist(pytest-runner)
python-ncclient : python3dist(pytest-runner)
python-nixio : python3dist(pytest-runner)
python-pytest-cases : python3dist(pytest-runner)
python-pytest-harvest : python3dist(pytest-runner)
python-pytest-steps : python3dist(pytest-runner)
python-reflink : python3dist(pytest-runner)
python-slackclient : python3dist(pytest-runner)
python-ssdeep : python3dist(pytest-runner)
python-token-bucket : python3dist(pytest-runner)
python-waterfurnace : python3dist(pytest-runner)
python-xdfile : python3dist(pytest-runner)
spec2scl : python3dist(pytest-runner)
vyper : python3dist(pytest-runner)
Best regards,
--
_______________________________________________
devel mailing list -- devel@xxxxxxxxxxxxxxxxxxxxxxx
To unsubscribe send an email to devel-leave@xxxxxxxxxxxxxxxxxxxxxxx
Fedora Code of Conduct: https://docs.fedoraproject.org/en-US/project/code-of-conduct/
List Guidelines: https://fedoraproject.org/wiki/Mailing_list_guidelines
List Archives: https://lists.fedoraproject.org/archives/list/devel@xxxxxxxxxxxxxxxxxxxxxxx
Do not reply to spam, report it: https://pagure.io/fedora-infrastructure/new_issue
