Skip to content

Commit 102b315

Browse files
committed
ci-dist update from PDLPorters
1 parent f7a60a8 commit 102b315

File tree

1 file changed

+16
-13
lines changed

1 file changed

+16
-13
lines changed

github-actions/ci-dist/action.yml

Lines changed: 16 additions & 13 deletions
Original file line numberDiff line numberDiff line change
@@ -199,7 +199,7 @@ runs:
199199
) }}; then
200200
if ${{ toJSON( runner.os == 'Linux' ) }}; then
201201
echo "::group::Install xvfb (via apt-get)"
202-
sudo apt-get install xvfb
202+
sudo apt-get install --no-install-recommends -y xvfb
203203
mkdir /tmp/runtime-runner && chmod 0700 /tmp/runtime-runner
204204
echo "XDG_RUNTIME_DIR=/tmp/runtime-runner" >> $GITHUB_ENV
205205
if ${{ fromJSON(inputs.build-enable-graphical-display || 'false') }}; then
@@ -257,7 +257,7 @@ runs:
257257
)
258258
uses: actions/cache@v4
259259
with:
260-
key: locallib-v2.${{ inputs.perl-deps-cache-version }}-${{ runner.os }}-${{ env.GHA_CACHE_PERL_V_HASH }}
260+
key: locallib-v2.${{ inputs.perl-deps-cache-version }}-${{ runner.os }}-${{ inputs.perl-version }}-${{ env.GHA_CACHE_PERL_V_HASH }}
261261
path: ~/perl5
262262
- name: target-setup-perl (ccache)
263263
# ccache support on Windows is not well-tested
@@ -284,8 +284,8 @@ runs:
284284
) }}; then
285285
echo "::group::setup-perl (setup local::lib)"
286286
# older Perls need this upgrade
287-
$MYPERL -S cpanm -n --local-lib=~/perl5 ExtUtils::MakeMaker
288-
$MYPERL -S cpanm -n --local-lib=~/perl5 local::lib
287+
$MYPERL -S cpanm -n --local-lib=~/perl5 ExtUtils::MakeMaker || ( cat ~/.cpanm/build.log && false )
288+
$MYPERL -S cpanm -n --local-lib=~/perl5 local::lib || ( cat ~/.cpanm/build.log && false )
289289
LOCALLIB_VARS="$($MYPERL -I ~/perl5/lib/perl5 -Mlocal::lib=--shelltype,bourne )"
290290
eval $( echo "$LOCALLIB_VARS" | $MYPERL -MFile::Spec -pne '
291291
$_ = "" if /^PATH=/;
@@ -334,8 +334,8 @@ runs:
334334
) }}; then
335335
echo "::group::test-release-testing (via cpanm)"
336336
cd ${{ inputs.path }}
337-
$BUILD_RUNNER_PREFIX $MYPERL -S cpanm -n --installdeps --with-develop . || cat ~/.cpanm/build.log
338-
$BUILD_RUNNER_PREFIX $MYPERL -S cpanm -n CPAN::Changes || cat ~/.cpanm/build.log
337+
$BUILD_RUNNER_PREFIX $MYPERL -S cpanm -n --installdeps --with-develop . || ( cat ~/.cpanm/build.log && false )
338+
$BUILD_RUNNER_PREFIX $MYPERL -S cpanm -n CPAN::Changes || ( cat ~/.cpanm/build.log && false )
339339
if [ -d xt ]; then
340340
$TEST_RUNNER_PREFIX $MYPERL -S prove -lr xt
341341
else
@@ -366,11 +366,14 @@ runs:
366366
fi
367367
echo "::endgroup::"
368368
fi
369-
- name: GNU make for Windows
369+
- name: set MYMAKE (Windows)
370370
shell: bash
371-
if: ${{ matrix.os == 'windows-latest' }}
372-
run: |
373-
choco install --verbose make
371+
if: runner.os == 'Windows'
372+
run: echo MYMAKE=gmake >> $GITHUB_ENV
373+
- name: set MYMAKE (non-Windows)
374+
shell: bash
375+
if: runner.os != 'Windows'
376+
run: echo MYMAKE=make >> $GITHUB_ENV
374377
- name: target-test (no coverage)
375378
shell: bash
376379
env:
@@ -388,7 +391,7 @@ runs:
388391
if [ -f Makefile.PL ]; then
389392
( $TEST_RUNNER_PREFIX make && $TEST_RUNNER_PREFIX make test ) || exit 1
390393
elif [ -f Build.PL ]; then
391-
( $TEST_RUNNER_PREFIX ./Build test ) || exit 1
394+
( $TEST_RUNNER_PREFIX $MYPERL Build test ) || exit 1
392395
else
393396
echo "No file Makefile.PL or Build.PL" >&2
394397
fi
@@ -409,8 +412,8 @@ runs:
409412
) }}; then
410413
echo "::group::test (with coverage)"
411414
cd ${{ inputs.path }}
412-
$MYPERL -S cpanm -n Devel::Cover::Report::Coveralls || cat ~/.cpanm/build.log
413-
$MYPERL -S cpanm -n git://github.com/PDLPorters/Devel--Cover.git@patches || cat ~/.cpanm/build.log
415+
$MYPERL -S cpanm -n Devel::Cover::Report::Coveralls || ( cat ~/.cpanm/build.log && false )
416+
$MYPERL -S cpanm -n https://github.com/PDLPorters/Devel--Cover.git@patches || ( cat ~/.cpanm/build.log && false )
414417
$TEST_RUNNER_PREFIX $MYPERL -S cover -test -relative_only -gcov_chdir -report Coveralls
415418
echo "::endgroup::"
416419
fi

0 commit comments

Comments
 (0)