Thanks for fixing this. Martin
On Wed, Aug 1, 2018 at 8:48 AM, Timothy Redaelli <[email protected]> wrote: > [[ ]] syntax is not supported, at least, by dash that Debian, Ubuntu and > other > linux distributions may use instead of bash. > > This commit uses, instead, a POSIX way that is compatible with any POSIX > shell (bash, dash, busybox sh, etc). > > CC: Martin Xu <[email protected]> > Fixes: 9763d17fbd05 ("utilities: check datapath exists before conntrack > flush") > > Signed-off-by: Timothy Redaelli <[email protected]> > --- > utilities/ovs-lib.in | 2 +- > 1 file changed, 1 insertion(+), 1 deletion(-) > > diff --git a/utilities/ovs-lib.in b/utilities/ovs-lib.in > index d6ef77b6e..c3b76ec94 100644 > --- a/utilities/ovs-lib.in > +++ b/utilities/ovs-lib.in > @@ -605,7 +605,7 @@ force_reload_kmod () { > stop_ovsdb > start_ovsdb || return 1 > > - if [[ $(ovs-dpctl show) ]]; then > + if [ -n "$(ovs-dpctl show)" ]; then > action "Flush old conntrack entries" ovs-appctl > dpctl/flush-conntrack > fi > stop_forwarding > -- > 2.17.1 > > _______________________________________________ dev mailing list [email protected] https://mail.openvswitch.org/mailman/listinfo/ovs-dev
