[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