Attention is currently required from: Paul Menzel, Leah Rowe. Arthur Heymans has posted comments on this change. ( https://review.coreboot.org/c/coreboot/+/51144 )
Change subject: util/chromeos: Use the same inventory each time and verify checksums ......................................................................
Patch Set 4:
(4 comments)
Patchset:
PS4: Try to do one change per commit.
File util/chromeos/crosfirmware.sh:
https://review.coreboot.org/c/coreboot/+/51144/comment/95fb197c_b377096d PS4, Line 34: https://web.archive.org/web/20210211071412/ I don't think this is a good idea. You generally want the latest version for your board. Also this would break any new board, not yet in that file.
https://review.coreboot.org/c/coreboot/+/51144/comment/1f241b58_6ae7b522 PS4, Line 45: use tabs for indentation.
https://review.coreboot.org/c/coreboot/+/51144/comment/381e9fd1_172512a0 PS4, Line 142: echo "acaf7450481136e72a1ee6c4fe3a2dea992d028f ${CONF}" > "${CONF_SHA1_FILE}" what is this?