[coreboot-gerrit] New patch to review for coreboot: buildgcc: Move a bunch of code into a function

[Zheng Bao (zheng.bao@amd.com)]

Zheng Bao (zheng.bao at amd.com) just uploaded a new patch set to gerrit, which you can find at http://review.coreboot.org/11318

-gerrit

commit 95eb7c66d96d3c46fb0fdf35d8998a8361f40462
Author: zbao <fishbaozi at gmail.com>
Date:   Mon Aug 24 22:08:36 2015 -0400

    buildgcc: Move a bunch of code into a function
    
    Change the code easier to understand.
    
    Change-Id: Ia815a27f7cc83c226a32e87485d712a5fbf4168e
    Signed-off-by: Zheng Bao <zheng.bao at amd.com>
    Signed-off-by: Zheng Bao <fishbaozi at gmail.com>
---
 util/crossgcc/buildgcc | 24 ++++++++++++++----------
 1 file changed, 14 insertions(+), 10 deletions(-)

diff --git a/util/crossgcc/buildgcc b/util/crossgcc/buildgcc
index d645a53..f39c4b6 100755
--- a/util/crossgcc/buildgcc
+++ b/util/crossgcc/buildgcc
@@ -174,6 +174,18 @@ searchtool()
 	false
 }
 
+check_sum() {
+	test -z "$CHECKSUM" || \
+	test "$(cat sum/$1.cksum 2>/dev/null | sed -e 's at .*\([0-9a-f]\{40,\}\).*@\1@')" = \
+	"$($CHECKSUM tarballs/$1 2>/dev/null | sed -e 's at .*\([0-9a-f]\{40,\}\).*@\1@')"
+}
+
+compute_sum() {
+	test ! -f sum/$1.cksum && test -f tarballs/$1 && \
+	(test -z "$CHECKSUM" || $CHECKSUM tarballs/$1 > sum/$1.cksum ) && \
+	printf "(checksum created. ${RED}Note. Please upload sum/$1.cksum if the corresponding archive is upgraded.)${NC}"
+}
+
 download() {
 	package=$1
 	archive="$(eval echo \$$package"_ARCHIVE")"
@@ -181,21 +193,13 @@ download() {
 	FILE=$(basename $archive)
 	printf " * $FILE "
 
-	test -f tarballs/$FILE && \
-		( test -z "$CHECKSUM" || \
-		  test "$(cat sum/$FILE.cksum 2>/dev/null | \
-				sed -e 's,.*\([0-9a-f]\{40\}\).*,\1,')" = \
-			"$($CHECKSUM tarballs/$FILE 2>/dev/null | \
-				sed -e 's,.*\([0-9a-f]\{40\}\).*,\1,')" ) && \
-			printf "(cached)" || (
+	test -f tarballs/$FILE && check_sum $FILE && printf "(cached)" || (
 		printf "(downloading from $archive)"
 		rm -f tarballs/$FILE
 		cd tarballs
 		wget --no-check-certificate -q $archive
 		cd ..
-		test ! -f sum/$FILE.cksum && test -f tarballs/$FILE && \
-			(test -z "$CHECKSUM" || $CHECKSUM tarballs/$FILE > sum/$FILE.cksum ) && \
-			printf "(checksum created. ${RED}Note. Please upload sum/$FILE.cksum if the corresponding archive is upgraded.${NC})"
+		compute_sum $FILE
 	)
 	if [ ! -f tarballs/$FILE ]; then
 		printf "\n${RED}Failed to download $FILE.${NC}\n"