[PATCH] t4205: replace .\+ with ..* in sed commands

Brian Gernhardt· Jul 1, 2013, 18:59 UTC · lore

OS X's sed only accepts basic regular expressions, which does not allow the + quantifier. However '..*' (anything, followed by zero or more anything) is the same as '.\+' (one or more anything) and valid in any regular expression language.

Signed-off-by: Brian Gernhardt <brian@gernhardtsoftware.com>
---
 t/t4205-log-pretty-formats.sh | 4 ++--
 1 file changed, 2 insertions(+), 2 deletions(-)

Show changes to t/t4205-log-pretty-formats.sh +2 −2

diff --git a/t/t4205-log-pretty-formats.sh b/t/t4205-log-pretty-formats.sh
index 719d132..3cfb744 100755
--- a/t/t4205-log-pretty-formats.sh
+++ b/t/t4205-log-pretty-formats.sh
@@ -192,7 +192,7 @@ test_expect_success 'left alignment formatting with trunc' "
 message ..
 message ..
 add bar  Z
-$(commit_msg "" "8" ".\+$")
+$(commit_msg "" "8" "..*$")
 EOF
 	test_cmp expected actual
 "
@@ -310,7 +310,7 @@ test_expect_success 'left/right alignment formatting with stealing' "
 short long  long long
 message ..   A U Thor
 add bar      A U Thor
-$(commit_msg "" "8" ".\+$")   A U Thor
+$(commit_msg "" "8" "..*$")   A U Thor
 EOF
 	test_cmp expected actual
 "

-- 
1.8.3.1.636.g893104c

Junio C Hamano· Jul 1, 2013, 19:26 UTC · re: Brian Gernhardt · lore

Re: [PATCH] t4205: replace .\+ with ..* in sed commands

Brian Gernhardt <brian@gernhardtsoftware.com> writes:

> OS X's sed only accepts basic regular expressions, which does not
> allow the + quantifier.  However '..*' (anything, followed by zero or
> more anything) is the same as '.\+' (one or more anything) and valid
> in any regular expression language.

Thanks for spotting this.

We shouldn't mark this as "OS X's sed is broken", but as "We try to stick to POSIX BRE, and calling ERE elements via backslash escape, e.g. \+, is a GNU extension we try to avoid". Obviously we are not always careful and sometimes these slip through the review process.

Show 28 quoted lines

>
> Signed-off-by: Brian Gernhardt <brian@gernhardtsoftware.com>
> ---
>  t/t4205-log-pretty-formats.sh | 4 ++--
>  1 file changed, 2 insertions(+), 2 deletions(-)
>
> diff --git a/t/t4205-log-pretty-formats.sh b/t/t4205-log-pretty-formats.sh
> index 719d132..3cfb744 100755
> --- a/t/t4205-log-pretty-formats.sh
> +++ b/t/t4205-log-pretty-formats.sh
> @@ -192,7 +192,7 @@ test_expect_success 'left alignment formatting with trunc' "
>  message ..
>  message ..
>  add bar  Z
> -$(commit_msg "" "8" ".\+$")
> +$(commit_msg "" "8" "..*$")
>  EOF
>  	test_cmp expected actual
>  "
> @@ -310,7 +310,7 @@ test_expect_success 'left/right alignment formatting with stealing' "
>  short long  long long
>  message ..   A U Thor
>  add bar      A U Thor
> -$(commit_msg "" "8" ".\+$")   A U Thor
> +$(commit_msg "" "8" "..*$")   A U Thor
>  EOF
>  	test_cmp expected actual
>  "

patcht4205: replace .\+ with ..* in sed commands