ci: Fix GNU testsuite comparison

tavianator · tavianator · commit cead7121185b · 2022-02-22T14:30:56.000-05:00
The uploaded log artifact maintains the directory structure, so just
compare it directly to the GNU find directory.
diff --git a/.github/workflows/compat.yml b/.github/workflows/compat.yml
@@ -70,9 +70,7 @@ jobs:
     - name: Compare failing tests against master
       shell: bash
       run: |
-        mkdir -p ./logs
-        cp findutils.gnu/{tests,{find,xargs}/testsuite}/*.log ./logs
-        ./findutils/util/diff-gnu.sh dl logs
+        ./findutils/util/diff-gnu.sh ./dl ./findutils.gnu
     - name: Compare against main results
       shell: bash
       run: |
diff --git a/util/diff-gnu.sh b/util/diff-gnu.sh
@@ -6,10 +6,10 @@ export LC_COLLATE=C
 
 # Extract the failing test lines from log files
 failing_tests() {
-    sed -En 's/FAIL: ([^,:]*)[,:].*/\1/p' "$@" | sort
+    sed -En 's/FAIL: ([^,:]*)[,:].*/\1/p' "$1"/{tests,{find,xargs}/testsuite}/*.log | sort
 }
 
-comm -3 <(failing_tests $1/*.log) <(failing_tests $2/*.log) | tr '\t' ',' | while IFS=, read old new foo; do
+comm -3 <(failing_tests "$1") <(failing_tests "$2") | tr '\t' ',' | while IFS=, read old new foo; do
     if [ -n "$old" ]; then
         echo "::warning ::Congrats! The GNU test $old is now passing!"
     fi
