[coreboot-gerrit] Change in coreboot[master]: tint: introduce the new tint build system with checksum verification

[build bot (Jenkins) (Code Review)]

build bot (Jenkins) has posted comments on this change. ( https://review.coreboot.org/23856 )

Change subject: tint: introduce the new tint build system with checksum verification
......................................................................


Patch Set 5:

(12 comments)

https://review.coreboot.org/#/c/23856/5/payloads/external/tint/generate_core.sh
File payloads/external/tint/generate_core.sh:

https://review.coreboot.org/#/c/23856/5/payloads/external/tint/generate_core.sh@38
PS5, Line 38: 		sed -n "/^${start}/,/^${end}/{/^${start}/{p;n};{p}}" $source >> $destination
line over 80 characters


https://review.coreboot.org/#/c/23856/5/payloads/external/tint/generate_core.sh@40
PS5, Line 40: 		sed -n "/^${start}/,/^${end}/{/^${start}/{p;n};/^${end}/{q};{p}}" $source >> $destination
line over 80 characters


https://review.coreboot.org/#/c/23856/5/payloads/external/tint/generate_core.sh@54
PS5, Line 54: FUNCTIONS="please_install searchtool download compute_hash error_hash_mismatch verify_hash unpack_and_patch"
line over 80 characters


https://review.coreboot.org/#/c/23856/5/payloads/external/tint/generate_core.sh@64
PS5, Line 64: import_from_file $buildgcc $corescript "# Find all the required tools" "# Allow" last_line_disabled || exit "$?"
line over 80 characters


https://review.coreboot.org/#/c/23856/5/payloads/external/tint/generate_core.sh@70
PS5, Line 70: import_from_file $buildgcc $corescript "if \[ \"\$HALT_FOR_TOOLS" "fi" || exit "$?"
line over 80 characters


https://review.coreboot.org/#/c/23856/5/payloads/external/tint/generate_core.sh@86
PS5, Line 86: sed -i -e "s/\tknown_hash=\"\$(get_known_hash.*/\tknown_hash=\"\$2\"/g" $corescript
line over 80 characters


https://review.coreboot.org/#/c/23856/5/payloads/external/tint/generate_core.sh@95
PS5, Line 95: # Insert prepare_${package} function call between the unpack and patch operations
line over 80 characters


https://review.coreboot.org/#/c/23856/5/payloads/external/tint/generate_core.sh@99
PS5, Line 99: 	sed -i -e "/\$TAR \$FLAGS \$(basename \$archive)/a prepare_\${package} || exit \"\$?\"" $corescript
line over 80 characters


https://review.coreboot.org/#/c/23856/5/payloads/external/tint/tintify_core.sh
File payloads/external/tint/tintify_core.sh:

https://review.coreboot.org/#/c/23856/5/payloads/external/tint/tintify_core.sh@42
PS5, Line 42: echo "if [ ! -z ./\${TINT_DIR} ] && [ -e ./\${TINT_DIR}/debian ]; then" >> $tintified
line over 80 characters


https://review.coreboot.org/#/c/23856/5/payloads/external/tint/tintify_core.sh@43
PS5, Line 43: echo "rm -rf ./\${TINT_DIR}/debian ./\${TINT_DIR}/typedefs.h ./\${TINT_DIR}/Makefile;" >> $tintified
line over 80 characters


https://review.coreboot.org/#/c/23856/5/payloads/external/tint/tintify_core.sh@61
PS5, Line 61: echo "printf \"Downloaded TINT tarball ... \${green}ok\${NC}\\n\"" >> ${tintified}
line over 80 characters


https://review.coreboot.org/#/c/23856/5/payloads/external/tint/tintify_core.sh@65
PS5, Line 65: echo "printf \"Unpacked and patched TINT... \${green}ok\${NC}\\n\"" >> ${tintified}
line over 80 characters



-- 
To view, visit https://review.coreboot.org/23856
To unsubscribe, or for help writing mail filters, visit https://review.coreboot.org/settings

Gerrit-Project: coreboot
Gerrit-Branch: master
Gerrit-MessageType: comment
Gerrit-Change-Id: I1d24f222d1b92030b81bba3951e243a2a9f37290
Gerrit-Change-Number: 23856
Gerrit-PatchSet: 5
Gerrit-Owner: Mike Banon <mikebdp2 at gmail.com>
Gerrit-Reviewer: Jonathan Neuschäfer <j.neuschaefer at gmx.net>
Gerrit-Reviewer: Martin Roth <martinroth at google.com>
Gerrit-Reviewer: Mike Banon <mikebdp2 at gmail.com>
Gerrit-Reviewer: Paul Menzel <paulepanter at users.sourceforge.net>
Gerrit-Reviewer: Stefan Reinauer <stefan.reinauer at coreboot.org>
Gerrit-Reviewer: build bot (Jenkins) <no-reply at coreboot.org>
Gerrit-Comment-Date: Wed, 15 Aug 2018 22:21:29 +0000
Gerrit-HasComments: Yes
Gerrit-HasLabels: No
-------------- next part --------------
An HTML attachment was scrubbed...
URL: <http://mail.coreboot.org/pipermail/coreboot-gerrit/attachments/20180815/5af3b72b/attachment.html>