-
Notifications
You must be signed in to change notification settings - Fork 424
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
Update bindgen rules to build clang from source. #1998
Conversation
dd3feae
to
6b7b8d7
Compare
Is there an option not to build Clang from source, for users who don't need that? |
You should still be able to register your own toolchains |
Is there an issue you’re running into building clang from source? |
@UebelAndre Cool change! I want to integrate bindgen-similar functionality between For reference, LLVM/bzlmod in rules_ll looks like this: https://github.com/eomii/rules_ll/blob/bdd243466d7c5e8be8a9d7f7fcb179b2a1ae084b/MODULE.bazel#L25-L50. The upstream patch is at https://reviews.llvm.org/D137008. I'll double-check that things are compatible with the usage in |
* Update bindgen rules to build clang from source. * Regenerate documentation
* Update bindgen rules to build clang from source. * Regenerate documentation
This change updates the default
rust_bindgen
rules to buildlibclang
andclang
from source using the Bazel rules sourced in the llvm-project repository for use inRustBindgen
actions. The version of Clang that Bindgen uses by default is updated14.0.6
which was what I was able to get building.Some breaking changes include:
llvm-project
defined in their WORKSPACE.rust_bindgen_repositories
is deleted as a means of forcing users to take a look at the updated dependencies.closes #919