-
Notifications
You must be signed in to change notification settings - Fork 4k
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
Fix worker and multiplex workers for DexBuilder and Desugar actions #17351
Fix worker and multiplex workers for DexBuilder and Desugar actions #17351
Conversation
b0426f4
to
16379d7
Compare
16379d7
to
939a81d
Compare
ff9594e
to
0ef0089
Compare
0ef0089
to
03bfcbb
Compare
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Only one minor nit that we can fix internally: there's a comment alluding to the new-deleted flag use_workers_with_bexbuilder
at https://github.com/bazelbuild/bazel/blob/master/tools/android/BUILD.tools#L51 that we'll want to delete. Otherwise LGTM. Thanks, Ben!
Opened up a cherry pick here for when this lands in master #17965 |
src/main/java/com/google/devtools/build/lib/rules/android/AndroidConfiguration.java
Show resolved
Hide resolved
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Overall LGTM, there is just one more thing to fix (that we can handle internally) - we'll have to move the deprecated @Option
stanza from BazelBuildGraveyardOptions class to the BuildGraveyardOptions class.
@bazel-io fork 6.2.0 |
Fixing up the DexBuilder and Desugar actions so that they correctly spawn worker or multiplexed worker actions. `--modify_execution_info` doesn't work as expected and is not additive, which results in the previous execution infos being removed. Closes bazelbuild#17351. PiperOrigin-RevId: 523696356 Change-Id: Iada7fb75df5b4d2e3ba1308110977899567f2bc2
Fixing up the DexBuilder and Desugar actions so that they correctly spawn worker or multiplexed worker actions.
--modify_execution_info
doesn't work as expected and is not additive, which results in the previous execution infos being removed.