Acked-by; Mark Michelson <mmich...@redhat.com>

On 11/17/23 23:28, Ihar Hrachyshka wrote:
While either check works, it's better to use a more explicit check.

Signed-off-by: Ihar Hrachyshka <ihrac...@redhat.com>
---
  tests/ovn-macros.at | 2 +-
  1 file changed, 1 insertion(+), 1 deletion(-)

diff --git a/tests/ovn-macros.at b/tests/ovn-macros.at
index 3191bb6ad..0b15bcc80 100644
--- a/tests/ovn-macros.at
+++ b/tests/ovn-macros.at
@@ -873,7 +873,7 @@ ovn_trace_client() {
  #
  fmt_pkt() {
      ctlfile=$ovs_base/scapy.ctl
-    if [[ ! -e $ctlfile ]]; then
+    if [[ ! -S $ctlfile ]]; then
          start_scapy_server
      fi
      ovs-appctl -t $ctlfile payload "$1"

_______________________________________________
dev mailing list
d...@openvswitch.org
https://mail.openvswitch.org/mailman/listinfo/ovs-dev

Reply via email to