From 6e2152f1b90754a8b34d9044dd55abfc8286f16a Mon Sep 17 00:00:00 2001 From: Hans Ulrich Niedermann Date: Wed, 14 Aug 2024 13:35:45 +0200 Subject: [PATCH] ci: Use printf(1) to avoid echo(1) problems Use printf(1) to avoid possible echo(1) incompatibilities like special escape sequences and the like. Also, "\n\n" means two linefeeds, without any implicit linefeeds being added. Thought for the future: Perhaps test-avrdude should run isatty() and, if it is not a TTY, not wait for linefeeds at all in the non- interactive case? --- .github/workflows/build.yml | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-) diff --git a/.github/workflows/build.yml b/.github/workflows/build.yml index a1878298..041a90cc 100644 --- a/.github/workflows/build.yml +++ b/.github/workflows/build.yml @@ -114,7 +114,7 @@ jobs: - name: Install run: sudo cmake --build build --target install - name: Dryrun_test - run: echo -e \\n | ./tools/test-avrdude -d0 -p"-cdryrun -pm2560" -p"-cdryrun -pavr64du28" + run: printf "\n\n" | ./tools/test-avrdude -d0 -p"-cdryrun -pm2560" -p"-cdryrun -pavr64du28" - name: Archive build artifacts if: always() uses: actions/upload-artifact@v3