Martin L Roth has uploaded this change for review. ( https://review.coreboot.org/c/coreboot/+/76936?usp=email )
Change subject: util/abuild: Better identify config string in log ......................................................................
util/abuild: Better identify config string in log
When using the --skip_set and --skip_unset arguments, the config line looked like a statement that the build was being skipped instead of abuild just printing the configuration.
This updates those config statements to better show that it's the config and not stating that this particular build is being skipped.
Signed-off-by: Martin Roth gaumless@gmail.com Change-Id: I6cc59f9b33dcda51aeb3640d449037a0aa054e36 --- M util/abuild/abuild 1 file changed, 7 insertions(+), 7 deletions(-)
git pull ssh://review.coreboot.org:29418/coreboot refs/changes/36/76936/1
diff --git a/util/abuild/abuild b/util/abuild/abuild index 233a661..1f64413 100755 --- a/util/abuild/abuild +++ b/util/abuild/abuild @@ -11,8 +11,8 @@
#set -x # Turn echo on....
-ABUILD_DATE="Feb 3, 2023" -ABUILD_VERSION="0.11.01" +ABUILD_DATE="Aug 3, 2023" +ABUILD_VERSION="0.11.02"
TOP=$PWD
@@ -809,12 +809,12 @@ ;; --skip_set) shift skipconfig_set="$1" - customizing="${customizing}, Skipping CONFIG_${skipconfig_set}=Y" + customizing="${customizing}, Skipping builds with CONFIG_${skipconfig_set}=Y" shift ;; --skip_unset) shift skipconfig_unset="$1" - customizing="${customizing}, Skipping CONFIG_${skipconfig_unset} not set" + customizing="${customizing}, Skipping builds with CONFIG_${skipconfig_unset} not set" shift ;; --asserts) shift @@ -920,10 +920,10 @@ fi
customizing=$(echo "$customizing" | cut -c3-) -if [ "$customizing" = "" ]; then - customizing="default configuration" +if [ -z "$customizing" ]; then + customizing="Default configuration" fi - +customizing="Config: ${customizing}" FAILED_BOARDS="$(realpath ${TARGET}/failed_boards)" PASSED_BOARDS="$(realpath ${TARGET}/passing_boards)"