Skip to content

Commit f3cfc3b

Browse files
committed
test: rename $satisfied to $satisfied_prereq
All other shell variables that are used to globally keep track of states related to prerequisite have "prereq" somewhere in their names. Be consistent and avoid potential name crashes with other kinds of satisfaction in the future. Signed-off-by: Junio C Hamano <[email protected]>
1 parent cdd159b commit f3cfc3b

File tree

1 file changed

+3
-3
lines changed

1 file changed

+3
-3
lines changed

t/test-lib-functions.sh

Lines changed: 3 additions & 3 deletions
Original file line numberDiff line numberDiff line change
@@ -221,9 +221,9 @@ write_script () {
221221
# capital letters by convention).
222222

223223
test_set_prereq () {
224-
satisfied="$satisfied$1 "
224+
satisfied_prereq="$satisfied_prereq$1 "
225225
}
226-
satisfied=" "
226+
satisfied_prereq=" "
227227

228228
test_have_prereq () {
229229
# prerequisites can be concatenated with ','
@@ -239,7 +239,7 @@ test_have_prereq () {
239239
for prerequisite
240240
do
241241
total_prereq=$(($total_prereq + 1))
242-
case $satisfied in
242+
case "$satisfied_prereq" in
243243
*" $prerequisite "*)
244244
ok_prereq=$(($ok_prereq + 1))
245245
;;

0 commit comments

Comments
 (0)