Skip to content

Support binary tools/bazel wrappers#696

Merged
fweikert merged 1 commit intobazelbuild:masterfrom
Snowflake-Labs:binary-bazel
Nov 12, 2025
Merged

Support binary tools/bazel wrappers#696
fweikert merged 1 commit intobazelbuild:masterfrom
Snowflake-Labs:binary-bazel

Conversation

@jmmv
Copy link
Contributor

@jmmv jmmv commented May 27, 2025

To do this, recognize new tools/bazel.<OSNAME>-<ARCH> and tools/bazel.<ARCH> names for the wrapper, and favor those over tools/bazel if they are present.

@jmmv
Copy link
Contributor Author

jmmv commented Jun 19, 2025

I pushed a new version of the change to (try to) fix the problems with the Windows tests... but now one of the Windows jobs fails with:

external\gazelle++go_deps+org_golang_x_sys\unix\vgetrandom_linux.go:7:9: file requires newer Go version go1.24
compilepkg: error running subcommand external\rules_go++go_sdk+bazelisk__download_0\pkg\tool\windows_amd64\compile.exe: exit status 2

which seems unrelated to my change given that it's coming from a dependency. (I'm not even sure why the Windows build would look into a file with linux in its name, but, well, Go.)

To do this, recognize new tools/bazel.<OSNAME>-<ARCH> and tools/bazel.<ARCH>
names for the wrapper, and favor those over tools/bazel if they are present.
@fweikert fweikert merged commit 8d88666 into bazelbuild:master Nov 12, 2025
2 checks passed
@jmmv jmmv deleted the binary-bazel branch November 12, 2025 16:24
rdesgroppes added a commit to DataDog/datadog-agent-buildimages that referenced this pull request Jan 29, 2026
### What does this PR do?
Update bazelisk from v1.27.0 to v1.28.1 across all supported platform
configurations (Linux amd64/arm64 and Windows amd64).

### Motivation
Ability to use platform-specific `tools/bazel*` wrappers:
- bazelbuild/bazelisk#696 (v1.28.0)

Ability to switch our `tools/bazel.bat` to `tools/bazel.ps1` on Windows:
- bazelbuild/bazelisk#732 (v1.28.0)
- bazelbuild/bazelisk#762 (v1.28.1)
rdesgroppes added a commit to DataDog/datadog-agent-buildimages that referenced this pull request Jan 29, 2026
### What does this PR do?
Update bazelisk from v1.27.0 to v1.28.1 across all supported platform
configurations (Linux amd64/arm64 and Windows amd64).

### Motivation
Ability to use platform-specific `tools/bazel*` wrappers:
- bazelbuild/bazelisk#696 (v1.28.0)

Ability to switch our `tools/bazel.bat` to `tools/bazel.ps1` on Windows:
- bazelbuild/bazelisk#732 (v1.28.0)
- bazelbuild/bazelisk#762 (v1.28.1)
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants