check-non-portable-shell.pl: `wc -l` may have leading WS
authorTorsten Bögershausen <tboegi@web.de>
Thu, 21 Dec 2017 21:26:46 +0000 (22:26 +0100)
committerJunio C Hamano <gitster@pobox.com>
Fri, 22 Dec 2017 21:00:51 +0000 (13:00 -0800)
commit7dbe8c8003f3bec04f9dbbfff2b4bff172e52d9c
treeb80e0cd9c536982229c0535d8e6e9dc97042f63d
parent936d1b989416a95f593bf81ccae8ac62cd83f279
check-non-portable-shell.pl: `wc -l` may have leading WS

Test scripts count number of lines in an output and check it againt
its expectation.  fb3340a6 ("test-lib: introduce test_line_count to
measure files", 2010-10-31) introduced a helper to show a failure in
such a test in a more readable way than comparing `wc -l` output with
a number.

Besides, on some platforms, "$(wc -l <file)" is padded with leading
whitespace on the left, so

test "$(wc -l <file)" = 4

would not work (most notably on macosX); the users of test_line_count
helper would not suffer from such a portability glitch.

Add a check in check-non-portable-shell.pl to find '"' between
`wc -l` and '=' and hint the user about test_line_count().

Signed-off-by: Torsten Bögershausen <tboegi@web.de>
Reviewed-by: Johannes Schindelin <Johannes.Schindelin@gmx.de>
Helped-by: Eric Sunshine <sunshine@sunshineco.com>
Signed-off-by: Junio C Hamano <gitster@pobox.com>
t/check-non-portable-shell.pl