Skip to content

Commit 5ac3102

Browse files
committed
Add missing apt-get update in CI test job
The CI `test` job's "Setup dependencies" step had `apt-get install` without having previously run `apt-get update`. This often does not work, and in a CI environment one should not typically expect it to work because package indexes in a virtual machine just provisioned from an image, if present at all, may be very outdated. But for some reason it had been working until recently, when breakages were observed, including in GitoxideLabs#1924 (though the breakage is not related to the changes in that PR). This runs `apt-get update` before `apt-get install` in that CI job, as had already been done in the other CI jobs and as had likely always been intended. This should make the "Setup dependencies" step work again.
1 parent 2ef2006 commit 5ac3102

File tree

1 file changed

+3
-1
lines changed

1 file changed

+3
-1
lines changed

.github/workflows/ci.yml

Lines changed: 3 additions & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -133,7 +133,9 @@ jobs:
133133
- uses: dtolnay/rust-toolchain@stable
134134
- uses: Swatinem/rust-cache@v2
135135
- name: Setup dependencies
136-
run: sudo apt-get install -y --no-install-recommends liblzma-dev
136+
run: |
137+
sudo apt-get update
138+
sudo apt-get install -y --no-install-recommends liblzma-dev
137139
- uses: extractions/setup-just@v3
138140
- uses: taiki-e/install-action@v2
139141
with:

0 commit comments

Comments
 (0)