diff options
author | Bram <perl-rt@wizbit.be> | 2022-09-11 22:18:31 +0200 |
---|---|---|
committer | Karl Williamson <khw@cpan.org> | 2022-09-13 18:52:00 -0600 |
commit | 7201856e3a53d05224c96b82b7b61a85d7cce1a6 (patch) | |
tree | cc10f54fce976aa17893d2d6d9e2a4eb09d9c8f5 /.github/workflows | |
parent | 54bc25ac5df8c35baddf0c0873d8beea89c576f7 (diff) | |
download | perl-7201856e3a53d05224c96b82b7b61a85d7cce1a6.tar.gz |
testsuite.yml: Remove fetching of tags
The only test that need the git tags (afaik) is porting/cmp_version.t;
But the fetching of tags isn't really working for that test.
Basically the repo is cloned with:
$ git clone --depth 1 https://github.com/perl/perl5
$ git fetch --depth=1 origin +refs/tags/*:refs/tags/*
And then later the test uses:
$ git describe --abbrev=0 HEAD
and it expects to find a tag.
However doing that:
$ git describe --abbrev=0 HEAD
fatal: No tags can describe '6a011f13d7690dbe2e03ad7500756c983bcb1834'.
So it's not really working..
This can also be seen in the CI logs[^1], the output contains:
porting/cmp_version.t ..... skipped: Git found, but no Git tags found
There are only 3 CI configs that do run the porting/cmp_version.t test:
- 'Sanity: Linux -Dusethreads' - this is intended
- 'linux i386/ubuntu (-Dusedevel)'[^2]
- 'cygwin'[^3]
[^1]: CI logs of: 'linux (-Uusethreads)', 'linux (-Duseshrplib ...)',
'linux (-Duserelocatableinc)', 'linux (-Dcc='clang')', 'linux (-Dcc='g++')',
'install', 'macOS (catalina) xcode 12 (-Uusethreads)',
'macOS (catalina) xcode 12 (-Duseithreads -Duseshrplib)', 'Windows msvc142',
'Windows mingw64', 'ASAN (-Accflags=-DPURIFY)', 'ASAN (-DDEBUGGING)',
'ASAN (-Dusethreads)', 'PERL_UNICODE (-Uusethreads)',
'PERL_UNICODE (-Dusethreads ...')
[^2]: it uses 'actions/checkout@v1' to do the checkout which does a full clone
[^3]: it manually runs `git clone`, doing a full clone (with all history)
Diffstat (limited to '.github/workflows')
-rw-r--r-- | .github/workflows/testsuite.yml | 15 |
1 files changed, 5 insertions, 10 deletions
diff --git a/.github/workflows/testsuite.yml b/.github/workflows/testsuite.yml index 3e0970019e..24962a53e0 100644 --- a/.github/workflows/testsuite.yml +++ b/.github/workflows/testsuite.yml @@ -164,10 +164,9 @@ jobs: sudo apt-get update sudo apt-get install -y libgdbm-dev libdb-dev - uses: actions/checkout@v2 - - name: git cfg + fetch tags + - name: git cfg run: | git config diff.renameLimit 999999 - git fetch --depth=1 origin +refs/tags/*:refs/tags/* - name: Configure run: | ./Configure -des -Dusedevel ${{ matrix.CONFIGURE_ARGS }} -Dprefix="$HOME/perl-blead" -DDEBUGGING @@ -261,10 +260,9 @@ jobs: sudo apt-get update sudo apt-get install -y libgdbm-dev libdb-dev - uses: actions/checkout@v2 - - name: git cfg + fetch tags + - name: git cfg run: | git config diff.renameLimit 999999 - git fetch --depth=1 origin +refs/tags/*:refs/tags/* - name: Configure run: | ./Configure -des -Dusedevel -Duseshrplib -Dusesitecustomize -Duselongdouble -Dprefix="$HOME/perl-blead" -DDEBUGGING @@ -523,10 +521,9 @@ jobs: sudo apt-get update sudo apt-get install -y libgdbm-dev libdb-dev - uses: actions/checkout@v2 - - name: git cfg + fetch tags + - name: git cfg run: | git config diff.renameLimit 999999 - git fetch --depth=1 origin +refs/tags/*:refs/tags/* - name: manicheck run: | perl Porting/manicheck --exitstatus @@ -575,10 +572,9 @@ jobs: sudo apt-get update sudo apt-get install -y libgdbm-dev libdb-dev - uses: actions/checkout@v2 - - name: git cfg + fetch tags + - name: git cfg run: | git config diff.renameLimit 999999 - git fetch --depth=1 origin +refs/tags/*:refs/tags/* - name: Configure run: | ./Configure -des -Dusedevel -Dcc="clang -fsanitize=address" -Dld="clang -fsanitize=address" ${{ matrix.CONFIGURE_ARGS }} -Dprefix="$HOME/perl-blead" @@ -624,10 +620,9 @@ jobs: sudo apt-get update sudo apt-get install -y libgdbm-dev libdb-dev - uses: actions/checkout@v2 - - name: git cfg + fetch tags + - name: git cfg run: | git config diff.renameLimit 999999 - git fetch --depth=1 origin +refs/tags/*:refs/tags/* - name: Show Locales run: locale -a - name: Configure |