utilities: don't use a bashism to check that datapath exists

[[ ]] 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 <martinxu9.ovs@gmail.com>
Fixes: 9763d17fbd05 ("utilities: check datapath exists before conntrack flush")
Signed-off-by: Timothy Redaelli <tredaelli@redhat.com>
Signed-off-by: Ben Pfaff <blp@ovn.org>
Acked-by: Aaron Conole <aconole@redhat.com>

diff --git a/utilities/ovs-lib.in b/utilities/ovs-lib.in
index f36bd0103e6edd7ad0124c86a07c73694b08432b..10c709792ae188ad5a82cfd56c5c0e1e093cfe71 100644 (file)
--- 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