perf: migrate bash launcher to hermetic-build#1045
Conversation
|
491171d to
dbccde5
Compare
| # Make wheel-declared console scripts reachable via `subprocess.run("name", ...)`. | ||
| _venv_bin = os.path.join(sys.prefix, "bin") | ||
| if _venv_bin not in os.environ.get("PATH", "").split(os.pathsep): | ||
| os.environ["PATH"] = _venv_bin + os.pathsep + os.environ.get("PATH", "") |
There was a problem hiding this comment.
Does anyone know of a better way of doing this? Should it be opt-in?
There was a problem hiding this comment.
💡 Codex Review
Here are some automated review suggestions for this pull request.
Reviewed commit: dbccde5e29
ℹ️ About Codex in GitHub
Your team has set up Codex to review pull requests in this repo. Reviews are triggered when you
- Open a pull request for review
- Mark a draft as ready
- Comment "@codex review".
If Codex has suggestions, it will comment; otherwise it will react with 👍.
Codex can also answer questions or update the PR. Try commenting "@codex address that feedback".
| @@ -1,5 +1,12 @@ | |||
| """quasi-public types.""" | |||
|
|
|||
| PyExecutableInfo = provider( | |||
There was a problem hiding this comment.
This replaces the __PEX_PY_BINARY_ENTRYPOINT__ we were inserting into the shell script.
I don't know the history behind py_pex_binary or if this is the ideal solution. @thesayyn?
There was a problem hiding this comment.
Because we can just use DefaultInfo.executable or something like that?
b5ecd19 to
fe6c33c
Compare
✨ Aspect Workflows Tasks📅 Fri May 29 21:53:27 UTC 2026 🔄 2 in progress tasks
❌ 1 failed task
✅ 4 successful tasks
🔁 Reproduce❌ test (test-e2e-bazel-8)Install ⏱ Last updated Fri May 29 21:56:39 UTC 2026 · 📊 GitHub API quota 263/15,000 (2% used, resets in 31m) |
py_binary startup benchmark
sys.path quality
|
…uncher native stub Co-Authored-By: Claude Sonnet 4.6 <noreply@anthropic.com>
e7aa847 to
f439e95
Compare


Changes are visible to end-users: yes
Use https://github.com/hermeticbuild/hermetic-launcher to launch the python toolchain instead of depending on a shell wrapper.
Test plan