This change adds some basic summary information to the MAKEALL script. The summary information includes how many boards were compiled, how many boards had compile warnings or errors, and which specific boards had compile warnings or errors.

This information is useful when doing compile testing to quickly determine which boards are broken.

As a side benefit, no empty $BOARD.ERR files are generated by MAKEALL. Previously, each board had a corresponding $BOARD.ERR file, even if the board compiled cleanly.

Signed-off-by: Peter Tyser ptyser@xes-inc.com --- Changes since v1: - Fix issue where summary was printed multiple times when a list was composed of sublists

Changes since v2: - Update script to only use POSIX arithmetic

Changes since v3: - Remove unnecessary expansion of variables inside of $(( ... ))

Changes since v4: - Catch additional termination signals - Use signal 0 to trigger printing of stats - Break POSIX math change into a separate patch

MAKEALL | 27 +++++++++++++++++++++++++++ 1 files changed, 27 insertions(+), 0 deletions(-)

diff --git a/MAKEALL b/MAKEALL index 1d50c34..fd06d8d 100755 --- a/MAKEALL +++ b/MAKEALL @@ -1,5 +1,9 @@ #!/bin/sh

+# Print statistics when we exit +trap exit 1 2 3 15 +trap print_stats 0 + # Determine number of CPU cores if no default was set : ${BUILD_NCPUS:="`getconf _NPROCESSORS_ONLN`"}

@@ -31,6 +35,11 @@ fi

LIST=""

+# Keep track of the number of builds and errors +ERR_CNT=0 +ERR_LIST="" +TOTAL_CNT=0 + ######################################################################### ## MPC5xx Systems ######################################################################### @@ -900,6 +909,14 @@ build_target() {

${MAKE} ${JOBS} all 2>&1 >${LOG_DIR}/$target.MAKELOG \ | tee ${LOG_DIR}/$target.ERR + if [ -s ${LOG_DIR}/$target.ERR ] ; then + ERR_CNT=$((ERR_CNT + 1)) + ERR_LIST="${ERR_LIST} $target" + else + rm ${LOG_DIR}/$target.ERR + fi + + TOTAL_CNT=$((TOTAL_CNT + 1))

${CROSS_COMPILE}size ${BUILD_DIR}/u-boot \ | tee -a ${LOG_DIR}/$target.MAKELOG @@ -907,7 +924,17 @@ build_target() {

#-----------------------------------------------------------------------

+print_stats() { + echo "" + echo "--------------------- SUMMARY ----------------------------" + echo "Boards compiled: ${TOTAL_CNT}" + if [ ${ERR_CNT} -gt 0 ] ; then + echo "Boards with warnings or errors: ${ERR_CNT} (${ERR_LIST} )" + fi + echo "----------------------------------------------------------" +}

+#----------------------------------------------------------------------- for arg in $@ do case "$arg" in