dangernoodle-io / breadboard / 30780431660

97%

push

github

fix: make generated bb_app_init.c a real build dependency (#1214)

cmake/bb_generated.cmake gains bb_regenerate_wire_or_fail(), which runs bbtool codegen unconditionally at every real-pass CMake configure and rewrites examples/smoke and examples/floor's generated/bb_app_init.c from the current // bbtool:init marker set, instead of only checking the existing file for staleness. Both example main/CMakeLists.txt call it before the existing bb_verify_generated_or_fail symbol-existence check (kept as cheap belt-and-braces defense-in-depth against a codegen bug or an out-of-band write) and register component/platform headers AND component/platform CMakeLists.txt files (boards.derive_component reads each composed component's own CMakeLists.txt for REQUIRES/PRIV_REQUIRES independently of its headers -- an edit there with no header touched previously reconfigured nothing), the consumer manifest, the bbtool sources, and bbtool.toml via CMAKE_CONFIGURE_DEPENDS so a changed input reliably triggers a reconfigure (and therefore regeneration).

bbtool codegen's writer now shares version.py's write-if-changed helper (extracted to core.write_if_changed, a second hand-rolled instance triggering the Consolidation convention) so a content-stable regeneration never bumps the generated file's mtime and never forces a needless downstream recompile. The shared helper writes atomically via a per-call-unique tmp file (tempfile.mkstemp, never a fixed .tmp name, so two racing writers to the same generated file can't tear each other's output) and removes that tmp file on any write failure instead of leaving it behind. bb_regenerate_wire_or_fail's FATAL_ERROR also separates codegen's stdout/stderr with a newline instead of concatenating them unseparated.

make floor gains a floor-gen prerequisite, matching every smoke-<board> target's existing smoke-gen-<board> prerequisite.

Closes the gap #1208 (B1-1371's verify-generated) deliberately left open: a stale bb_app_init.c whose call... (continued)

8294 of 8858 branches covered (93.63%)

Branch coverage included in aggregate %.

14024 of 14179 relevant lines covered (98.91%)

2609.35 hits per line