Skip to content

[pull] main from bazel-contrib:main#516

Merged
pull[bot] merged 2 commits intogarymm:mainfrom
bazel-contrib:main
Feb 8, 2026
Merged

[pull] main from bazel-contrib:main#516
pull[bot] merged 2 commits intogarymm:mainfrom
bazel-contrib:main

Conversation

@pull
Copy link
Copy Markdown

@pull pull Bot commented Feb 8, 2026

See Commits and Changes for more details.


Created by pull[bot] (v2.0.0-alpha.4)

Can you help keep this open source service alive? 💖 Please sponsor : )

This fixes an issue where the bootstrap script would fail if env was not
in the PATH, which is the case on NixOS, or when Bazel's strict action
env is enabled.

To fix, the bootstrap checks if env exists, and if not, falls back to
/usr/bin/env.

Fixes #3575
The `imports` attribute is treated as target-relative in non-venv mode,
but venv-mode
was passing the value through without the target-relative processing.
This went unnoticed
because the main use case, pypi packages, are a the repo root, so target
and repo
relative paths are the same.

To fix, use join the input value to the target's path, as done in the
non-venv
`get_imports()` function.

Fixes #3565
@pull pull Bot locked and limited conversation to collaborators Feb 8, 2026
@pull pull Bot added the ⤵️ pull label Feb 8, 2026
@pull pull Bot merged commit 708b07b into garymm:main Feb 8, 2026
2 checks passed
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant