From 52b932df3b32505b9534546e053727a04259bc53 Mon Sep 17 00:00:00 2001 From: Martin Roth Date: Sun, 20 Apr 2025 14:03:19 -0600 Subject: [PATCH] util/abuild: Group printfs to timestamps file together This fixes the abuild warning/suggestion. Change-Id: I3a28811becfde69c3e539406bde5938445f16c29 Signed-off-by: Martin Roth Reviewed-on: https://review.coreboot.org/c/coreboot/+/87374 Reviewed-by: Matt DeVillier Tested-by: build bot (Jenkins) --- util/abuild/abuild | 12 +++++++----- 1 file changed, 7 insertions(+), 5 deletions(-) diff --git a/util/abuild/abuild b/util/abuild/abuild index 6199b7d2b1..4a01737af0 100755 --- a/util/abuild/abuild +++ b/util/abuild/abuild @@ -426,11 +426,13 @@ compile_target() cd "${build_dir}" || return $? timestamps="abuild.timestamps" - printf "Build started %s\n" "${ts_basetime_str}" > "${timestamps}" - printf "BASETIME_SECONDS %d\n" ${ts_exec_shell} >> "${timestamps}" - printf "TS_0 %d\n" ${ts_0} >> "${timestamps}" - printf "TS_1 %d\n" ${ts_1} >> "${timestamps}" - printf "TS_2 %d\n" ${ts_2} >> "${timestamps}" + { + printf "Build started %s\n" "${ts_basetime_str}" + printf "BASETIME_SECONDS %d\n" ${ts_exec_shell} + printf "TS_0 %d\n" ${ts_0} + printf "TS_1 %d\n" ${ts_1} + printf "TS_2 %d\n" ${ts_2} + } > "${timestamps}" duration=$(ts_delta_seconds ${ts_0} ${ts_2}) duration_str=$(ts_delta_string ${ts_0} ${ts_2})