[ovs-dev] [PATCH] testsuite: make a sed command more portable

Ben Pfaff blp at nicira.com
Tue Jun 4 19:22:53 UTC 2013


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.



More information about the dev mailing list