[coreboot-gerrit] New patch to review for coreboot: buildgcc: Move a bunch of code into a function
Zheng Bao (zheng.bao@amd.com)
gerrit at coreboot.org
Tue Aug 25 04:30:26 CEST 2015
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"
More information about the coreboot-gerrit
mailing list