Skip to content

Revert "Remove uses of incompatible_use_toolchain_transition now th…#454

Merged
nacl merged 1 commit intobazelbuild:mainfrom
katre:restore-toolchain-transition
Nov 5, 2021
Merged

Revert "Remove uses of incompatible_use_toolchain_transition now th…#454
nacl merged 1 commit intobazelbuild:mainfrom
katre:restore-toolchain-transition

Conversation

@katre
Copy link
Copy Markdown
Contributor

@katre katre commented Oct 28, 2021

…at it is enabled by (#452)"

This is still needed for bazel 4.x compatibility, and it a no-op in
Bazel 5.x, so it should be kept.

The API will be removed in Bazel 6.0.

Part of bazelbuild/bazel#14127.

This reverts commit e203d5b.

…at it is enabled by (#452)"

This is still needed for bazel 4.x compatibility, and it a no-op in
Bazel 5.x, so it should be kept.

The API will be removed in Bazel 6.0.

This reverts commit e203d5b.
@katre katre requested review from aiuto and nacl as code owners October 28, 2021 16:15
@aiuto
Copy link
Copy Markdown
Collaborator

aiuto commented Oct 28, 2021

Does the option actually help with 4.x? I am using
load("@rules_cc//cc:find_cc_toolchain.bzl", "find_cc_toolchain")
to pick up the toolchain. The migration logic seems to be there.

@katre
Copy link
Copy Markdown
Contributor Author

katre commented Oct 28, 2021

4.x will still work without this, but the tools may end up in the wrong configuration (especially with cross-compilation or remote executors).

Copy link
Copy Markdown
Collaborator

@nacl nacl left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

LGTM; thanks!

@nacl nacl merged commit f108859 into bazelbuild:main Nov 5, 2021
mostynb added a commit to mostynb/rules_pkg that referenced this pull request Jun 23, 2025
…een removed from bazel

This breaks compatibility with bazel 4.x, in order to fix compatibility with
the tip of bazel's master branch.

Resolves bazelbuild#962
See also: bazelbuild#452, bazelbuild#454
tonyaiuto added a commit that referenced this pull request Jun 25, 2025
…een removed from bazel (#963)

I think this might break compatibility with bazel 4.x, in order to fix
compatibility with the tip of bazel's master branch.

Resolves #962
See also: #452, #454

cc @katre
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.

3 participants