-
Notifications
You must be signed in to change notification settings - Fork 990
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
[question] Compatibility with bazel 7.0 #15363
Comments
Hi @marlamb Thanks for your question. |
Hi @marlamb Thanks for the question. I am so glad to see you involved in testing the BazelDeps generator 👏 I just tried the Bazel hello-world example (https://bazel.build/start/cpp), and it's not working for Bazel v7.0.0 yet. IMO, I think it's worth waiting for those Bazel examples to be updated to see the changes we should make in the Anyway, do you have any other Hello-World example working with Bazel 7.0.0? |
Unfortunately I don't have further examples. I am completely new to |
Same issue. Looking forward to seeing the Bazel 7 support. 😄 |
The lack of bazel 7 support has started to impact us, as we are now all-in on bazel 7 and are aiming for 100% module based builds. So I have fixed up the cpp-example to work with bazel 7 and conan in the hope that it will make it easier for you all to support bazel 7. Here it is: https://github.com/peakschris/bazel7-conan-module-example The readme explains what is going on. I've taken the same approach used by rules_dotnet, wrapping the repo file in a module extension. I'm not sure that this is the best approach, but it is one approach, and it works. My repo requires bazel 7.2.0rc2, but prior versions can be supported by un-including the conan.MODULE.bazel file and moving its contents into the main MODULE.bazel file. Using the include statement makes for cleaner MODULE files, but is a brand new feature. I will also start a conversation on the Bazel issues page to get some inputs into this structure and see if that community has any ideas. |
Hi all, This is the PR that you want to track and test if possible: #16196 It is ready to be merged for next Conan 2.4 release, to be released soon. Early feedback over that PR very welcome! |
@memsharded @franramirez688 and @Neeeflix thank you! Amazing to see that so much work has been done on this :-). Is there a way I can grab a windows executable containing the PR to test? |
You can run from source with (from https://github.com/conan-io/conan?tab=readme-ov-file#setup)
Then your |
Hi! I have a question regarding the compatibility of
conan
withbazel
. I tried to make my first steps following this guide, which works fine forbazel 6.2.0
, but fails for me usingbazel 7.0.0
at the build step with the following error message:Getting rid of the warning by passing
--noenable_bzlmod
does not solve the errors and I have now two questions:bzlmod
mode ofbazel
within the generated files? As it is the new default it looks worthwhile to me.bazel
(which I don't know, yet...)?For both it would be great, depending on your preferred solution, to update the linked guide above.
Thanks in advance!
Have you read the CONTRIBUTING guide?
The text was updated successfully, but these errors were encountered: