Skip to content

Commit

Permalink
qa/new-grind: fixups
Browse files Browse the repository at this point in the history
- syntax error left behind in previous commit
- if $PCPQA_VALGRIND=both, run both the non-valgrind and the
  valgrind variants, even if valgrind is installed

The second change has NOT been retrofitted to older valgrind test
pairs, but is in the new 1912/1913 (in the next commit).
  • Loading branch information
kmcdonell committed Sep 11, 2023
1 parent 31f2807 commit d90e977
Showing 1 changed file with 3 additions and 2 deletions.
5 changes: 3 additions & 2 deletions qa/new-grind
Original file line number Diff line number Diff line change
Expand Up @@ -158,9 +158,10 @@ NF == 1 && $1 == "#" && done == 0 {
{ print }
/^ *do_valgrind=true/ { print "elif which valgrind >/dev/null 2>&1"
print "then"
print " _notrun \"valgrind variant qa/'$newseq' will be run\""
print " [ $PCPQA_VALGRIND = both ] || \\"
print " _notrun \"valgrind variant qa/'$newseq' will be run\""
next
}
}'
mv $oldseq.new-grind $oldseq
chmod 755 $oldseq
fi
Expand Down

0 comments on commit d90e977

Please sign in to comment.