[ovs-dev] [PATCH] testsuite: make a sed command more portable
Ben Pfaff
blp at nicira.com
Tue Jun 4 19:43:51 UTC 2013
On Tue, Jun 04, 2013 at 12:22:53PM -0700, Ben Pfaff wrote:
> On Tue, Jun 04, 2013 at 02:36:29PM -0400, Ed Maste wrote:
> > On 4 June 2013 13:01, Ben Pfaff <blp at nicira.com> wrote:
> > > I see that POSIX does not allow x\{,1\} (I had not realized that
> > > before), so change #2 one makes sense to me. But POSIX says that
> > > \{m,\} is allowed, so I don't see why change #1 is necessary. Does
> > > NetBSD not implement \{m,\}?
> >
> > FWIW, FreeBSD's sed accepts \{m,\} but not \{,n\}, so I'm happy to see
> > part 2 be committed at least.
>
> I'm happy to commit the whole thing. I only want to know whether I
> need to avoid \{m,\} elsewhere, since I didn't previously know of this
> issue. My usual shell utility compatibility reference, the Autoconf
> manual, doesn't mention this shortcoming.
I applied this to master, but I'd still like to know.
More information about the dev
mailing list