Attention is currently required from: Martin L Roth.
Kyösti Mälkki has posted comments on this change. ( https://review.coreboot.org/c/coreboot/+/75803?usp=email )
Change subject: util/abuild: Add per-build statistics tarfile ......................................................................
Patch Set 2:
(3 comments)
File util/abuild/abuild:
https://review.coreboot.org/c/coreboot/+/75803/comment/4270129d_5fff3f0e : PS2, Line 440: echo "$BUILD_NAME" >> "$PASSED_BOARDS" Not in the scope, but compile_target runs concurrently with upto 32 concurrent cpus. Is there some implicit locking happening, or is it possible that some lines are lost here?
https://review.coreboot.org/c/coreboot/+/75803/comment/4798c054_98e08091 : PS2, Line 449: echo "$BUILD_NAME" >> "$FAILED_BOARDS" Same here.
https://review.coreboot.org/c/coreboot/+/75803/comment/f0fbe87e_9f566842 : PS2, Line 457: sha256sum "${build_dir}/coreboot.rom" >> "${checksum_file}_platform" Same here.