gh-133951: Remove lib64->lib symlink in venv creation #137139
Conversation
|
All commit authors signed the Contributor License Agreement. |
|
Most changes to Python require a NEWS entry. Add one using the blurb_it web app or the blurb command-line tool. If this change has little impact on Python users, wait for a maintainer to apply the |
|
Most changes to Python require a NEWS entry. Add one using the blurb_it web app or the blurb command-line tool. If this change has little impact on Python users, wait for a maintainer to apply the |
|
More generally, could it cause issues with platforms that are configured with
Is there a possibility that this could somehow conflict with legacy venvs? or with some weird sysconfig? If this is causing issues on platforms without symlink support why not just checking if that platform supports symlinks and, if not, do nothing? |
The difference on these platforms is that platlib and purelib modules will be installed into separated directory (
I think this only affects venv creation. Legacy venvs should still be able to run.
In my understanding this symlink is not needed in anyway. Other paths created by venv are all looked up from venv scheme sysconfig except this symlink which is hard-coded and created even unnecessary. So I remove it for consistency. |
There was a problem hiding this comment.
Thanks for the PR!
There's a possibility it would break user code when users are hardcoding the path, or not using the venv scheme. Though, the code is already fundamentally broken in both those situations, so I agree with this change.
That said, if possible, I'd like @vsajip to also have a look before merging.
@Cycloctane, please ping me in 1~2 weeks if the PR hasn't moved.
@FFY00 :) |
|
@Cycloctane, sorry for the delay! |
…ython#137139)" This reverts commit a7a4855.
…37139) * Remove lib64->lib symlink in venv directory * fix test * remove unused import * add news
4 participants
venv creates a lib64->lib symlink on posix platforms (except osx) to make purelib and platlib always share the same directory in all dists (#65396). This can cause problems on filesystems without symlink support. It is also not necessary anymore because venv now creates lib paths in venv directory based on sysconfig. Others should also use sysconfig to get the correct lib paths.