"dev" <dev-boun...@openvswitch.org> wrote on 07/05/2016 04:35:04 AM:

> From: YAMAMOTO Takashi <yamam...@ovn.org>
> To: dev@openvswitch.org
> Cc: YAMAMOTO Takashi <yamam...@ovn.org>
> Date: 07/05/2016 04:40 AM
> Subject: [ovs-dev] [PATCH] ovn.at: Use = instead of == for test
> Sent by: "dev" <dev-boun...@openvswitch.org>
>
> == is a GNU extension which might not be available.
>
> Signed-off-by: YAMAMOTO Takashi <yamam...@ovn.org>
> ---
>  tests/ovn.at | 4 ++--
>  1 file changed, 2 insertions(+), 2 deletions(-)
>
> diff --git a/tests/ovn.at b/tests/ovn.at
> index feb68d3..3436417 100644
> --- a/tests/ovn.at
> +++ b/tests/ovn.at
> @@ -839,7 +839,7 @@ for is in 1 2 3; do
>
>                  if test $d != $s && test $s != 11; then acl2=$d;
> else acl2=; fi
>                  if test $d != $s && test $d != 33; then acl3=$d;
> else acl3=; fi
> -                if test $d == $s || (test $js == 1 && test $d == 33);
then
> +                if test $d = $s || (test $js = 1 && test $d = 33); then
>                      # Source of 11, 21, or 31 and dest of 33 shouldbe
dropped
>                      # due to the 4th ACL that uses address_set(set1).
>                      acl4=
> @@ -1998,7 +1998,7 @@ test_arp() {
>      as $hv ovs-appctl netdev-dummy/receive vif$inport $request
>      #as $hv ovs-appctl ofproto/trace br-int in_port=$inport $request
>      if test $drop != 1; then
> -        if test X$reply_ha == X; then
> +        if test X$reply_ha = X; then
>              # Expect to receive the broadcast ARP on the other
> logical switch ports
>              # if no reply is expected.
>              local i j
> --
> 2.5.4 (Apple Git-61)

The above is pretty darn trivial and it does work, so...

Acked-By: Ryan Moats <rmo...@us.ibm.com>
_______________________________________________
dev mailing list
dev@openvswitch.org
http://openvswitch.org/mailman/listinfo/dev

Reply via email to