diff options
author | Eric Sunshine <sunshine@sunshineco.com> | 2018-07-13 01:52:04 -0400 |
---|---|---|
committer | Junio C Hamano <gitster@pobox.com> | 2018-07-16 14:55:01 -0700 |
commit | c433600593b1db15a9ba80136dc2096e192322b6 (patch) | |
tree | fbf1de56f0d5ce93d1816dee873b50d5791dd164 /t | |
parent | t/check-non-portable-shell: stop being so polite (diff) | |
download | tgif-c433600593b1db15a9ba80136dc2096e192322b6.tar.xz |
t/check-non-portable-shell: make error messages more compact
Error messages emitted by this linting script are long and noisy,
consisting of several sections:
<test-script>:<line#>: error: <explanation>: <failed-shell-text>
The line of failed shell text, usually coming from within a test body,
is often indented by one or two TABs, with the result that the actual
(important) text is separated from <explanation> by a good deal of empty
space. This can make for a difficult read, especially on typical
80-column terminals.
Make the messages more compact and perhaps a bit easier to digest by
folding out the leading whitespace from <failed-shell-text>.
Signed-off-by: Eric Sunshine <sunshine@sunshineco.com>
Signed-off-by: Junio C Hamano <gitster@pobox.com>
Diffstat (limited to 't')
-rwxr-xr-x | t/check-non-portable-shell.pl | 2 |
1 files changed, 2 insertions, 0 deletions
diff --git a/t/check-non-portable-shell.pl b/t/check-non-portable-shell.pl index 11028578ff..f6dbe28b19 100755 --- a/t/check-non-portable-shell.pl +++ b/t/check-non-portable-shell.pl @@ -10,6 +10,8 @@ my $exit_code=0; sub err { my $msg = shift; + s/^\s+//; + s/\s+$//; print "$ARGV:$.: error: $msg: $_\n"; $exit_code = 1; } |