Request 941699 (accepted)

This request supersedes: request 941558 (Show diff)

Overview

Request 941699 accepted

- Using URLs for Patches looks like a great idea, but it is too
unstable. Please, don't.
- Add pytest-pr8664-py3.10-test_trial_error-fail.patch
* gh#pytest-dev/pytest#8664
- Add pytest-pr9417-py3.10.1-fail.patch
* gh#pytest-dev/pytest#9417
- Disable libalternatives usage until all python-pytest* packages
are on the same page (see sr#926611)
- Disable all update-alternatives install calls if libalternatives
is used.
- Use libalternatives instead of update-alternatives.

Created by mcepl over 2 years ago
In state accepted
Supersedes 941558

Submit python-pytest

Comments for request 941699 2
Comments for request 941558 3

Dominique Leuenberger (dimstar) - over 2 years ago

reviewer

61+%pre
62+# If libalternatives is used: Removing old update-alternatives entries.
63+%python_libalternatives_reset_alternative pytest

shouldn't that also be wrapped in if %{with libalternatives}?

Benjamin Greiner (bnavigator) - over 2 years ago

No, the check is within the macro itself.

Matej Cepl (mcepl) - over 2 years ago

source maintainer

@bnavigator Using URLs for patches seems like a good idea, but I find it too unstable for us. I’ve got already plenty of errors like this. Please, don’t do it.

Benjamin Greiner (bnavigator) - over 2 years ago

Merged PRs are usually safe for that. The problem was that a quilt refresh for the rebased patch also refreshed the second one.

Matej Cepl (mcepl) - over 2 years ago

source maintainer

That’s another reason … most of our patches are at least refreshed. Binary identity is really not that important.