Re: [PATCH] Allow combined diff to ignore white-spaces
Johannes Sixt <j.sixt@viscovery.net> writes:
Show 8 quoted lines
> This form of 'echo' is not sufficiently portable. How about:
>
> tr -d Q <<-\EOF >test &&
>
> always coalesce
> eol space coalesce Q
> ...
> EOF
Show 19 quoted lines
>> +test_expect_success 'check combined output (no ignore space)' '
>> + git show | test_i18ngrep "^-\s*eol spaces" &&
>> ...
>> + git show | test_i18ngrep "^--\s*always coalesce"
>
> This loses the exit code of git show. We usually write this as
>
> git show >actual &&
> grep "^- *eol spaces" &&
> grep "^- *eol space coalesce" &&
> ...
>
> (Same for later tests.)
>
> There is nothing i18n-ish in the test patterns. Use regular grep.
>
> BTW, there is compare_diff_patch() in diff-lib.sh. You can use it to
> compare diff output to expected output. Then you do not need a grep
> invocation for each line of the test file.
All good suggestions. Thanks.