dangernoodle-io / breadboard / 24751481154

100%

push

github

fix(cmake): generate embedded assets at configure time to sidestep PlatformIO path doubling (#55)

Under PlatformIO's scons on Linux CI, build-time add_custom_command with OUTPUT
under CMAKE_CURRENT_BINARY_DIR triggers a path-doubling bug: the generated file
lands at <build_dir>/x.c but scons then tries to compile <build_dir>/<build_dir>/x.c.o.

Switch bb_embed_assets() from build-time add_custom_command to configure-time
execute_process. Generated .c files now exist as real files before scons starts
its dependency scan, avoiding the doubled-path issue. Rebuild-on-input-change
is handled via set_property(DIRECTORY APPEND PROPERTY CMAKE_CONFIGURE_DEPENDS),
which forces CMake reconfigure if the input HTML changes, rerunning execute_process.

Fixes TaipanMiner PR #129 where bb_prov_default_form was not found on any board.
Also updates examples/elecrow-p4-hmi7 to include bb_prov_default_form in REQUIRES
and call bb_prov_default_form_get(), proving bb_embed_assets works end-to-end.

18 of 18 branches covered (100.0%)

Branch coverage included in aggregate %.

30 of 30 relevant lines covered (100.0%)

18.1 hits per line