[PATCH v4 0/1] MR762: Draft: ci: Delimit building each commit with a collapsible section.

[Giovanni Mascellani (＠giomasce)]

--
  v4: ci: Delimit building each commit with a collapsible section.
 
https://gitlab.winehq.org/wine/vkd3d/-/merge_requests/762

[PATCH v4 1/1] ci: Delimit building each commit with a collapsible section.

[Giovanni Mascellani]

From: Giovanni Mascellani <gmascellani(a)codeweavers.com>

---
 gitlab/build-crosstest | 7 ++++---
 gitlab/build-linux     | 7 ++++---
 gitlab/build-mac       | 7 ++++---
 gitlab/build-mingw     | 7 ++++---
 tests/driver.c         | 8 ++++++--
 5 files changed, 22 insertions(+), 14 deletions(-)

diff --git a/gitlab/build-crosstest b/gitlab/build-crosstest
index 4e6c5b65f..71ffede97 100755
--- a/gitlab/build-crosstest
+++ b/gitlab/build-crosstest
@@ -1,10 +1,9 @@
 #!/bin/bash
 
-echo "Building $(git log -1)"
-echo "---"
-
 COMMIT=$(printf '%03d-%s' $(git cherry $CI_MERGE_REQUEST_DIFF_BASE_SHA HEAD^ | wc -l) $(git rev-parse --short HEAD))
 
+echo -e "\e[0Ksection_start:`date +%s`:commit_$COMMIT\r\e[0KBuilding commit $COMMIT"
+
 set -Eeuxo pipefail
 
 # Building with -Wno-array-bounds because MinGW headers currently emit
@@ -25,3 +24,5 @@ rsync -Rr config.log tests/*.txt tests/*.exe ../artifacts/$COMMIT || true
 cp tests/driver.cross64.exe ../artifacts
 
 git reset --hard
+
+echo -e "\e[0Ksection_end:`date +%s`:commit_$COMMIT\r\e[0K"
diff --git a/gitlab/build-linux b/gitlab/build-linux
index b149132c4..2a2a40c20 100755
--- a/gitlab/build-linux
+++ b/gitlab/build-linux
@@ -1,10 +1,9 @@
 #!/bin/bash
 
-echo "Building $(git log -1)"
-echo "---"
-
 COMMIT=$(printf '%03d-%s' $(git cherry $CI_MERGE_REQUEST_DIFF_BASE_SHA HEAD^ | wc -l) $(git rev-parse --short HEAD))
 
+echo -e "\e[0Ksection_start:`date +%s`:commit_$COMMIT\r\e[0KBuilding commit $COMMIT"
+
 set -Eeuxo pipefail
 
 ./autogen.sh
@@ -26,3 +25,5 @@ mkdir -p ../artifacts/$COMMIT
 rsync -Rr config.log doc/* test-suite.log tests/*.log tests/*/*.log ../artifacts/$COMMIT || true
 
 git reset --hard
+
+echo -e "\e[0Ksection_end:`date +%s`:commit_$COMMIT\r\e[0K"
diff --git a/gitlab/build-mac b/gitlab/build-mac
index a31196862..c0efbdf0c 100755
--- a/gitlab/build-mac
+++ b/gitlab/build-mac
@@ -1,10 +1,9 @@
 #!/bin/bash
 
-echo "Building $(git log -1)"
-echo "---"
-
 COMMIT=$(printf '%03d-%s' $(git cherry $CI_MERGE_REQUEST_DIFF_BASE_SHA HEAD^ | wc -l) $(git rev-parse --short HEAD))
 
+echo -e "\033[0Ksection_start:`date +%s`:commit_$COMMIT\r\033[0KBuilding commit $COMMIT"
+
 set -Eeuxo pipefail
 
 ./autogen.sh
@@ -25,3 +24,5 @@ mkdir -p ../artifacts/$COMMIT
 rsync -Rr config.log test-suite.log tests/*.log tests/*/*.log ../artifacts/$COMMIT || true
 
 git reset --hard
+
+echo -e "\033[0Ksection_end:`date +%s`:commit_$COMMIT\r\033[0K"
diff --git a/gitlab/build-mingw b/gitlab/build-mingw
index 6d4cf189c..3937d2244 100755
--- a/gitlab/build-mingw
+++ b/gitlab/build-mingw
@@ -1,10 +1,9 @@
 #!/bin/bash
 
-echo "Building $(git log -1)"
-echo "---"
-
 COMMIT=$(printf '%03d-%s' $(git cherry $CI_MERGE_REQUEST_DIFF_BASE_SHA HEAD^ | wc -l) $(git rev-parse --short HEAD))
 
+echo -e "\e[0Ksection_start:`date +%s`:commit_$COMMIT\r\e[0KBuilding commit $COMMIT"
+
 set -Eeuxo pipefail
 
 ./autogen.sh
@@ -23,3 +22,5 @@ cp config.log ../artifacts/$COMMIT || true
 cp destdir/usr/local/bin/* ../artifacts/$COMMIT || true
 
 git reset --hard
+
+echo -e "\e[0Ksection_end:`date +%s`:commit_$COMMIT\r\e[0K"
diff --git a/tests/driver.c b/tests/driver.c
index 95f39adbb..8ab316fc4 100644
--- a/tests/driver.c
+++ b/tests/driver.c
@@ -19,6 +19,7 @@
 #include <stdio.h>
 #include <stdbool.h>
 #include <stdlib.h>
+#include <time.h>
 #include <windows.h>
 #include <shlobj.h>
 
@@ -141,8 +142,8 @@ static bool run_tests_for_directory(const char *commit_dir)
     if (!test_arch)
         test_arch = "64";
 
-    printf("Building %s\n", commit_dir);
-    printf("---\n");
+    printf("\e[0Ksection_start:%lld:commit_%s\r\e[0KBuilding commit %s\n",
+            (long long int)time(NULL), commit_dir, commit_dir);
 
     sprintf(list_filename, "artifacts/%s/tests/shader_tests.txt", commit_dir);
     list_file = fopen(list_filename, "r");
@@ -232,6 +233,9 @@ static bool run_tests_for_directory(const char *commit_dir)
         }
     }
 
+    printf("\e[0Ksection_end:%lld:commit_%s\r\e[0K\n",
+            (long long int)time(NULL), commit_dir);
+
     return ret;
 }
 
-- 
GitLab

https://gitlab.winehq.org/wine/vkd3d/-/merge_requests/762

[PATCH v4 0/1] MR762: ci: Delimit building each commit with a collapsible section. - approved

[Giovanni Mascellani (＠giomasce)]

This merge request was approved by Giovanni Mascellani.
-- 
https://gitlab.winehq.org/wine/vkd3d/-/merge_requests/762

[PATCH v4 0/1] MR762: ci: Delimit building each commit with a collapsible section.

[Henri Verbeet (＠hverbeet)]

> ```diff
> +    printf("\e[0Ksection_start:%lld:commit_%s\r\e[0KBuilding commit %s\n",
> +            (long long int)time(NULL), commit_dir, commit_dir);
> ```

This warns about %lld here, unless I remove -D__USE_MINGW_ANSI_STDIO=0 from CROSS_CFLAGS. I didn't look much into why that doesn't happen on the CI, but I think we could just use uint64_t and either PRId64 or %I64d instead?

-- 
https://gitlab.winehq.org/wine/vkd3d/-/merge_requests/762#note_67261