ci/install-dependencies: handle "sparse" job package installs
This just matches the style/location of the package installation for other jobs. There should be no functional change. I did flip the order of the options and command-name ("-y update" instead of "update -y") for consistency with other lines in the same file. Note also that we have to reorder the dependency install with the "checkout" action, so that we actually have the "ci" scripts available. Signed-off-by: Jeff King <peff@peff.net> Acked-by: Johannes Schindelin <johannes.schindelin@gmx.de> Signed-off-by: Junio C Hamano <gitster@pobox.com>
Jeff King committed
Jul 26, 2021 at 14:22 UTC
27f45ccf336d70e9078075eb963fb92541da8690
2 files changed
+7
-4
.github/workflows/main.yml
+2
-4
@@ -366,11 +366,9 @@ jobs:
366
artifact: sparse-20.04
367
- name: Install the current `sparse` package
368
run: sudo dpkg -i sparse-20.04/sparse_*.deb
369
- - name: Install other dependencies
370
- run: |
371
- sudo apt-get update -q &&
372
- sudo apt-get install -q -y libssl-dev libcurl4-openssl-dev libexpat-dev gettext zlib1g-dev
369
- uses: actions/checkout@v2
370
+ - name: Install other dependencies
371
+ run: ci/install-dependencies.sh
372
- run: make sparse
373
documentation:
374
needs: ci-config
ci/install-dependencies.sh
+5
@@ -65,6 +65,11 @@ StaticAnalysis)
65
sudo apt-get -q -y install coccinelle libcurl4-openssl-dev libssl-dev \
66
libexpat-dev gettext make
67
;;
68
+sparse)
69
+ sudo apt-get -q update -q
70
+ sudo apt-get -q -y install libssl-dev libcurl4-openssl-dev \
71
+ libexpat-dev gettext zlib1g-dev
72
+ ;;
73
Documentation)
74
sudo apt-get -q update
75
sudo apt-get -q -y install asciidoc xmlto docbook-xsl-ns make