Re: [Quilt-dev] Non-portable sed command in find_patch

[John Vandenberg]

On 9/22/05, Andreas Gruenbacher <address@hidden> wrote:
> On Wednesday 21 September 2005 22:11, John Vandenberg wrote:
> > On 9/22/05, Jean Delvare <address@hidden> wrote:
> > > Hi all,
> > >
> > > There is a non-portable sed command used in the find_patch function:
> > >
> > > sed -e "/^$bre\(\|\.patch\|\.diff\?\)\(\|\.gz\|\.bz2\)\([
> > > "$'\t'"]\|$\)/!d"
>
> Not ure what breaks. Does this sed support the construct sed -ne
> 's/aaa/bbb/p'? If so, then I'd prefer that.

Andreas, it was this part of the expression that caused the problem on OS X:

\([ "$'\t'"]\|$\)

I dont recall what it was exactly, but I think it was the dollar sign
embedded in the or.

I tested that "/bin/sed -ne 's/root/superuser/p' /etc/passwd" works on
Solaris; I wont be near OS X until next monday, but it should be
similar to Solaris.
--
John