# t5704: problem with OS X sed 3 messages from 2012-03-06 to 2012-03-06. Participants: Brian Gernhardt, Jakub Narebski. Thread: https://gitlist.dev/t/29862 ## Brian Gernhardt, 2012-03-06 18:10 Subject: t5704: problem with OS X sed Message-ID: URL: https://gitlist.dev/e/E40E6E02-8960-4A10-9E71-28F9CDB120F8%40silverinsanity.com ``` 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, 2012-03-06 21:01 Subject: Re: t5704: problem with OS X sed Message-ID: URL: https://gitlist.dev/e/m362ehtpgp.fsf%40localhost.localdomain In-Reply-To: ``` Brian Gernhardt writes: > 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, 2012-03-06 21:18 Subject: Re: t5704: problem with OS X sed Message-ID: <9980BCE6-7933-40A5-BA96-5B93B553E666@silverinsanity.com> URL: https://gitlist.dev/e/9980BCE6-7933-40A5-BA96-5B93B553E666%40silverinsanity.com In-Reply-To: ``` On Mar 6, 2012, at 4:01 PM, Jakub Narebski wrote: > 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 ```