diff options
author | Leah Rowe <leah@libreboot.org> | 2023-05-15 00:42:36 +0100 |
---|---|---|
committer | Leah Rowe <leah@libreboot.org> | 2023-05-15 00:42:36 +0100 |
commit | e47aaa8ff0f4ccd6fdb50055a50b4a9affd0a81e (patch) | |
tree | e162db75ee1c0ee17b1142809ec6876bff96f570 /resources/scripts/download/coreboot | |
parent | 31d8fcd3fdc57af09072cf1fca71b7c7fc2ef49e (diff) |
download/coreboot: prune errant comments
Diffstat (limited to 'resources/scripts/download/coreboot')
-rwxr-xr-x | resources/scripts/download/coreboot | 15 |
1 files changed, 2 insertions, 13 deletions
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 |