ci-dist update from PDLPorters

mohawk2 · mohawk2 · commit f68a489ab34a · 2025-04-15T21:29:47.000+01:00
diff --git a/github-actions/ci-dist/action.yml b/github-actions/ci-dist/action.yml
@@ -147,107 +147,72 @@ runs:
   steps:
     # Target: none
     # Shared step to do initial set up
-    - name: initial setup
+    - name: set SYSTEM_CORES (Linux)
       shell: bash
-      run: |
-          echo "::group::initial setup"
-          export SYSTEM_CORES=1;
-          if ${{ toJSON( runner.os == 'Linux' ) }}; then
-            export SYSTEM_CORES="$(grep -c ^processor /proc/cpuinfo 2>/dev/null || echo 1 )"
-          elif ${{ toJSON( runner.os == 'macOS' ) }}; then
-            export SYSTEM_CORES="$(sysctl -n hw.ncpu || echo 1)"
-          elif ${{ toJSON( runner.os == 'Windows' ) }}; then
-            export SYSTEM_CORES=$( powershell -NoProfile -Command "& { Get-WmiObject Win32_Processor | Select-Object -Property NumberOfCores }" | perl -0777 -ne 'print $1, "\n" while /(\d+)/g' );
-          fi
-          echo "SYSTEM_CORES=$SYSTEM_CORES" >> $GITHUB_ENV
-          if ${{ toJSON(
-                 fromJSON(inputs.build-enable-parallel || 'true')
-              ) }}; then
-            echo "MAKEFLAGS=-j$(( SYSTEM_CORES + 1))" >> $GITHUB_ENV
-          fi
-
-          #if ${{ toJSON( runner.os == 'Windows' ) }}; then
-          #  # Set TEMP to full path instead of 8.3 short filenames.
-          #  # See <https://github.com/actions/virtual-environments/issues/712>.
-          #  # $USERPROFILE\\AppData\\Local\\Temp
-          #  [ -d 'C:\tmp' ] || mkdir 'C:\tmp';
-          #  echo 'TMP=C:\tmp' >> $GITHUB_ENV
-          #  echo 'TEMP=C:\tmp' >> $GITHUB_ENV
-          #fi
+      if: runner.os == 'Linux'
+      run: echo SYSTEM_CORES="$(grep -c ^processor /proc/cpuinfo 2>/dev/null || echo 1 )" >> $GITHUB_ENV
+    - name: set SYSTEM_CORES (macOS)
+      shell: bash
+      if: runner.os == 'macOS'
+      run: echo SYSTEM_CORES="$(sysctl -n hw.ncpu || echo 1)" >> $GITHUB_ENV
+    - name: set SYSTEM_CORES (Windows)
+      shell: bash
+      if: runner.os == 'Windows'
+      run: echo SYSTEM_CORES=$( powershell -NoProfile -Command "& { Get-WmiObject Win32_Processor | Select-Object -Property NumberOfCores }" | perl -0777 -ne 'print $1, "\n" while /(\d+)/g' ) >> $GITHUB_ENV
+    - name: set MAKEFLAGS
+      shell: bash
+      if: fromJSON(inputs.build-enable-parallel)
+      run: echo "MAKEFLAGS=-j$(( SYSTEM_CORES + 1))" >> $GITHUB_ENV
 
-          # This is needed for Clang 12 due to
-          # <https://github.com/Perl/perl5/issues/18780>.
-          if ${{ toJSON( runner.os == 'macOS' ) }}; then
-            PATCH_EUMM_DIR=$HOME/build_aux
-            [ -d $PATCH_EUMM_DIR ] || mkdir $PATCH_EUMM_DIR
-            if [ ! -f $PATCH_EUMM_DIR/clang12_no_warnings.pm ]; then
-              echo 'package clang12_no_warnings; use ExtUtils::MakeMaker::Config; $Config{ccflags} = join " ", $Config{ccflags}, "-Wno-compound-token-split-by-macro"; 1;' > $PATCH_EUMM_DIR/clang12_no_warnings.pm
-              echo "PERL5OPT=${PERL5OPT:+${PERL5OPT} }-I$PATCH_EUMM_DIR -Mclang12_no_warnings" >> $GITHUB_ENV
-            fi
-          fi
-          echo "::endgroup::"
+    - name: Suppress Clang 12+ compiler spew
+      shell: bash
+      if: runner.os == 'macOS'
+      run: |
+        # This is needed for Clang 12 due to
+        # <https://github.com/Perl/perl5/issues/18780>.
+        PATCH_EUMM_DIR=$HOME/build_aux
+        [ -d $PATCH_EUMM_DIR ] || mkdir $PATCH_EUMM_DIR
+        if [ ! -f $PATCH_EUMM_DIR/clang12_no_warnings.pm ]; then
+          echo 'package clang12_no_warnings; use ExtUtils::MakeMaker::Config; $Config{ccflags} = join " ", $Config{ccflags}, "-Wno-compound-token-split-by-macro"; 1;' > $PATCH_EUMM_DIR/clang12_no_warnings.pm
+          echo "PERL5OPT=${PERL5OPT:+${PERL5OPT} }-I$PATCH_EUMM_DIR -Mclang12_no_warnings" >> $GITHUB_ENV
+        fi
     # Shared step for setting up graphical display.
-    - name: Set up graphical display
+    - name: Set up graphical display (via apt-get)
       shell: bash
+      if: runner.os == 'Linux' && (fromJSON(inputs.build-enable-graphical-display) || fromJSON(inputs.test-enable-graphical-display))
       run: |
-          if ${{ toJSON(
-                (
-                  fromJSON(inputs.build-enable-graphical-display || 'false')
-                  ||
-                  fromJSON(inputs.test-enable-graphical-display || 'false')
-                )
-              ) }}; then
-            if ${{ toJSON( runner.os == 'Linux' ) }}; then
-              echo "::group::Install xvfb (via apt-get)"
-              sudo apt-get install xvfb
-              mkdir /tmp/runtime-runner && chmod 0700 /tmp/runtime-runner
-              echo "XDG_RUNTIME_DIR=/tmp/runtime-runner" >> $GITHUB_ENV
-              if ${{ fromJSON(inputs.build-enable-graphical-display || 'false') }}; then
-                echo "BUILD_RUNNER_PREFIX=xvfb-run -a" >> $GITHUB_ENV
-              fi
-              if ${{ fromJSON(inputs.test-enable-graphical-display || 'false') }}; then
-                echo "TEST_RUNNER_PREFIX=xvfb-run -a" >> $GITHUB_ENV
-              fi
-              echo "::endgroup::"
-            fi
-          fi
+        sudo apt-get install --no-install-recommends -y xvfb
+        mkdir /tmp/runtime-runner && chmod 0700 /tmp/runtime-runner
+        echo "XDG_RUNTIME_DIR=/tmp/runtime-runner" >> $GITHUB_ENV
+        if ${{ fromJSON(inputs.build-enable-graphical-display) }}; then
+          echo "BUILD_RUNNER_PREFIX=xvfb-run -a" >> $GITHUB_ENV
+        fi
+        if ${{ fromJSON(inputs.test-enable-graphical-display) }}; then
+          echo "TEST_RUNNER_PREFIX=xvfb-run -a" >> $GITHUB_ENV
+        fi
     # Target: target-setup-perl
     - name: target-setup-perl (actions-setup-perl)
       shell: bash
+      if: fromJSON(inputs.target-all) || fromJSON(inputs.target-setup-perl)
       run: |
-          if ${{ toJSON(
-                (  fromJSON(inputs.target-all)
-                || fromJSON(inputs.target-setup-perl)
-                )
-              ) }}; then
-            echo "::group::setup-perl (actions-setup-perl)"
-            git clone \
-                    https://github.com/shogo82148/actions-setup-perl.git \
-                    -b v1 \
-                    .github/actions/actions-setup-perl;
-            node .github/actions/actions-setup-perl/dist/setup/index.js
-            echo "::endgroup::"
-          fi
+        git clone \
+                https://github.com/shogo82148/actions-setup-perl.git \
+                -b v1 \
+                .github/actions/actions-setup-perl;
+        node .github/actions/actions-setup-perl/dist/setup/index.js
       env:
         INPUT_DISTRIBUTION: ${{ inputs.perl-distribution }}
         INPUT_PERL-VERSION: ${{ inputs.perl-version }}
         INPUT_MULTI-THREAD: ${{ inputs.perl-threaded }}
         INPUT_ENABLE-MODULES-CACHE: false
     - name: target-setup-perl (perl -V)
       shell: bash
+      if: fromJSON(inputs.target-all) || fromJSON(inputs.target-setup-perl)
       run: |
-          if ${{ toJSON(
-                (  fromJSON(inputs.target-all)
-                || fromJSON(inputs.target-setup-perl)
-                )
-              ) }}; then
-            echo "::group::setup-perl (perl -V)"
-            export MYPERL=$(which -a perl | grep -m 1 hostedtoolcache)
-            echo "MYPERL=$MYPERL" >> $GITHUB_ENV
-            $MYPERL -V
-            $MYPERL -MDigest::MD5=md5_hex -e 'print "GHA_CACHE_PERL_V_HASH=", md5_hex(`$^X -V`), "\n"' >> $GITHUB_ENV
-            echo "::endgroup::"
-          fi
+        export MYPERL=$(which -a perl | grep -m 1 hostedtoolcache)
+        echo "MYPERL=$MYPERL" >> $GITHUB_ENV
+        $MYPERL -V
+        $MYPERL -MDigest::MD5=md5_hex -e 'print "GHA_CACHE_PERL_V_HASH=", md5_hex(`$^X -V`), "\n"' >> $GITHUB_ENV
     - name: target-setup-perl (cache ~/perl5)
       if: |
                 fromJSON(inputs.perl-deps-enable-cache)
@@ -257,160 +222,114 @@ runs:
                 )
       uses: actions/cache@v4
       with:
-        key: locallib-v2.${{ inputs.perl-deps-cache-version }}-${{ runner.os }}-${{ env.GHA_CACHE_PERL_V_HASH }}
+        key: locallib-v2.${{ inputs.perl-deps-cache-version }}-${{ runner.os }}-${{ inputs.perl-version }}-${{ env.GHA_CACHE_PERL_V_HASH }}
         path: ~/perl5
     - name: target-setup-perl (ccache)
       # ccache support on Windows is not well-tested
       if: |
-                fromJSON(inputs.ccache-enable-cache)
-                &&
-                (  fromJSON(inputs.target-all)
-                || fromJSON(inputs.target-setup-perl)
-                )
-                &&
-                (
-                  runner.os != 'Windows'
-                )
+        runner.os != 'Windows' &&
+        fromJSON(inputs.ccache-enable-cache) &&
+        ( fromJSON(inputs.target-all) ||
+          fromJSON(inputs.target-setup-perl)
+        )
       uses: Chocobo1/setup-ccache-action@v1
       with:
         override_cache_key: ccache-v0.${{ inputs.ccache-cache-version }}-${{ runner.os }}-${{ env.GHA_CACHE_PERL_V_HASH }}
     - name: target-setup-perl (setup local::lib)
       shell: bash
+      if: fromJSON(inputs.target-all) || fromJSON(inputs.target-setup-perl)
       run: |
-          if ${{ toJSON(
-                (  fromJSON(inputs.target-all)
-                || fromJSON(inputs.target-setup-perl)
-                )
-              ) }}; then
-            echo "::group::setup-perl (setup local::lib)"
-            # older Perls need this upgrade
-            $MYPERL -S cpanm -n --local-lib=~/perl5 ExtUtils::MakeMaker
-            $MYPERL -S cpanm -n --local-lib=~/perl5 local::lib
-            LOCALLIB_VARS="$($MYPERL -I ~/perl5/lib/perl5 -Mlocal::lib=--shelltype,bourne )"
-            eval $( echo "$LOCALLIB_VARS" | $MYPERL -MFile::Spec -pne '
-              $_ = "" if /^PATH=/;
-              s/; export.*$/;/;
-              s/^[^=]+=/echo -n "$&"; echo /;
-              my $sep = File::Spec->catfile("", "");
-              s,/perl5,${sep}perl5,g;' ) >> $GITHUB_ENV
-            if ${{ toJSON( runner.os == 'Windows' ) }}; then
-              pwsh -c 'echo "'$( cygpath -w $( realpath ~/perl5/bin ) )'" | Out-File -FilePath $env:GITHUB_PATH -Encoding utf8 -Append'
-            else
-              echo "$HOME/perl5/bin" >> $GITHUB_PATH
-            fi
-            echo "PERL_CPANM_OPT=-l ~/perl5" >> $GITHUB_ENV
-            echo "::endgroup::"
-          fi
+        # older Perls need this upgrade
+        $MYPERL -S cpanm -n --local-lib=~/perl5 ExtUtils::MakeMaker || ( cat ~/.cpanm/build.log && false )
+        $MYPERL -S cpanm -n --local-lib=~/perl5 local::lib || ( cat ~/.cpanm/build.log && false )
+        LOCALLIB_VARS="$($MYPERL -I ~/perl5/lib/perl5 -Mlocal::lib=--shelltype,bourne )"
+        eval $( echo "$LOCALLIB_VARS" | $MYPERL -MFile::Spec -pne '
+          $_ = "" if /^PATH=/;
+          s/; export.*$/;/;
+          s/^[^=]+=/echo -n "$&"; echo /;
+          my $sep = File::Spec->catfile("", "");
+          s,/perl5,${sep}perl5,g;' ) >> $GITHUB_ENV
+        if ${{ toJSON( runner.os == 'Windows' ) }}; then
+          pwsh -c 'echo "'$( cygpath -w $( realpath ~/perl5/bin ) )'" | Out-File -FilePath $env:GITHUB_PATH -Encoding utf8 -Append'
+        else
+          echo "$HOME/perl5/bin" >> $GITHUB_PATH
+        fi
+        echo "PERL_CPANM_OPT=-l ~/perl5" >> $GITHUB_ENV
     # Target: target-install-dist-perl-deps
     - name: target-install-dist-perl-deps (via cpanm)
       shell: bash
+      if: fromJSON(inputs.target-all) || fromJSON(inputs.target-install-dist-perl-deps)
       run: |
-          if ${{ toJSON(
-                (  fromJSON(inputs.target-all)
-                || fromJSON(inputs.target-install-dist-perl-deps)
-                )
-              ) }}; then
-            echo "::group::install-dist-perl-deps (via cpanm)"
-            cd ${{ inputs.path }}
-            if [ -n "${{ inputs.dist-perl-deps-configure }}" ]; then
-              $BUILD_RUNNER_PREFIX $MYPERL -S cpanm --dev -n ${{ inputs.dist-perl-deps-configure }} || ( cat ~/.cpanm/build.log && false )
-            fi
-            $BUILD_RUNNER_PREFIX $MYPERL -S cpanm -n --installdeps --dev . || ( cat ~/.cpanm/build.log && false )
-            echo "::endgroup::"
-          fi
+        cd ${{ inputs.path }}
+        if [ -n "${{ inputs.dist-perl-deps-configure }}" ]; then
+          $BUILD_RUNNER_PREFIX $MYPERL -S cpanm --dev -n ${{ inputs.dist-perl-deps-configure }} || ( cat ~/.cpanm/build.log && false )
+        fi
+        $BUILD_RUNNER_PREFIX $MYPERL -S cpanm -n --installdeps --dev . || ( cat ~/.cpanm/build.log && false )
     # Target: target-test-release-testing
     # Run release tests, before others as may install useful stuff
     - name: target-test-release-testing (via cpanm)
       shell: bash
+      if: (fromJSON(inputs.target-all) || fromJSON(inputs.target-test-release-testing)) && fromJSON(inputs.test-enable-release-testing)
       env:
         AUTOMATED_TESTING: 1
         RELEASE_TESTING: 1
       run: |
-          if ${{ toJSON(
-                (  fromJSON(inputs.target-all)
-                || fromJSON(inputs.target-test-release-testing)
-                )
-                && fromJSON(inputs.test-enable-release-testing || 'false')
-              ) }}; then
-            echo "::group::test-release-testing (via cpanm)"
-            cd ${{ inputs.path }}
-            $BUILD_RUNNER_PREFIX $MYPERL -S cpanm -n --installdeps --with-develop . || cat ~/.cpanm/build.log
-            $BUILD_RUNNER_PREFIX $MYPERL -S cpanm -n CPAN::Changes || cat ~/.cpanm/build.log
-            if [ -d xt ]; then
-              $TEST_RUNNER_PREFIX $MYPERL -S prove -lr xt
-            else
-              echo "Warning: no directory xt/" >&2
-            fi
-            echo "::endgroup::"
-          fi
+        cd ${{ inputs.path }}
+        $BUILD_RUNNER_PREFIX $MYPERL -S cpanm -n --installdeps --with-develop . || ( cat ~/.cpanm/build.log && false )
+        $BUILD_RUNNER_PREFIX $MYPERL -S cpanm -n CPAN::Changes || ( cat ~/.cpanm/build.log && false )
+        if [ -d xt ]; then
+          $TEST_RUNNER_PREFIX $MYPERL -S prove -lr xt
+        else
+          echo "Warning: no directory xt/" >&2
+        fi
     # Target: target-test
     - name: target-test (prepare for testing)
       shell: bash
+      if: ${{ 0+(fromJSON(inputs.target-all) || fromJSON(inputs.target-test)) }}
       env:
         AUTOMATED_TESTING: 1
         HARNESS_OPTIONS: ${{ inputs.test-harness-options }}
       run: |
-          if ${{ toJSON(
-                (  fromJSON(inputs.target-all)
-                || fromJSON(inputs.target-test)
-                )
-              ) }}; then
-            echo "::group::test (prepare for testing)"
-            cd ${{ inputs.path }}
-            if [ -f Makefile.PL ]; then
-              ( [ -f Makefile ] || $BUILD_RUNNER_PREFIX $MYPERL Makefile.PL )
-            elif [ -f Build.PL ]; then
-              ( [ -f Build ] || $BUILD_RUNNER_PREFIX $MYPERL Build.PL )
-            else
-              echo "No file Makefile.PL or Build.PL" >&2
-            fi
-            echo "::endgroup::"
-          fi
-    - name: GNU make for Windows
+        cd ${{ inputs.path }}
+        if [ -f Makefile.PL ]; then
+          ( [ -f Makefile ] || $BUILD_RUNNER_PREFIX $MYPERL Makefile.PL )
+        elif [ -f Build.PL ]; then
+          ( [ -f Build ] || $BUILD_RUNNER_PREFIX $MYPERL Build.PL )
+        else
+          echo "No file Makefile.PL or Build.PL" >&2
+        fi
+    - name: set MYMAKE (Windows)
       shell: bash
-      if: ${{ matrix.os == 'windows-latest' }}
-      run: |
-        choco install --verbose make
+      if: runner.os == 'Windows'
+      run: echo MYMAKE=gmake >> $GITHUB_ENV
+    - name: set MYMAKE (non-Windows)
+      shell: bash
+      if: runner.os != 'Windows'
+      run: echo MYMAKE=make >> $GITHUB_ENV
     - name: target-test (no coverage)
       shell: bash
+      if: ${{ (fromJSON(inputs.target-all) || fromJSON(inputs.target-test)) && !fromJSON(inputs.test-enable-coverage) }}
       env:
         AUTOMATED_TESTING: 1
         HARNESS_OPTIONS: ${{ inputs.test-harness-options }}
       run: |
-          if ${{ toJSON(
-                (  fromJSON(inputs.target-all)
-                || fromJSON(inputs.target-test)
-                )
-                && ! fromJSON(inputs.test-enable-coverage || 'false')
-              ) }}; then
-            echo "::group::test (no coverage)"
-            cd ${{ inputs.path }}
-            if [ -f Makefile.PL ]; then
-              ( $TEST_RUNNER_PREFIX make && $TEST_RUNNER_PREFIX make test ) || exit 1
-            elif [ -f Build.PL ]; then
-              ( $TEST_RUNNER_PREFIX ./Build test ) || exit 1
-            else
-              echo "No file Makefile.PL or Build.PL" >&2
-            fi
-            echo "::endgroup::"
-          fi
+        cd ${{ inputs.path }}
+        if [ -f Makefile.PL ]; then
+          ( $TEST_RUNNER_PREFIX $MYMAKE && $TEST_RUNNER_PREFIX $MYMAKE test ) || exit 1
+        elif [ -f Build.PL ]; then
+          ( $TEST_RUNNER_PREFIX $MYPERL Build test ) || exit 1
+        else
+          echo "No file Makefile.PL or Build.PL" >&2
+        fi
     - name: target-test (with coverage)
       shell: bash
+      if: ${{ (fromJSON(inputs.target-all) || fromJSON(inputs.target-test)) && fromJSON(inputs.test-enable-coverage) }}
       env:
         AUTOMATED_TESTING: 1
         HARNESS_OPTIONS: ${{ inputs.test-harness-options }}
         GITHUB_TOKEN: ${{ inputs.github-token }}
       run: |
-          if ${{ toJSON(
-                (  fromJSON(inputs.target-all)
-                || fromJSON(inputs.target-test)
-                )
-                && fromJSON(inputs.test-enable-coverage || 'false')
-              ) }}; then
-            echo "::group::test (with coverage)"
-            cd ${{ inputs.path }}
-            $MYPERL -S cpanm -n Devel::Cover::Report::Coveralls || cat ~/.cpanm/build.log
-            $MYPERL -S cpanm -n git://github.com/PDLPorters/Devel--Cover.git@patches || cat ~/.cpanm/build.log
-            $TEST_RUNNER_PREFIX $MYPERL -S cover -test -relative_only -gcov_chdir -report Coveralls
-            echo "::endgroup::"
-          fi
+        cd ${{ inputs.path }}
+        $MYPERL -S cpanm -n Devel::Cover::Report::Coveralls || ( cat ~/.cpanm/build.log && false )
+        $MYPERL -S cpanm -n https://github.com/PDLPorters/Devel--Cover.git@patches || ( cat ~/.cpanm/build.log && false )
+        $TEST_RUNNER_PREFIX $MYPERL -S cover -test -relative_only -gcov_chdir -report Coveralls
