-
Notifications
You must be signed in to change notification settings - Fork 3.8k
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
Migrate Automatic Exec Groups by adding a toolchain parameter to the affected actions #10514
Conversation
I can see that this PR request fails with |
Yeah, we support the two most recent versions. So we'll drop Bazel 5.x in January. |
Okay, that means we should not add |
@ejona86 @sergiitk I'm interested in when this change will be inside a new release since it's needed for my further migration of Automatic Exec Groups. Do you maybe know when can this happen approximately? |
@kotlaja, v1.59.0 was scheduled for this past Tuesday. It was delayed because of some conversations with protobuf. It looks like it was released yesterday. Release notes and notification are still in the works, but the release is available. |
This is a step forward for the full migration of Automatic Exec Groups (AEGs). This change will be effective once AEGs are enabled.
Goal of this migration:
toolchain
attribute.I've added
None
since the executable (protoc
) seems not to be from a real toolchain. Moreover, the rule itself doesn't register any toolchains.Ref cl/559113276.