Skip to content

Commit 0cd3930

Browse files
committed
testsuite.yml: test dist/ modules on threaded perls too
This originally also tested on Windows, but Storable fails a test on 5.24, and several other modules fail on 5.10 and 5.8.
1 parent f4a775e commit 0cd3930

File tree

1 file changed

+23
-6
lines changed

1 file changed

+23
-6
lines changed

.github/workflows/testsuite.yml

Lines changed: 23 additions & 6 deletions
Original file line numberDiff line numberDiff line change
@@ -795,9 +795,9 @@ jobs:
795795
# \__,_|_|___/\__| |_| |_| |_|\___/ \__,_|\__,_|_|\___||___/
796796
797797
dist-modules:
798-
name: dist-modules
798+
name: dist ${{ matrix.os.label }} ${{ matrix.perl-version }} ${{ matrix.threads.label }}
799799
needs: sanity_check
800-
runs-on: ubuntu-latest
800+
runs-on: ${{ matrix.os.id }}
801801
if: (! cancelled() && (needs.sanity_check.outputs.run_all_jobs == 'true' || needs.sanity_check.outputs.ci_force_dist_modules == 'true'))
802802
803803
env:
@@ -812,17 +812,34 @@ jobs:
812812
strategy:
813813
fail-fast: false
814814
matrix:
815+
# once some failures are resolved we may be able to add Windows to this
816+
os:
817+
- id: ubuntu-latest
818+
label: Linux
815819
perl-version:
816-
- 'latest'
820+
- '5.38'
817821
- '5.24'
818822
- '5.10'
819823
- '5.8'
820-
821-
container:
822-
image: perldocker/perl-tester:${{ matrix.perl-version }}
824+
threads:
825+
- enable: true
826+
label: -Dusethreads
827+
- enable: false
828+
label: -Uusethreads
823829
824830
steps:
825831
- uses: actions/checkout@v4
832+
- name: Set up perl
833+
uses: shogo82148/actions-setup-perl@v1.30.0
834+
with:
835+
perl-version: ${{ matrix.perl-version }}
836+
multi-thread: ${{ matrix.threads.id }}
837+
install-modules:
838+
ExtUtils::MakeMaker
839+
Perl::OSType
840+
Scalar::Util
841+
Socket
842+
version
826843
- name: perl -V
827844
run: perl -V
828845
- name: Build and test dist modules

0 commit comments

Comments
 (0)