ci: fix macOS lanes broken by the re-MINIT harness test

The 043 harness links with GNU ld's --unresolved-symbols=ignore-all;
macOS ld64 rejects the flag, the harness fails to compile, and the test
failed on every macOS lane. The technique is ELF-only (ld64's
-undefined dynamic_lookup binds eagerly under chained fixups), so the
test now skips on non-Linux.

The failure also exposed a diagnostics hole in the workflow: under
bash -e, the RESULT=$(php run-tests.php ...) capture aborts the step on
run-tests' non-zero exit before echo "$RESULT" runs, so failing lanes
logged nothing about which test failed. The capture no longer aborts,
and a missing summary line (run-tests crashed) now counts as failure
instead of slipping past the numeric check.

043 verified still passing, not skipped, on Linux.

Author: iliaal

.github/workflows/tests.yml

@@ -75,10 +75,10 @@ jobs:
           export TEST_PHP_EXECUTABLE=$(which php)
           export TEST_PHP_ARGS="-d extension=$(pwd)/modules/mdparser.so"
           export NO_INTERACTION=1
-          RESULT=$(php run-tests.php --show-diff -g FAIL,BORK,LEAK,XLEAK tests/ 2>&1)
+          RESULT=$(php run-tests.php --show-diff -g FAIL,BORK,LEAK,XLEAK tests/ 2>&1) || true
           echo "$RESULT"
           FAILED=$(printf '%s\n' "$RESULT" | awk '/Tests failed/ { print $4; exit }')
-          if [ "$FAILED" -gt 0 ]; then
+          if [ "${FAILED:-1}" -gt 0 ]; then
             echo "::error::$FAILED test(s) failed"
             exit 1
           fi
@@ -114,10 +114,10 @@ jobs:
           export TEST_PHP_EXECUTABLE=$(which php)
           export TEST_PHP_ARGS="-d extension=$(pwd)/modules/mdparser.so"
           export NO_INTERACTION=1
-          RESULT=$(php run-tests.php --show-diff -g FAIL,BORK,LEAK,XLEAK tests/ 2>&1)
+          RESULT=$(php run-tests.php --show-diff -g FAIL,BORK,LEAK,XLEAK tests/ 2>&1) || true
           echo "$RESULT"
           FAILED=$(printf '%s\n' "$RESULT" | awk '/Tests failed/ { print $4; exit }')
-          if [ "$FAILED" -gt 0 ]; then
+          if [ "${FAILED:-1}" -gt 0 ]; then
             echo "::error::$FAILED test(s) failed under ZTS"
             exit 1
           fi
@@ -150,10 +150,10 @@ jobs:
           export TEST_PHP_EXECUTABLE=$(which php)
           export TEST_PHP_ARGS="-d extension=$(pwd)/modules/mdparser.so"
           export NO_INTERACTION=1
-          RESULT=$(php run-tests.php --show-diff -g FAIL,BORK,LEAK,XLEAK tests/ 2>&1)
+          RESULT=$(php run-tests.php --show-diff -g FAIL,BORK,LEAK,XLEAK tests/ 2>&1) || true
           echo "$RESULT"
           FAILED=$(printf '%s\n' "$RESULT" | awk '/Tests failed/ { print $4; exit }')
-          if [ "$FAILED" -gt 0 ]; then
+          if [ "${FAILED:-1}" -gt 0 ]; then
             echo "::error::$FAILED test(s) failed"
             exit 1
           fi

tests/043_mshutdown_reminit_registry.phpt

@@ -4,7 +4,11 @@ MSHUTDOWN then re-MINIT must leave the cmark extension registry usable
 mdparser
 --SKIPIF--
 <?php
-if (substr(PHP_OS, 0, 3) === 'WIN') die('skip needs cc + build-tree objects');
+/* The harness link relies on GNU ld's --unresolved-symbols=ignore-all;
+ * macOS ld64 rejects the flag and its -undefined dynamic_lookup
+ * equivalent binds eagerly under chained fixups, so the technique is
+ * ELF/GNU-ld only. */
+if (PHP_OS !== 'Linux') die('skip GNU ld + build-tree objects required');
 if (!is_file(dirname(__DIR__) . '/.libs/mdparser.o')) die('skip build-tree .libs objects not present');
 $cc = trim((string)shell_exec('command -v cc 2>/dev/null'));
 if ($cc === '') die('skip no cc in PATH');