Skip to content

Commit 180e10c

Browse files
tautschnigadpaco-awszhassan-aws
authored
Fix cbmc-update CI job (rust-lang#3156)
We had a spurious update attempt logged in rust-lang#3155 for the job prior to this fix would empty out the version strings. This was caused by use of undefined variables. Resolves: rust-lang#3155 By submitting this pull request, I confirm that my contribution is made under the terms of the Apache 2.0 and MIT licenses. Co-authored-by: Adrian Palacios <73246657+adpaco-aws@users.noreply.github.com> Co-authored-by: Zyad Hassan <88045115+zhassan-aws@users.noreply.github.com>
1 parent a4e69eb commit 180e10c

File tree

1 file changed

+3
-3
lines changed

1 file changed

+3
-3
lines changed

.github/workflows/cbmc-update.yml

Lines changed: 3 additions & 3 deletions
Original file line numberDiff line numberDiff line change
@@ -30,7 +30,7 @@ jobs:
3030
env:
3131
GH_TOKEN: ${{ github.token }}
3232
run: |
33-
grep ^CBMC_VERSION kani-dependencies >> $GITHUB_ENV
33+
grep ^CBMC_VERSION kani-dependencies | sed 's/"//g' >> $GITHUB_ENV
3434
CBMC_LATEST=$(gh -R diffblue/cbmc release list | grep Latest | awk '{print $1}' | cut -f2 -d-)
3535
echo "CBMC_LATEST=$CBMC_LATEST" >> $GITHUB_ENV
3636
# check whether the version has changed at all
@@ -47,8 +47,8 @@ jobs:
4747
elif ! git ls-remote --exit-code origin cbmc-$CBMC_LATEST ; then
4848
CBMC_LATEST_MAJOR=$(echo $CBMC_LATEST | cut -f1 -d.)
4949
CBMC_LATEST_MINOR=$(echo $CBMC_LATEST | cut -f2 -d.)
50-
sed -i "s/^CBMC_MAJOR=.*/CBMC_MAJOR=\"$CBMC_MAJOR\"/" kani-dependencies
51-
sed -i "s/^CBMC_MINOR=.*/CBMC_MINOR=\"$CBMC_MINOR\"/" kani-dependencies
50+
sed -i "s/^CBMC_MAJOR=.*/CBMC_MAJOR=\"$CBMC_LATEST_MAJOR\"/" kani-dependencies
51+
sed -i "s/^CBMC_MINOR=.*/CBMC_MINOR=\"$CBMC_LATEST_MINOR\"/" kani-dependencies
5252
sed -i "s/^CBMC_VERSION=.*/CBMC_VERSION=\"$CBMC_LATEST\"/" kani-dependencies
5353
git diff
5454
if ! ./scripts/kani-regression.sh ; then

0 commit comments

Comments
 (0)