threads / discuss / 29862

t5704: problem with OS X sed

Subject: t5704: problem with OS X sed

## tl;dr

3 messages between Mar 6, 2012 and Mar 6, 2012.

replies: 2people: 2as markdown or json

Brian Gernhardt· Mar 6, 2012, 18:10 UTC · lore

t5704's newest test has always failed for my on OS X.  It's taken a little while for me to find the time to try to fix it.  Unfortunately, my sed-fu has proven not up to the task:

sed: 1: "/^-/{p;q}": extra characters at the end of q command not ok - 7 ridiculously long subject in boundary # # : >file4 && # test_tick && # git add file4 && # printf "%01200d # " 0 | git commit -F - && # test_commit fifth && # git bundle create long-subject-bundle.bdl HEAD^..HEAD && # git bundle list-heads long-subject-bundle.bdl >heads && # test -s heads && # git fetch long-subject-bundle.bdl && # sed -n "/^-/{p;q}" long-subject-bundle.bdl >boundary && # grep "^-$_x40 " boundary #

I've tried various bits of whitespace or splitting the commands in two, but haven't hit on the proper combination of commands.

Of course, if I remove the sed invocation, I get the following error from grep: "Regular expression too big"

It seems to me that using sed to find a single line and then grepping for it is somewhat redundant.

If someone else has suggestions on how to make this test work on OS X, I'm happy to help test them.

~~ Brian Gernhardt

Jakub Narebski· Mar 6, 2012, 21:01 UTC · re: Brian Gernhardt · lore

Re: t5704: problem with OS X sed

Brian Gernhardt <benji@silverinsanity.com> writes:

Show 30 quoted lines

> t5704's newest test has always failed for my on OS X.  It's taken a little while for me to find the time to try to fix it.  Unfortunately, my sed-fu has proven not up to the task:
> 
> sed: 1: "/^-/{p;q}": extra characters at the end of q command
> not ok - 7 ridiculously long subject in boundary
> #	
> #		: >file4 &&
> #		test_tick &&
> #		git add file4 &&
> #		printf "%01200d
> #	" 0 | git commit -F - &&
> #		test_commit fifth &&
> #		git bundle create long-subject-bundle.bdl HEAD^..HEAD &&
> #		git bundle list-heads long-subject-bundle.bdl >heads &&
> #		test -s heads &&
> #		git fetch long-subject-bundle.bdl &&
> #		sed -n "/^-/{p;q}" long-subject-bundle.bdl >boundary &&
> #		grep "^-$_x40 " boundary
> #	
> 
> I've tried various bits of whitespace or splitting the commands in
> two, but haven't hit on the proper combination of commands.
> 
> Of course, if I remove the sed invocation, I get the following error
> from grep: "Regular expression too big"
> 
> It seems to me that using sed to find a single line and then
> grepping for it is somewhat redundant.
> 
> If someone else has suggestions on how to make this test work on OS
> X, I'm happy to help test them.

Thomas Rast proposed a patch to fix this issue, as reported by BJ Hargrave, in

  Re: [ANNOUNCE] Git 1.7.9.3
  Message-ID: <87y5rd4wci.fsf@thomas.inf.ethz.ch>
  http://thread.gmane.org/gmane.linux.kernel/1262318/focus=192355

as

  [PATCH] t5704: fix nonportable sed/grep usages

HTH

-- 
Jakub Narebski

Brian Gernhardt· Mar 6, 2012, 21:18 UTC · re: Jakub Narebski · lore

Re: t5704: problem with OS X sed

On Mar 6, 2012, at 4:01 PM, Jakub Narebski wrote:

Show 10 quoted lines

> Thomas Rast proposed a patch to fix this issue, as reported by
> BJ Hargrave, in
> 
>  Re: [ANNOUNCE] Git 1.7.9.3
>  Message-ID: <87y5rd4wci.fsf@thomas.inf.ethz.ch>
>  http://thread.gmane.org/gmane.linux.kernel/1262318/focus=192355
> 
> as
> 
>  [PATCH] t5704: fix nonportable sed/grep usages

Ah, reported and fixed in an announce e-mail while I was working on it myself.

Unsurprisingly, the patch also works for me.

Thanks Thomas and Jakub, ~~ Brian G