[ovs-dev] [PATCH] ovs-vsctl-bashcomp: Make compatible with busybox "awk".

Ben Pfaff blp at ovn.org
Wed Aug 9 18:36:47 UTC 2017


On Fri, Aug 04, 2017 at 09:54:26AM -0400, Lance Richardson wrote:
> > From: "Ben Pfaff" <blp at ovn.org>
> > To: dev at openvswitch.org
> > Cc: "Ben Pfaff" <blp at ovn.org>, "Stuart Cardall" <developer at it-offshore.co.uk>
> > Sent: Friday, 14 July, 2017 12:42:54 AM
> > Subject: [ovs-dev] [PATCH] ovs-vsctl-bashcomp: Make compatible with busybox	"awk".
> > 
> > It seems that awk in busybox doesn't think that an empty string is part of
> > a larger string, but that GNU awk does.  This commit adds an extra test to
> > make _ovs_vsctl_check_startswith_string work either way.
> > 
> > This allows the following tests to pass with busybox awk:
> > 
> > vsctl bashcomp unit tests
> > 
> >   7: vsctl-bashcomp - basic verification             ok
> >   8: vsctl-bashcomp - argument completion            ok
> > 
> > Reported-by: Stuart Cardall <developer at it-offshore.co.uk>
> > Signed-off-by: Ben Pfaff <blp at ovn.org>
> > ---
> 
> Makes sense, verified that these test cases now pass in the Alpine
> environment with busybox awk.
> 
> Acked-by: Lance Richardson <lrichard at redhat.com>

Thanks for the review.  I applied this to master and branch-2.8.


More information about the dev mailing list