Skip to content
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

[8.0.1] Forward @bazel_tools//tools/sh:toolchain_type to rules_shell #24793

Merged
merged 1 commit into from
Jan 2, 2025

Conversation

bazel-io
Copy link
Member

This is required for compatibility with modules that rely on the auto-registered sh_toolchain to obtain a shebang, e.g. rules_js.

Also remove implementations and toolchain registrations in favor of forwarders.

Closes #24748.

PiperOrigin-RevId: 709149200
Change-Id: I758cf403c24f6b5e6f00bbf5b4e0313a30773f78

Commit 720e15e

This is required for compatibility with modules that rely on the auto-registered `sh_toolchain` to obtain a shebang, e.g. `rules_js`.

Also remove implementations and toolchain registrations in favor of forwarders.

Closes bazelbuild#24748.

PiperOrigin-RevId: 709149200
Change-Id: I758cf403c24f6b5e6f00bbf5b4e0313a30773f78
@bazel-io bazel-io requested a review from a team as a code owner December 23, 2024 22:16
@bazel-io bazel-io added team-Configurability platforms, toolchains, cquery, select(), config transitions awaiting-review PR is awaiting review from an assigned reviewer labels Dec 23, 2024
@iancha1992 iancha1992 enabled auto-merge December 23, 2024 22:32
@iancha1992 iancha1992 added this pull request to the merge queue Jan 2, 2025
Merged via the queue into bazelbuild:release-8.0.1 with commit 039ff9b Jan 2, 2025
46 checks passed
@github-actions github-actions bot removed the awaiting-review PR is awaiting review from an assigned reviewer label Jan 2, 2025
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
team-Configurability platforms, toolchains, cquery, select(), config transitions
Projects
None yet
Development

Successfully merging this pull request may close these issues.

5 participants