From 8aeabdf65e5d4373e4d66490c5e8987f2999fd4e Mon Sep 17 00:00:00 2001
From: chrysn <chrysn@fsfe.org>
Date: Fri, 24 Nov 2023 21:11:53 +0100
Subject: [PATCH] CI: Overhaul debug output

At least with the current GitHub versions, ::echo ::on doesn't do
anything.
---
 .github/workflows/buildtest.yml | 16 ++++++----------
 1 file changed, 6 insertions(+), 10 deletions(-)

diff --git a/.github/workflows/buildtest.yml b/.github/workflows/buildtest.yml
index 1c07ac30..bf3027c0 100644
--- a/.github/workflows/buildtest.yml
+++ b/.github/workflows/buildtest.yml
@@ -28,6 +28,7 @@ jobs:
         path: RIOT
     - name: Patch .cargo/config.toml to use current checkout
       run: |
+        set -x
         cd RIOT
         rm -f .cargo/config.toml
         mkdir -p .cargo # Keep working if RIOT ever decides it doesn't need overrides any more
@@ -38,9 +39,7 @@ jobs:
       # No sense in running this in parallel -- this will download the index
       # and all relevant crates once, and after that, just make some notes in Cargo.lock
       run: |
-        # It appears that there has to be output before :: commands really catch on
-        echo "Pulling updates"
-        echo "::echo ::on"
+        set -x
         for MANIF in $(find RIOT -name Cargo.toml)
         do
             echo "::group::Updating ${MANIF}"
@@ -70,6 +69,7 @@ jobs:
         path: RIOT
     - name: Patch .cargo/config.toml to use current checkout
       run: |
+        set -x
         cd RIOT
         rm -f .cargo/config.toml
         mkdir -p .cargo # Keep working if RIOT ever decides it doesn't need overrides any more
@@ -80,9 +80,7 @@ jobs:
       # No sense in running this in parallel -- this will download the index
       # and all relevant crates once, and after that, just make some notes in Cargo.lock
       run: |
-        # It appears that there has to be output before :: commands really catch on
-        echo "Pulling updates"
-        echo "::echo ::on"
+        set -x
         for MANIF in $(find RIOT -name Cargo.toml)
         do
             echo "::group::Updating ${MANIF}"
@@ -95,15 +93,13 @@ jobs:
 
     - name: Build and run tests
       run: |
+        set -x
         DIRS=$(find RIOT/tests -name Makefile | xargs -n1 dirname)
         export RIOTBASE=$(pwd)/RIOT
-        # It appears that there has to be output before :: commands really catch on
-        echo "Building ${DIRS} on ${{ matrix.board }}"
-        echo "::echo ::on"
         for D in ${DIRS}; do
           echo "::group::Building ${D}"
           cd ${D}
-          BOARDS=${{ matrix.board }} make info-boards-supported | grep -q . || ( cd -; continue )
+          BOARDS=${{ matrix.board }} make info-boards-supported | grep -q . || ( echo "Board is not supported for this test"; cd -; continue )
           BOARD=${{ matrix.board }} make all
           echo "::endgroup::"
           if make test/available BOARD=native && [ "native" = "${{ matrix.board }}" ]; then