]> git.ipfire.org Git - thirdparty/git.git/commit
t4061: use POSIX compliant regex(7)
authorĐoàn Trần Công Danh <congdanhqx@gmail.com>
Wed, 25 Mar 2020 15:06:14 +0000 (22:06 +0700)
committerJunio C Hamano <gitster@pobox.com>
Wed, 25 Mar 2020 15:54:37 +0000 (08:54 -0700)
commit1f27522d8c651db5d254d352377051d7071edcdc
tree03ad5775223de974e071163b2a20c4cb9f465a71
parent274b9cc25322d9ee79aa8e6d4e86f0ffe5ced925
t4061: use POSIX compliant regex(7)

BRE interprets `+` literally, and
`\+` is undefined for POSIX BRE, from:
https://pubs.opengroup.org/onlinepubs/9699919799/basedefs/V1_chap09.html#tag_09_03_02

> The interpretation of an ordinary character preceded
> by an unescaped <backslash> ( '\\' ) is undefined, except for:
> - The characters ')', '(', '{', and '}'
> - The digits 1 to 9 inclusive
> - A character inside a bracket expression

This test is failing with busybox sed, the default sed of Alpine Linux

We have 2 options here:

- Using literal `+` because BRE will interpret it as-is, or
- Using character class `[+]` to defend against a sed that expects ERE

ERE-expected sed is theoretical at this point,
but we haven't found it, yet.
And, we may run into other problems with that sed.
Let's go with first option and fix it later if that sed could be found.

Signed-off-by: Đoàn Trần Công Danh <congdanhqx@gmail.com>
Signed-off-by: Junio C Hamano <gitster@pobox.com>
t/t4061-diff-indent.sh