-
Notifications
You must be signed in to change notification settings - Fork 1.3k
Ensure correct site packages are visible to rustpython #4924
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
|
Issue probably lies here:
Notice purelib points to This function eventually gets called in |
I think I found the issue:
when create venv from rustpython, it already have correct _INSTALL_SCHEMES (with rustpython in it), |
Thank you! Let me try some tricks to minimize |
Do you know how other distributions handle it? It seems we'd better just read |
Co-Authored-By: Ankit Kumar Pandey <itsankitkp@gmail.com>
4dd536c
to
aad1d84
Compare
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.
Thank you so much! Let's fix it first and then think about the better or the correct way.
Makes sense, thanks |
This work should resolves #4911
Aim
Note: Instead of having both python and rustpython, I am including only one as venv/lib/rustpython3.11/site-packages, as venv/lib/python3.11/ severs no purpose and create confusion.