Skip to content

Commit

Permalink
ci: fix .github/workflows/scribble_license.yml
Browse files Browse the repository at this point in the history
Since we're using the `'full'` Racket distribution, the `racket-index`
package will already be installed: we can't `raco pkg install` again.
Instead, we should `raco pkg update` to the potentially-changed version
to be tested. This should fix failures like
<https://github.com/racket/racket/actions/runs/3230196005/jobs/5288363302>:

    Run raco pkg install --auto -j $(nproc) pkgs/racket-index
      raco pkg install --auto -j $(nproc) pkgs/racket-index
      shell: /bin/bash -e {0}
    raco pkg install: package is currently installed in a wider scope
      package: racket-index
      installed scope: installation
      given scope: user
    Error: Process completed with exit code 1.

The revised workflow is still not fully robust against more complex
changes to `racket-index` and its dependencies, but it should serve for
making sure that `LICENSE.txt`, `racket/src/LICENSE.txt`, and
`pkgs/racket-index/scribblings/main/license.scrbl` stay in sync.

Related to racket#4460
  • Loading branch information
LiberalArtist committed Oct 12, 2022
1 parent 97ab306 commit 082d4e6
Showing 1 changed file with 2 additions and 2 deletions.
4 changes: 2 additions & 2 deletions .github/workflows/scribble_license.yml
Original file line number Diff line number Diff line change
Expand Up @@ -26,8 +26,8 @@ jobs:
distribution: 'full'
variant: 'CS'
version: 'current'
- name: Scribble License Files
run: raco pkg install --auto -j $(nproc) pkgs/racket-index
- name: Update racket-index
run: raco pkg update -j $(nproc) --batch --auto pkgs/racket-index

generation-check:
runs-on: ubuntu-18.04
Expand Down

0 comments on commit 082d4e6

Please sign in to comment.