Skip to content

Commit

Permalink
scap: Install git as a workaround
Browse files Browse the repository at this point in the history
Sinon erroneously currently requires git to be installed: sinonjs/sinon#2557

However, sinon is a testing tool which should even be used anyways: mitre/saf#1919
  • Loading branch information
candrews committed Oct 18, 2023
1 parent caa6d2c commit 975df7c
Show file tree
Hide file tree
Showing 2 changed files with 10 additions and 0 deletions.
5 changes: 5 additions & 0 deletions .github/workflows/build.yml
Original file line number Diff line number Diff line change
Expand Up @@ -109,6 +109,11 @@ jobs:
# shellcheck shell=sh
set -eu
apk add curl docker jq openscap-docker npm gcompat unzip
# Install git to workaround https://github.com/sinonjs/sinon/issues/2557
# Remove when sinon 16.1.2 (or later) is released, or when using MITRE_SAF_VERSION that includes https://github.com/mitre/saf/pull/1919
apk add git
npm install -g "@microsoft/sarif-multitool@${MICROSOFT_SARIF_MULTITOOL_VERSION}"
npm install -g "@mitre/saf@${MITRE_SAF_VERSION}"
mkdir -p "${SSG_DIR}"
Expand Down
5 changes: 5 additions & 0 deletions .gitlab-ci.yml
Original file line number Diff line number Diff line change
Expand Up @@ -152,6 +152,11 @@ scap:
set -eu
printf "\e[0Ksection_start:%s:prerequisites[collapsed=true]\r\e[0KInstalling prerequisites...\n" "$(date +%s)"
apk add curl docker openscap-docker npm gcompat unzip
# Install git to workaround https://github.com/sinonjs/sinon/issues/2557
# Remove when sinon 16.1.2 (or later) is released, or when using MITRE_SAF_VERSION that includes https://github.com/mitre/saf/pull/1919
apk add git
npm install -g "@microsoft/sarif-multitool@${MICROSOFT_SARIF_MULTITOOL_VERSION}"
npm install -g "@mitre/saf@${MITRE_SAF_VERSION}"
mkdir ssg
Expand Down

0 comments on commit 975df7c

Please sign in to comment.