Skip to content

Commit

Permalink
work with distdir instead of tarball.
Browse files Browse the repository at this point in the history
Not sure what part of github workflow has changed, but having a
tarball in artifact (which itself is a zip-archive) would lead us to a
weird situation that the name of the directory instide the tarball
becomes the the as the the tarballs itself, and makes `tar xfz` always
fails.

I guess it can be bypassed by avoid making a tarball in the first place.
  • Loading branch information
gugod committed Jun 2, 2024
1 parent dd5a319 commit cfc910c
Showing 1 changed file with 10 additions and 10 deletions.
20 changes: 10 additions & 10 deletions .github/workflows/ci.yml
Original file line number Diff line number Diff line change
Expand Up @@ -10,12 +10,12 @@ jobs:
steps:
- uses: actions/checkout@v4
- run: cpanm --quiet --notest App::ModuleBuildTiny
- run: mbtiny dist
- run: echo ./App-perlbrew-*.tar.gz
- run: mbtiny distdir
- run: echo ./App-perlbrew-*
- uses: actions/upload-artifact@v4
with:
name: App-perlbrew-tarball
path: ./App-perlbrew-*.tar.gz
name: App-perlbrew-distdir
path: ./App-perlbrew-*
retention-days: 5

cpanm-dist:
Expand All @@ -31,9 +31,9 @@ jobs:
perl-version: ${{ matrix.version }}
- uses: actions/download-artifact@v4
with:
name: App-perlbrew-tarball
- run: echo ./App-perlbrew-*.tar.gz
- run: cpanm --verbose ./App-perlbrew-*.tar.gz
name: App-perlbrew-distdir
- run: echo ./App-perlbrew-*
- run: cpanm --verbose ./App-perlbrew-*

cpanm-dist-in-container:
needs: mbtiny-dist
Expand All @@ -47,6 +47,6 @@ jobs:
steps:
- uses: actions/download-artifact@v4
with:
name: App-perlbrew-tarball
- run: echo ./App-perlbrew-*.tar.gz
- run: cpanm --verbose ./App-perlbrew-*.tar.gz
name: App-perlbrew-distdir
- run: echo ./App-perlbrew-*
- run: cpanm --verbose ./App-perlbrew-*

0 comments on commit cfc910c

Please sign in to comment.