-
Notifications
You must be signed in to change notification settings - Fork 4.1k
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
Bazel 8.0.0rc2: autoload does not work with WORKSPACE repos #24235
Comments
Moved here after initial clarification in #23899 |
@bazel-io fork 8.0.0 |
gentle ping @comius |
Yes, it’s supposedly fixed. Needs to be verified. |
How could I test it with e.g. bazelisk? |
Just use the latest commit from https://github.com/bazelbuild/bazel/commits/release-8.0.0/ |
I got a different error, probably related to a too low rules_java version with this. Checking further. |
I think this bug is fixed. |
Even though I'd prefer to check another RC version as I have a hard time to get to a working state with my repo. My latest issue is a skyframe RuntimeException (after I have fixed my rules_java version, and ensured that rules_jvm_external setup is done correctly). |
Thanks for the confirmation, we'll have rc3 hopefully today. |
A fix for this issue has been included in Bazel 8.0.0 RC3. Please test out the release candidate and report any issues as soon as possible. |
It's a bit hard to test this, as in my example repo I had new errors appearing for RC3.
What version of rules_java should be used with Bazel 8.0.0? Then I tried upgrading rules_java to 8.*. Only for rules_java 8.5.0+ it did not show me any more errors from rules_java itself, but then it showed me errors from rules_scala (that I guess should be adjusted to the Bazel 8.0.0 + rules_java changes). My understanding of which error comes "before" or "after" another is probably lacking, so you could conclude from this feedback whether the original bug is fixed or not. I guess it is fixed (that is my gut feeling, but I cannot be sure). |
I guess if I can see rules_java/rules_scala being used with a WORKSPACE repo means, that autoloading works. |
@gergelyfabian Can you please test rc4? |
Same thing as for RC3, I'm blocked by the incompatibilities of rules_scala vs. rules_java. I believe that is not a bazel bug, and this one should be fixed. |
@gergelyfabian I've just posted my experimental results to bazelbuild/rules_scala#1652. Most of the Bazel 8 and rules_java 8 incompatibilities can be easily addressed once Bzlmodification lands, and what I think is the only remaining one may be resolved by a The tradeoffs are that users will be forced to upgrade to Protobuf v28.3 under Bazel 6 and 7, and Protobuf v29 under Bazel 8, and the Update from bazelbuld/rules_scala#1652: I spoke too soon: I can get a So it seems we could bump |
Thank you! This kind of confirms this bug can be closed. |
Description of the bug:
Testing Bazel 8.0.0rc2 I got such an error:
This worked with Bazel 7.3.2.
Which category does this issue belong to?
No response
What's the simplest, easiest way to reproduce this bug? Please provide a minimal example if possible.
Reproduction: https://github.com/gergelyfabian/bazel-scala-example/tree/bazel-8.0.0rc1
Simply run
bazel build //...
in this repo at branchbazel-8.0.0rc1
Which operating system are you running Bazel on?
Ubuntu 24.04
What is the output of
bazel info release
?release 8.0.0rc2
If
bazel info release
returnsdevelopment version
or(@non-git)
, tell us how you built Bazel.No response
What's the output of
git remote get-url origin; git rev-parse HEAD
?If this is a regression, please try to identify the Bazel commit where the bug was introduced with bazelisk --bisect.
No response
Have you found anything relevant by searching the web?
No response
Any other information, logs, or outputs that you want to share?
Related: #23899
The text was updated successfully, but these errors were encountered: