From e47aaa8ff0f4ccd6fdb50055a50b4a9affd0a81e Mon Sep 17 00:00:00 2001 From: Leah Rowe Date: Mon, 15 May 2023 00:42:36 +0100 Subject: download/coreboot: prune errant comments --- resources/scripts/download/coreboot | 15 ++------------- 1 file changed, 2 insertions(+), 13 deletions(-) (limited to 'resources/scripts/download/coreboot') diff --git a/resources/scripts/download/coreboot b/resources/scripts/download/coreboot index d7f7a82a..5b2284d8 100755 --- a/resources/scripts/download/coreboot +++ b/resources/scripts/download/coreboot @@ -28,10 +28,6 @@ set -u -e main() { - # In this script, set -u is used to check for undefined variables, and - # the test command doesn't do any lazy evaluation, so we can't use - # a syntax like that: [ $# -eq 1 -a "$1" = "--help" ]. - if [ $# -eq 1 ] && [ "$1" = "--help" ] ; then usage exit 0 @@ -78,10 +74,6 @@ downloadfor() { cbtree="undefined" cbrevision="undefined" - # The loop will always exit, but this while loop is crafted - # such that a tree referencing a tree that references another - # tree is possible (and so on and so forth) - while true; do cbrevision="undefined" cbtree="undefined" @@ -226,11 +218,8 @@ downloadfor() { fi done - # extra.sh could be used to patch submodules, if you wanted to - # It's impossible to predict what submodules will be available, and - # it's rare that you'd want to patch them, so this is handled by - # extra.sh on a per-board basis - # In fact, extra.sh can be used for anything you want. + # extra.sh can be used for anything + # but should *only* be a last resort if [ -f "../../resources/coreboot/${_board}/extra.sh" ]; then "../../resources/coreboot/${_board}/extra.sh" \ || touch ../../build_error -- cgit v1.2.1