The files are auto-generated by (effectively, if not in reality) running bazel on the remote machine and then copying the generated files into the repo. It's not a great idea to modify them.

We don't have a hermetic C++ toolchain, so we still need to manage that ourselves. Our remote build images have a C++ toolchain installed, so we need to tell Bazel about it. Because the build is remote, we can't figure this stuff out at run time (since we need to know everything before sending the actions to the remote machine), and this is the way to configure it.

The files are auto-generated by (effectively, if not in reality) running bazel on the remote machine and then copying the generated files into the repo. It's not a great idea to modify them.

Got it. I think as long as it's documented how to re-generate these files should we need - we are fine.

We don't have a hermetic C++ toolchain, so we still need to manage that ourselves. Our remote build images have a C++ toolchain installed, so we need to tell Bazel about it. Because the build is remote, we can't figure this stuff out at run time (since we need to know everything before sending the actions to the remote machine), and this is the way to configure it.

Ok, so Bazel doesn't ship with a hermetic C++ toolchain. Would using something like https://github.com/uber/hermetic_cc_toolchain help us? Probably not now, but eventually.

Got it. I think as long as it's documented how to re-generate these files should we need - we are fine.

We just ran https://github.com/bazelbuild/bazel-toolchains

Ok, so Bazel doesn't ship with a hermetic C++ toolchain. Would using something like https://github.com/uber/hermetic_cc_toolchain help us? Probably not now, but eventually.

I spent an ungodly amount of time attempting to get the zig-cc toolchain working and gave up. If all we wanted to do was support Linux, it'd be easier, but we want to support the matrix of compiling for and on Windows, Mac OS, and Linux. For that we need various sysroots, and the legality of distributing these is unknown. That makes it a really difficult problem to solve. It's also not something that we can solve in this PR :)

Codecov Report

Patch coverage: 75.34% and project coverage change: +0.01 🎉

Comparison is base (e79b2f2) 54.87% compared to head (a5c0cad) 54.89%.

❗ Current head a5c0cad differs from pull request most recent head c594b77. Consider uploading reports for the commit c594b77 to get more accurate results

📣 This organization is not using Codecov’s GitHub App Integration. We recommend you install it so Codecov can continue to function properly for your repositories. Learn more

@@            Coverage Diff             @@
##            trunk   #11887      +/-   ##
==========================================
+ Coverage   54.87%   54.89%   +0.01%     
==========================================
  Files          85       86       +1     
  Lines        5698     5733      +35     
  Branches      230      233       +3     
==========================================
+ Hits         3127     3147      +20     
- Misses       2341     2353      +12     
- Partials      230      233       +3     

☔ View full report in Codecov by Sentry.
📢 Do you have feedback about the report comment? Let us know in this issue.