Re: [PATCH v2 1/6] t4061: use POSIX compliance regex(7)
- From
Danh Doan <congdanhqx@gmail.com>
- Date
- Mar 22, 2020, 01:52 UTC
- Message-ID
- <20200322015233.GB11801@danh.dev>
- In-Reply-To
- <4830bd3aaf6cadc02c00732f8447646064a6ae3e.1584838133.git.congdanhqx@gmail.com>
On 2020-03-22 07:55:10+0700, Đoàn Trần Công Danh <congdanhqx@gmail.com> wrote:
Show 15 quoted lines
> 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
>
> Fix it by using literal `+` instead.
>
> Signed-off-by: Đoàn Trần Công Danh <congdanhqx@gmail.com>Due to error in my part, please ignore the v2 with 6 patches. It's the same with v1.
Please look into v2 with 8 patches instead.
Sorry for the noise.
-- Danh