-
Notifications
You must be signed in to change notification settings - Fork 440
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
Revert "Pin to Bazel 7.3.2" #2998
Conversation
This reverts commit 53cb328.
Looks like this is failing due to missing tools on MacOS
|
Can you try to migrate to the Apple Silicon machines, basically replace |
Trying here: #3000 |
CI is green there! Just need an approval! |
After updating the macos workers the failure persists
|
Is it because |
bazelbuild/bazel@9e98051 is backported in 7.4.0 which uses One potential fix is to use |
cc toolchain config scripts have been moved to rules_cc: https://github.com/bazelbuild/rules_cc/blob/main/cc/private/toolchain/osx_cc_wrapper.sh.tpl It might be a good idea to fix it there. And with Bzlmod, if you add a bazel_dep on rules_cc, the toolchain from rules_cc will be picked up automatically. |
I tried to bump This also seems like a breaking change. I thought |
For MacOS this feels like an acceptable path forward. I'd rather not need to expose |
It's not yet fixed, as you can see in that file, I guess this change slipped through because it was not tested against rules_rust, which uses the tool in an action without |
I sent bazelbuild/rules_cc#275, but I also noticed your build is not yet fully on Bzlmod, so even if we have a new rules_cc, it's not easy to get the fix from it in WORKSPACE. So I guess the fast way to work around this issue is still to set |
BEGIN_PUBLIC Copybara import of the project: -- e485793 by Yun Peng <[email protected]>: Use full path for dirname and basename in osx_cc_wrapper.sh.tpl Context: bazelbuild/rules_rust#2998 END_PUBLIC COPYBARA_INTEGRATE_REVIEW=#275 from bazelbuild:meteorcloudy-patch-4 e485793 PiperOrigin-RevId: 697557587 Change-Id: I6ceb3d5f0fd747acb758926750fa48493edbdf3e
I was finally able to get
|
It's not updated in WORKSPACE? |
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.
Hurrah! 🎉
@@ -1 +1 @@ | |||
7.3.2 | |||
7.4.1 |
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.
Is there a reason we're keeping this file rather than deleting it?
Reverts #2957
Bazel 7.4.1 is now available https://github.com/bazelbuild/bazel/releases/tag/7.4.1