Conversation
We've heared from several people that they tried passing the flag and it did not have any efect (as expected from a no-op), so let's undocument the flag so it's no longer discoverable. On a related note, we should document `--bes_upload_mode` which seems to be what people want to do.
|
cc @lfpino |
lfpino
left a comment
There was a problem hiding this comment.
We should just delete this flag at this point. It's been a no-op for years.
|
Removing sounds tricky to me, looks like lots of builds would breaks because people are setting it. If we can cherry-pick to 5.x, we should be able to remove in 6.0 |
|
@meisterT I can do that, but that graveyard seems Bazel specific. Can you verify that moving it there won't break Blaze (or take care of also adding it to the Blaze graveyard when merging)? |
|
Yep, that works for both Blaze and Bazel. |
|
Done, PTAL |
|
What's the status of this? I'd like to get this into 5.0 /cc @Wyverald |
|
@meisterT started the import process last year, but he's still on vacation until next week. In the meantime, @michaeledgar left the following comment:
|
|
Sorry for the delay, done. PTAL |
|
Any updates here? @meisterT @michaeledgar |
|
ping? I'd like to get this into 5.1 |
|
I'll check with @meisterT |
|
Sorry for the delay, giving it a kick internally. |
|
@Yannic will you create the cherrypick against the release branch? |
|
I can do it. |
We've heared from several people that they tried passing the flag and it did not have any efect (as expected from a no-op), so let's undocument the flag so it's no longer discoverable. On a related note, we should document `--bes_upload_mode` which seems to be what people want to do. Closes bazelbuild#14333. PiperOrigin-RevId: 435317406
We've heared from several people that they tried passing the flag and it did not have any efect (as expected from a no-op), so let's undocument the flag so it's no longer discoverable. On a related note, we should document `--bes_upload_mode` which seems to be what people want to do. Closes bazelbuild#14333. PiperOrigin-RevId: 435317406 Co-authored-by: Yannic Bonenberger <yannic@engflow.com>
We've heared from several people that they tried passing the flag and it did
not have any efect (as expected from a no-op), so let's undocument the flag
so it's no longer discoverable.
On a related note, we should document
--bes_upload_modewhich seems to bewhat people want to do.