-
Notifications
You must be signed in to change notification settings - Fork 21
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
Add a pip convenience symlink also for the python 3.9 container #2077
Conversation
Created a staging project on OBS for 6: home:defolos:BCI:Staging:SLE-15-SP6:6-2077 Build ResultsRepository
Repository
Repository
Repository
Repository
Repository
Repository
Repository
Build succeeded ✅ To run BCI-tests against this PR, use the following command: OS_VERSION=15.6 TARGET=custom BASEURL=registry.opensuse.org/home/defolos/bci/staging/sle-15-sp6/6-2077/ tox -- -n auto The following images can be pulled from the staging project:
|
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
LGTM, just one thing to consider: do we want to convert this script so that it fails once the symlink is in place? Otherwise I'm afraid it'll linger around for all eternity ™️
34f7522
to
55c96c7
Compare
I don't expect the underlying bug, which is python 3.9 specific, to be fixed as there is an explicit rm on the pip link in the package and no update-alternatives hooks which all the other pip variants have. |
the python3.9-pip package does not provide it for historic reasons, and the testsuite expects it to be there (rightfully). Co-authored-by: Dan Čermák <[email protected]>
55c96c7
to
500a648
Compare
the python3.9-pip package does not provide it for historic reasons, and the testsuite expects it to be there (rightfully).