[coreboot-gerrit] Patch set updated for coreboot: abuild: Use 20 lines of context for Kconfig errors

Stefan Reinauer (stefan.reinauer@coreboot.org) gerrit at coreboot.org
Tue Dec 15 02:34:13 CET 2015


Stefan Reinauer (stefan.reinauer at coreboot.org) just uploaded a new patch set to gerrit, which you can find at https://review.coreboot.org/12720

-gerrit

commit 3c82e26e9dd984e3ee4677a4acd2854245d8d055
Author: Stefan Reinauer <stefan.reinauer at coreboot.org>
Date:   Mon Dec 14 16:36:45 2015 -0800

    abuild: Use 20 lines of context for Kconfig errors
    
    The current default of 6 lines leaves us with no context
    about the actual error:
    
    *** ERROR: 3 warnings encountered, and warnings are errors.
    
    coreboot-gerrit/util/kconfig/Makefile:38: recipe for target 'oldconfig' failed
    make[1]: *** [oldconfig] Error 1
    make[1]: Leaving directory 'coreboot-gerrit'
    
    Change-Id: I67e7d740e7b3b1c66005dc1bf50557a20bc15428
    Signed-off-by: Stefan Reinauer <stefan.reinauer at coreboot.org>
---
 util/abuild/abuild | 2 +-
 1 file changed, 1 insertion(+), 1 deletion(-)

diff --git a/util/abuild/abuild b/util/abuild/abuild
index cddcb57..777e176 100755
--- a/util/abuild/abuild
+++ b/util/abuild/abuild
@@ -223,7 +223,7 @@ function create_config
 		if [ "$quiet" == "false" ]; then printf "  $MAINBOARD config created.\n"; fi
 		return 0
 	else
-		# Does this ever happen?
+		CONTEXT=20
 		if [ "$quiet" == "false" ]; then printf "$MAINBOARD config creation FAILED!\nLog excerpt:\n"; fi
 		tail -n $CONTEXT $build_dir/config.log 2> /dev/null || tail -$CONTEXT $build_dir/config.log
 		return 1



More information about the coreboot-gerrit mailing list