-
Notifications
You must be signed in to change notification settings - Fork 22
github: add --ignore-installed to pip install of requirements
#293
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
Conversation
25343b4 to
151b639
Compare
jameshcorbett
left a comment
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.
Looks OK but can you explain what's going on?
|
The rabbit master hopped to this PR too quickly! I'm fairly certain it's a problem with a mismatch between system installed packages in the container and what pip thinks is installed. As to why it's showing up now, I have no idea. 🤔 |
|
I guess I'll fess up to finding the "solution" on a very random blog |
|
Maybe use commit title: "github: add Also the workflow specifies |
|
I'm fine with that, but FWIW, the error message that prompted this specifies debian: |
Problem: recently, our github workflow for spelling and linkcheck got confused between pre-installed ubuntu packages and python's pip installed packages. Just ignore the ubuntu packages. Install our own. It doesn't seem to take much longer.
151b639 to
80a8adb
Compare
--ignore-installed to pip install of requirements
jameshcorbett
left a comment
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.
👍
grondo
left a comment
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!
Problem: recently, our github workflow for spelling and
linkcheck got confused between pre-installed debian packages
and python's pip installed packages.
Just ignore the debian packages. Install our own. It doesn't
seem to take much longer.
From: https://pip.pypa.io/en/stable/cli/pip_install/