Fix PBS encodings error by making pyvenv.cfg home absolute at runtime#1073
Draft
xangcastle wants to merge 1 commit into
Draft
Fix PBS encodings error by making pyvenv.cfg home absolute at runtime#1073xangcastle wants to merge 1 commit into
xangcastle wants to merge 1 commit into
Conversation
✨ Aspect Workflows Tasks📅 Tue Jun 2 13:31:56 UTC 2026 ❌ 3 failed tasks
✅ 4 successful tasks
🛠️ Fix❌ buildifier (buildifier)Install 🔁 Reproduce❌ buildifier (buildifier)❌ test (test-e2e-bazel-8 · test-e2e-bazel-9)Install ⏱ Last updated Tue Jun 2 13:34:31 UTC 2026 · 📊 GitHub API quota 9/15,000 (0% used, resets in 57m) |
py_binary startup benchmark
sys.path quality
|
dd75249 to
78777fc
Compare
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
Add this suggestion to a batch that can be applied as a single commit.This suggestion is invalid because no changes were made to the code.Suggestions cannot be applied while the pull request is closed.Suggestions cannot be applied while viewing a subset of changes.Only one suggestion per line can be applied in a batch.Add this suggestion to a batch that can be applied as a single commit.Applying suggestions on deleted lines is not supported.You must change the existing code in this line in order to create a valid suggestion.Outdated suggestions cannot be applied.This suggestion has been applied or marked resolved.Suggestions cannot be applied from pending reviews.Suggestions cannot be applied on multi-line comments.Suggestions cannot be applied while the pull request is queued to merge.Suggestion cannot be applied right now. Please check back later.
PBS encodings fix
Python 3.11/3.12 PBS reimplemented prefix discovery in getpath.py. Its resolvedpath() fails on multi-hop relative symlinks that traverse components across runfiles boundaries. When resolution fails Python falls back to the /install compile-time prefix.
The launcher workaround (exporting PYTHONHOME) doesn't help when nvoked through a venv symlink with -I, because getpath.py reads the venv's pyvenv.cfg first and ignores PYTHONHOME when the relative home = be resolved.
Fix: rewrite pyvenv.cfg at runtime with an absolute home= path so getpath.py discovers the stdlib without following symlinks.
Dep group fallback fix:
Packages with a single available dep_group or single cfg previously required an explicit dep_group attribute, even though there was no ambiguity. Add //conditions:default fallbacks in uv_hub/repository.bzl and uv_project/repository.bzl so these packages resolve automatically.
Changes are visible to end-users: yes/no
Test plan