[bazel] add more documentation around our Bazel build#32597
[bazel] add more documentation around our Bazel build#32597ParkMyCar merged 1 commit intoMaterializeInc:mainfrom
Conversation
|
@ptravers @DAlperin @SangJunBak I tagged all of y'all in this PR because I think you either have experience with Bazel or have expressed interest in it. Please let me know if there's anything here that doesn't make sense, or there might be parts I'm missing! |
| # Match the DWARF version used by Rust | ||
| # Match the DWARF version used by Rust. | ||
| # | ||
| # Note(parkmycar): This might not be necessary but seemed nice to do. |
There was a problem hiding this comment.
do we want to move to dwarf 5? it looks like it has native rust support and is the latest version supported in the clang frontend.
see: https://doc.rust-lang.org/stable/unstable-book/compiler-flags/dwarf-version.html
and: rust-lang/rust#136926
There was a problem hiding this comment.
Nice! I think when I looked at this in the past -Zdwarf-version was still unstable. We could probably bump macOS to dwarf 4 but we should validate that PolarSignals supports dwarf 5 before we go that far.
I'll make these changes in a separate PR if it's okay with you!
There was a problem hiding this comment.
I added a ticket. don't feel the need to do this before your departure. we can follow up. cheers!
https://github.com/MaterializeInc/database-issues/issues/9312
This PR adds more documentation for how Bazel is setup at Materialize. It's goal is to cover both building Rust crates with Bazel in general, and any parts of our setup specific to Materialize.
Rendered
Motivation
Knowledge sharing and redundancy
Checklist
$T ⇔ Proto$Tmapping (possibly in a backwards-incompatible way), then it is tagged with aT-protolabel.