From e47aaa8ff0f4ccd6fdb50055a50b4a9affd0a81e Mon Sep 17 00:00:00 2001
From: Leah Rowe <leah@libreboot.org>
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')

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