abuild: give --silent the silent treatment

It's obsolete and without effect.

Change-Id: I17c1f8bffc2048a79de58cdc2a70651796db6ba6
Signed-off-by: Patrick Georgi <pgeorgi@chromium.org>
Reviewed-on: https://review.coreboot.org/20390
Tested-by: build bot (Jenkins) <no-reply@coreboot.org>
Reviewed-by: Nico Huber <nico.h@gmx.de>
This commit is contained in:
Patrick Georgi 2017-06-27 14:02:18 +02:00 committed by Nico Huber
parent 9413cb5f95
commit 55ea013842
1 changed files with 8 additions and 15 deletions

View File

@ -75,10 +75,6 @@ MAKE=$i
# this can be changed to junit by -J # this can be changed to junit by -J
mode=text mode=text
# silent mode.. no compiler calls, only warnings in the log files.
# this is disabled per default but can be enabled with -s
silent=
# quiet mode: only print pass, failure, and 'skipped' messages # quiet mode: only print pass, failure, and 'skipped' messages
quiet=false quiet=false
@ -274,14 +270,10 @@ function update_config
# shellcheck disable=SC2059 # shellcheck disable=SC2059
printf "$configoptions" >> "${config_file}" printf "$configoptions" >> "${config_file}"
# shellcheck disable=SC2086 yes "" 2>/dev/null | $MAKE oldconfig "$verboseopt" "DOTCONFIG=${config_file}" "obj=${build_dir}" "objutil=$TARGET/sharedutils" &> "${build_dir}/config.log" ; \
# Disabling SC2086 for "$silent" only
yes "" 2>/dev/null | $MAKE oldconfig $silent "DOTCONFIG=${config_file}" "obj=${build_dir}" "objutil=$TARGET/sharedutils" &> "${build_dir}/config.log" ; \
CONFIG_OK=$? CONFIG_OK=$?
if [ $CONFIG_OK -eq 0 ]; then if [ $CONFIG_OK -eq 0 ]; then
# shellcheck disable=SC2086 $MAKE savedefconfig "$verboseopt" DEFCONFIG="${defconfig_file}" DOTCONFIG="${config_file}" obj="${build_dir}" objutil="$TARGET/sharedutils" &>> "${build_dir}/config.log"
# Disabling SC2086 for "$silent" only
$MAKE savedefconfig $silent DEFCONFIG="${defconfig_file}" DOTCONFIG="${config_file}" obj="${build_dir}" objutil="$TARGET/sharedutils" &>> "${build_dir}/config.log"
return $? return $?
else else
return 1 return 1
@ -348,8 +340,7 @@ function compile_target
CURR=$( pwd ) CURR=$( pwd )
#stime=`perl -e 'print time();' 2>/dev/null || date +%s` #stime=`perl -e 'print time();' 2>/dev/null || date +%s`
# shellcheck disable=SC2086 eval $BUILDPREFIX $MAKE "$verboseopt" DOTCONFIG="${build_dir}/config.build" obj="${build_dir}" objutil="$TARGET/sharedutils" BUILD_TIMELESS=$TIMELESS \
eval $BUILDPREFIX $MAKE $silent DOTCONFIG="${build_dir}/config.build" obj="${build_dir}" objutil="$TARGET/sharedutils" BUILD_TIMELESS=$TIMELESS \
&> "${build_dir}/make.log" ; \ &> "${build_dir}/make.log" ; \
MAKE_FAILED=$? MAKE_FAILED=$?
cp .xcompile "${build_dir}/xcompile.build" cp .xcompile "${build_dir}/xcompile.build"
@ -551,7 +542,6 @@ Options:\n"
[-R|--root <path>] Absolute path to coreboot sources [-R|--root <path>] Absolute path to coreboot sources
(defaults to $ROOT) (defaults to $ROOT)
[--scan-build] Use clang's static analyzer [--scan-build] Use clang's static analyzer
[-s|--silent] Omit compiler calls in logs
[--timeless] Generate timeless builds [--timeless] Generate timeless builds
[-t|--target <vendor/board>] Attempt to build target vendor/board only [-t|--target <vendor/board>] Attempt to build target vendor/board only
[-T|--test] Submit image(s) to automated test system [-T|--test] Submit image(s) to automated test system
@ -567,6 +557,7 @@ Options:\n"
[-V|--version] Print version number and exit [-V|--version] Print version number and exit
[-h|--help] Print this help and exit [-h|--help] Print this help and exit
[-s|--silent] obsolete
__END_OF_HELP __END_OF_HELP
} }
@ -625,6 +616,7 @@ fi
chromeos=false chromeos=false
clean_work=false clean_work=false
verboseopt='V=0'
customizing="" customizing=""
configoptions="" configoptions=""
# testclass needs to be undefined if not used for variable expansion to work # testclass needs to be undefined if not used for variable expansion to work
@ -636,7 +628,7 @@ while true ; do
-a|--all) shift; buildall=true;; -a|--all) shift; buildall=true;;
-d|--dir) shift; configdir="$1"; shift;; -d|--dir) shift; configdir="$1"; shift;;
-r|--remove) shift; remove=true;; -r|--remove) shift; remove=true;;
-v|--verbose) shift; verbose=true; silent='V=1';; -v|--verbose) shift; verbose=true; verboseopt='V=1';;
-q|--quiet) shift; quiet=true;; -q|--quiet) shift; quiet=true;;
-V|--version) shift; myversion; exit 0;; -V|--version) shift; myversion; exit 0;;
-h|--help) shift; myversion; myhelp; exit 0;; -h|--help) shift; myversion; myhelp; exit 0;;
@ -649,7 +641,8 @@ while true ; do
test "$1" == "1" && cpuconfig=" on 1 cpu" test "$1" == "1" && cpuconfig=" on 1 cpu"
expr "$1" : '-\?[0-9]\+$' > /dev/null && test "0$1" -gt 1 && cpuconfig=" on $1 cpus in parallel" expr "$1" : '-\?[0-9]\+$' > /dev/null && test "0$1" -gt 1 && cpuconfig=" on $1 cpus in parallel"
shift;; shift;;
-s|--silent) shift; silent="-s";; # obsolete option
-s|--silent) shift;;
--scan-build) shift --scan-build) shift
scanbuild=true scanbuild=true
customizing="${customizing}, scan-build" customizing="${customizing}, scan-build"