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

Migration needed for Automatic Exec Groups #2282

Open
sgowroji opened this issue Oct 9, 2024 · 1 comment · May be fixed by #2306
Open

Migration needed for Automatic Exec Groups #2282

sgowroji opened this issue Oct 9, 2024 · 1 comment · May be fixed by #2306
Labels
need: upstream support An issue that needs changes in upstream code

Comments

@sgowroji
Copy link
Member

sgowroji commented Oct 9, 2024

Migration to Automatic Exec Groups (AEGs) is needed since its incompatible flag (--incompatible_auto_exec_groups) will be enabled for Bazel@Head by default next week (week of 14th of October), thus it will be breaking rules_python. Please migrate and fix potential errors, in order to unblock your project and users who depend on your project. Potential errors can be triggered by building your project with this incompatible flag. Note that AEGs will not be flipped by default for Bazel 8, but for Bazel at head after Bazel 8 cut.

For more details, please check:

Never heard of incompatible flags before? We have documentation that explains everything.

If you have any questions, please file an issue in https://github.com/bazelbuild/bazel/issues.

@aignas aignas added this to the v1.0.0 milestone Oct 15, 2024
@aignas aignas linked a pull request Oct 15, 2024 that will close this issue
@aignas aignas removed this from the v1.0.0 milestone Oct 15, 2024
@rickeylev
Copy link
Contributor

FYI: #2306 sets the flag in our project, but it looks like protobuf isn't ready yet.

@aignas aignas added the need: upstream support An issue that needs changes in upstream code label Oct 24, 2024
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
need: upstream support An issue that needs changes in upstream code
Projects
None yet
Development

Successfully merging a pull request may close this issue.

3 participants