Message ID | 1446596356-11661-1-git-send-email-aatteka@nicira.com |
---|---|
State | Accepted |
Headers | show |
On Tue, Nov 03, 2015 at 04:19:16PM -0800, Ansis Atteka wrote: > It is very easy to misuse these macros, because when the COMMAND > returns exit code "0" it is actually considered as if condition > evaluated to "true" and not "false" as some might think. > > This patch ensures that this is clearly reflected in documentation. > > Signed-off-by: Ansis Atteka <aatteka@nicira.com> Acked-by: Ben Pfaff <blp@nicira.com>
On Tue, Nov 3, 2015 at 4:22 PM, Ben Pfaff <blp@nicira.com> wrote: > On Tue, Nov 03, 2015 at 04:19:16PM -0800, Ansis Atteka wrote: >> It is very easy to misuse these macros, because when the COMMAND >> returns exit code "0" it is actually considered as if condition >> evaluated to "true" and not "false" as some might think. >> >> This patch ensures that this is clearly reflected in documentation. >> >> Signed-off-by: Ansis Atteka <aatteka@nicira.com> > > Acked-by: Ben Pfaff <blp@nicira.com> Thanks, I pushed it.
diff --git a/tests/ovs-macros.at b/tests/ovs-macros.at index 541b042..9195f67 100644 --- a/tests/ovs-macros.at +++ b/tests/ovs-macros.at @@ -103,7 +103,21 @@ m4_define([OVS_WAIT], [ovs_wait_cond () { $1 } ovs_wait], [0], [ignore], [ignore], [$2])]) + +dnl OVS_WAIT_UNTIL(COMMAND) +dnl +dnl Executes shell COMMAND in a loop until it returns +dnl zero return code. If COMMAND did not return +dnl zero code within reasonable time limit, then +dnl the test fails. m4_define([OVS_WAIT_UNTIL], [OVS_WAIT([$1], [$2])]) + +dnl OVS_WAIT_WHILE(COMMAND) +dnl +dnl Executes shell COMMAND in a loop until it returns +dnl non-zero return code. If COMMAND did not return +dnl non-zero code within reasonable time limit, then +dnl the test fails. m4_define([OVS_WAIT_WHILE], [OVS_WAIT([if $1; then return 1; else return 0; fi], [$2])])
It is very easy to misuse these macros, because when the COMMAND returns exit code "0" it is actually considered as if condition evaluated to "true" and not "false" as some might think. This patch ensures that this is clearly reflected in documentation. Signed-off-by: Ansis Atteka <aatteka@nicira.com> --- tests/ovs-macros.at | 14 ++++++++++++++ 1 file changed, 14 insertions(+)