-
-
Notifications
You must be signed in to change notification settings - Fork 139
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
Virtualenvs created from links are broken #380
Comments
Do you think this is a case of astral-sh/uv#8429 or are you suggesting we can fix this at build time? |
I expect a sys variable is resolved incorrectly during interpreter startup, causing the wrong home in
|
I need a little more exposition alongside these examples, I don't follow what you're demonstrating there. I would be surprised if this wasn't a sysconfig issue, but of course it'd be great to fix here if we can. |
As shown, pbs creates broken venv when the base interpreter is a symlink. This is a bug, since it should be able to do this, as system interpreter and pyenv succeed at this. The crash happens because This bug is the most likely candidate to crash the test suite in astral-sh/uv#8481 (comment). We need to figure out how |
Interestingly, after symlinking, some of the
|
That would definitely affect compilations of extension modules. |
Ohh interesting, it's because Python does:
|
I'm pretty sure I had to do something in PyOxidizer to work around this. Essentially mucking with the Python interpreter config to get it to resolve paths correctly because the default logic was insufficient. We should consider involving a CPython maintainer on this one as the behavior is a smell. ISTR Windows having special logic for path resolution because portable installs are a thing on Windows. But since most UNIX-like installs are to a fixed path, CPython doesn't support dynamic paths as well as we'd like. If CPython doesn't recognize the issue, we may have to patch our CPython build to handle dynamic install paths/symlibks correctly. This could be done with a custom site customize file. But since this included file can be disabled, we may need to patch C code. |
I haven't looked closely at this specific case, but we've definitely removed (Tangentially related, python/cpython#124825 is a proposal I filed to revisit adding lexical relative path resolution to |
I'm looking into this a bit. |
Ok, so for reference, the way prefix gets set in these builds is here. If the executable is at |
Ok... If you symlink an interpreter (but don't create a virtualenv), then So, above, it would still be Specifically, when you're not in a virtualenv, the However... if you create a virtualenv from a symlinked interpreter, then So we'd then fail to find |
I don't know if we should be setting |
One thing to note in the "symlink to Sorry, my own scratch notes... When I symlink
When I symlink
Ahhh, ok. The reason for this, I think, is that the When we launch from a virtualenv, we take whatever is in
The combination of these behaviors means that whatever we put in We can solve this in two ways:
(1) seems easier to me, though (2) might be the "better" fix? I'm just not confident on how we would implement it or how it would affect other, non-PBS Pythons. |
## Summary This PR improves our "don't fully resolve symlinks" behavior for `python-build-standalone` builds based on learnings from astral-sh/python-build-standalone#380 (comment). Specifically, we can now robustly detect whether a target executable will lead to a valid `prefix` or not, and iteratively resolve symlinks until we find a valid target executable. ## Test Plan ### Direct symlink to `python` Correctly resolves to the symlink target, rather than the symlink itself. ``` ❯ ln -s /Users/crmarsh/.local/share/uv/python/cpython-3.12.6-macos-aarch64-none/bin/python foo ❯ cargo run venv --python ./foo ❯ cat .venv/pyvenv.cfg home = /Users/crmarsh/.local/share/uv/python/cpython-3.12.6-macos-aarch64-none/bin implementation = CPython uv = 0.5.7 version_info = 3.12.6 include-system-site-packages = false prompt = uv ❯ .venv/bin/python -c "import sys" ``` ### Symlink to the Python installation Correctly does _not_ resolve the symlink. ``` ❯ ln -s /Users/crmarsh/.local/share/uv/python/cpython-3.12.6-macos-aarch64-none bar ❯ cargo run venv --python ./bar ❯ cat .venv/pyvenv.cfg home = /Users/crmarsh/workspace/uv/bar/bin implementation = CPython uv = 0.5.7 version_info = 3.12.6 include-system-site-packages = false prompt = uv ❯ .venv/bin/python -c "import sys" ``` ### Direct symlink to `python` in a symlinked Python installation Correctly resolves the direct symlink, but not the symlink of the Python installation. ``` ❯ ln -s bar/bin/python baz ❯ cargo run venv --python ./baz ❯ cat .venv/pyvenv.cfg home = /Users/crmarsh/workspace/uv/bar/bin implementation = CPython uv = 0.5.7 version_info = 3.12.6 include-system-site-packages = false prompt = uv ❯ .venv/bin/python -c "import sys" ```
Unlike a system interpreter or a pyenv installation, pbs can't create venv from a symlink to the pbs installation location. PYTHONHOME is in the wrong location (that of the link, i think), which causes the encoding error.
I've only tested linux, i expect windows isn't affected (different venv mechanism, no symlinks by default), but i expect macos is affected, too.
The text was updated successfully, but these errors were encountered: