-
Notifications
You must be signed in to change notification settings - Fork 2k
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
Remove the recommendation about the bundled package #12318
Comments
I personally think we should recommend virtualenv + pip in the first place; at least, we won't have any surprises (or there should be none!) @n-peugnet do you follow me for that call? |
IMO virtualenv + pip is a safe and portable default that would work as expected everywhere. The virtualenv part should be mandatory in the install instructions instead of recommended afterwards currently. Another possibility would be to recommend pipx as suggested in the pip install warning. I'm not used to this tool but is seems like a fine way to install python CLI tools with dependency isolation. I don't think the recommendation to use system packages should be removed. For most of the people this is probably the easiest way to install Sphinx, but maybe a warning should be added that mixing user installed pip and system packages can cause issues? |
Is this sufficiently addressed through #12458? |
Let's say yes cheers, and can always re-open if necessary |
Originally posted by @picnixz in #12239 (comment)
The text was updated successfully, but these errors were encountered: