From 075e99c7dac44721b503341a9f91837d73a7c9fd Mon Sep 17 00:00:00 2001
From: Timo Wilken <timo.wilken@cern.ch>
Date: Wed, 28 Feb 2024 18:07:46 +0100
Subject: [PATCH] Distinguish between unpacking tarball and compiling

Make it obvious to the user whether we're actually compiling a package or just
unpacking a prebuilt tarball.
---
 alibuild_helpers/build.py | 12 ++++++------
 1 file changed, 6 insertions(+), 6 deletions(-)

diff --git a/alibuild_helpers/build.py b/alibuild_helpers/build.py
index 1a7ebc0a..117275f2 100644
--- a/alibuild_helpers/build.py
+++ b/alibuild_helpers/build.py
@@ -1010,9 +1010,6 @@ def doBuild(args, parser):
       "runtime_requires": " ".join(spec["runtime_requires"]),
     })
 
-    banner("Building %s@%s", spec["package"],
-           args.develPrefix if "develPrefix" in args and spec["package"] in develPkgs
-           else spec["version"])
     # Define the environment so that it can be passed up to the
     # actual build script
     buildEnvironment = [
@@ -1076,9 +1073,12 @@ def doBuild(args, parser):
       build_command = "%s -e -x %s/build.sh 2>&1" % (BASH, quote(scriptDir))
 
     debug("Build command: %s", build_command)
-    progress = ProgressPrint("%s is being built (use --debug for full output)" % spec["package"])
-    err = execute(build_command, printer=debug if args.debug or not sys.stdout.isatty() else progress)
-    progress.end("failed" if err else "ok", err)
+    progress = ProgressPrint("%s %s@%s (use --debug for full output)" % (
+        "Unpacking tarball for" if cachedTarball else "Compiling", spec["package"],
+        args.develPrefix if "develPrefix" in args and spec["is_devel_pkg"] else spec["version"],
+    ))
+    err = execute(build_command, printer=progress)
+    progress.end("failed" if err else "done", err)
     report_event("BuildError" if err else "BuildSuccess", spec["package"], " ".join((
       args.architecture,
       spec["version"],