tests: fix and add lint for non-portable seq
authorÆvar Arnfjörð Bjarmason <avarab@gmail.com>
Fri, 24 Aug 2018 15:20:12 +0000 (15:20 +0000)
committerJunio C Hamano <gitster@pobox.com>
Mon, 27 Aug 2018 18:31:18 +0000 (11:31 -0700)
commitb2fa7a2372f42af404a942a4691a5341af8868e9
tree5a7320926ea9d35dffce9a37d44ebbe981cfd4c3
parent2a59a6ef2066c786b69253e148cccdc92cfdb434
tests: fix and add lint for non-portable seq

The seq command is not in POSIX, and doesn't exist on
e.g. OpenBSD. We've had the test_seq wrapper since d17cf5f3a3 ("tests:
Introduce test_seq", 2012-08-04), but use of it keeps coming back,
e.g. in the recently added "fetch negotiator" tests being added here.

So let's also add a check to "make test-lint". The regex is aiming to
capture the likes of $(seq ..) and "seq" as a stand-alone command,
without capturing some existing cases where we e.g. have files called
"seq", as \bseq\b would do.

Signed-off-by: Ævar Arnfjörð Bjarmason <avarab@gmail.com>
Signed-off-by: Junio C Hamano <gitster@pobox.com>
t/check-non-portable-shell.pl
t/t5552-skipping-fetch-negotiator.sh
t/t5703-upload-pack-ref-in-want.sh