abuild: Update failed boards handling
- Use TARGET variable for location of passing/failing boards files. This should better handle the directory, wherever it is. - Don't save make.log location if make.log is being deleted. Change-Id: I28e55feef85c9b642ac5ff70ecef113cf7978707 Signed-off-by: Martin Roth <gaumless@gmail.com> Reviewed-on: https://review.coreboot.org/27596 Tested-by: build bot (Jenkins) <no-reply@coreboot.org> Reviewed-by: Paul Menzel <paulepanter@users.sourceforge.net> Reviewed-by: Patrick Georgi <pgeorgi@google.com>
This commit is contained in:
parent
4e71436d5f
commit
7a98591a1d
|
@ -379,7 +379,11 @@ function compile_target
|
||||||
printf "failed\n" > compile.status
|
printf "failed\n" > compile.status
|
||||||
printf "%s build FAILED after %ss!\nLog excerpt:\n" "$BUILD_NAME" "${duration}"
|
printf "%s build FAILED after %ss!\nLog excerpt:\n" "$BUILD_NAME" "${duration}"
|
||||||
tail -n $CONTEXT make.log 2> /dev/null || tail -$CONTEXT make.log
|
tail -n $CONTEXT make.log 2> /dev/null || tail -$CONTEXT make.log
|
||||||
echo "$BUILD_NAME - Log: ${TOP}/${build_dir}/make.log" >> "$FAILED_BOARDS"
|
if [ "$clean_work" = "true" ]; then
|
||||||
|
echo "$BUILD_NAME" >> "$FAILED_BOARDS"
|
||||||
|
else
|
||||||
|
echo "$BUILD_NAME - Log: ${build_dir}/make.log" >> "$FAILED_BOARDS"
|
||||||
|
fi
|
||||||
failed=1
|
failed=1
|
||||||
fi
|
fi
|
||||||
cd "$CURR" || return $?
|
cd "$CURR" || return $?
|
||||||
|
@ -740,8 +744,8 @@ if [ "$customizing" = "" ]; then
|
||||||
customizing="default configuration"
|
customizing="default configuration"
|
||||||
fi
|
fi
|
||||||
|
|
||||||
FAILED_BOARDS="${COREBOOT_BUILD_DIR:-${TOP}/coreboot-builds}/failed_boards"
|
FAILED_BOARDS="${TARGET}/failed_boards"
|
||||||
PASSED_BOARDS="${COREBOOT_BUILD_DIR:-${TOP}/coreboot-builds}/passing_boards"
|
PASSED_BOARDS="${TARGET}/passing_boards"
|
||||||
|
|
||||||
if [ "$recursive" = "false" ]; then
|
if [ "$recursive" = "false" ]; then
|
||||||
rm -f "$FAILED_BOARDS" "$PASSED_BOARDS"
|
rm -f "$FAILED_BOARDS" "$PASSED_BOARDS"
|
||||||
|
|
Loading…
Reference in New Issue