bazelbuild/rules_rust

Flag --incompatible_disable_starlark_host_transitions will break rules_rust in Bazel 7.0

keertk opened this issue · 5 comments

keertk commented

Incompatible flag --incompatible_disable_starlark_host_transitions will be enabled by default in the next major release (Bazel 7.0), thus breaking rules_rust. Please migrate to fix this and unblock the flip of this flag.

The flag is documented here: bazelbuild/bazel#17032.

Please check the following CI builds for build and test results:

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/continuous-integration.

illicitonion commented

Following the links to the CI builds, it looks like all the rules_rust ones are green. @keertk can you point at an example rules_rust failure with this flag?

meteorcloudy commented

@illicitonion Sorry the green is a bit confusing, it just means incompatible flag testing for this project finished correctly.

But the actual result is

Migration is needed for the following flags:
  --incompatible_disable_starlark_host_transitions

You'll have to click on the expandable log to see the actual failure:

image

jfirebaugh commented

Judging by the above logs, this reduces to protocolbuffers/protobuf#11333 and will be fixed when rules_rust can upgrade its com_google_protobuf dependency to a version that avoids the deprecated API.

  • 👍2
sgowroji commented

Hi @illicitonion , Can you share any significant update on this issue. Thanks!

illicitonion commented

This is blocked on protocolbuffers/protobuf#11333 - I don't believe rules_rust itself uses a host transition anywhere.

  • 👍1