[PATCH v2 7/8] t4124: fix test for non-compliant diff(1)
- From
Đoàn Trần Công Danh <congdanhqx@gmail.com>
- Date
- Mar 22, 2020, 00:55 UTC
- Message-ID
- <10f39c3d30d13e9141f081f985a0620954cc7493.1584838148.git.congdanhqx@gmail.com>
- In-Reply-To
- <cover.1584838148.git.congdanhqx@gmail.com>
POSIX's diff(1) requires output in normal diff format. However, busybox's diff's output is written in unified format.
POSIX requires no option for normal-diff format.
A hint in test-lib-functions::test_cmp said `diff -u` isn't available everywhere.
Workaround this problem by assuming `diff(1)` output is unified if we couldn't make anything from normal-diff format.
Signed-off-by: Đoàn Trần Công Danh <congdanhqx@gmail.com> --- t/t4124-apply-ws-rule.sh | 6 ++++++ 1 file changed, 6 insertions(+)
diff --git a/t/t4124-apply-ws-rule.sh b/t/t4124-apply-ws-rule.sh index 971a5a7512..075b1912be 100755 --- a/t/t4124-apply-ws-rule.sh +++ b/t/t4124-apply-ws-rule.sh @@ -52,6 +52,12 @@ test_fix () { # find touched lines $DIFF file target | sed -n -e "s/^> //p" >fixed + # busybox's diff(1) output unified format + if ! test -s fixed; then + $DIFF file target | + grep -v '^+++ target' | + sed -e "/^+/s/+//" >fixed + fi # the changed lines are all expected to change fixed_cnt=$(wc -l <fixed)
-- 2.26.0.rc2.310.g2932bb562d