diff options
| author | Leah Rowe <leah@libreboot.org> | 2023-05-14 14:33:21 +0100 | 
|---|---|---|
| committer | Leah Rowe <leah@libreboot.org> | 2023-05-14 14:34:54 +0100 | 
| commit | 7b36ffc1e586664f06637b1a1bd07233d18ddb9e (patch) | |
| tree | e9e2a0a32d273f1bfb80aa248e74da78ae57ae6a | |
| parent | 963b524722c016c05be5d68d570aa9e8623b55c5 (diff) | |
download/mrc: handle exit status within subshell
the previous code merely exited from the subshell,
but the intended behaviour is for the entire script
to halt execution, and exit with non-zero status.
this patch fixes that bug.
| -rwxr-xr-x | resources/scripts/download/mrc | 14 | 
1 files changed, 11 insertions, 3 deletions
| diff --git a/resources/scripts/download/mrc b/resources/scripts/download/mrc index 69c9eff2..63ec0100 100755 --- a/resources/scripts/download/mrc +++ b/resources/scripts/download/mrc @@ -43,8 +43,11 @@ _mrc_complete="mrc/haswell/mrc.bin"  cbdir="coreboot/default"  cbfstool="${cbdir}/util/cbfstool/cbfstool" +sname="" +  main()  { +	sname=${0}  	printf "Downloading Intel MRC blobs\n"  	check_existing && exit 0 @@ -90,9 +93,7 @@ fetch_mrc()  		download_image ${_url2} ${_file} ${_sha1sum}  	fi  	if [ ! -f $_file ]; then -		printf "%s not downloaded / verification failed. Exiting.\n" \ -				${_file} -		exit 1 +		fail "%{_file} not downloaded / verification failed."  	fi  	extract_partition ROOT-A ${_file} root-a.ext2 @@ -173,4 +174,11 @@ extract_coreboot()  	rm -r "${_unpacked}"  } +fail() +{ +	printf "%s: ERROR: %s\n" +			${sname} ${1} +	exit 1 +} +  main $@ | 
