[ovs-dev] [PATCH] tests: Fix PKIDIR checks in AT_SKIP.

Han Zhou zhouhan at gmail.com
Wed Jun 2 06:32:59 UTC 2021


On Tue, Jun 1, 2021 at 11:42 AM Ben Pfaff <blp at ovn.org> wrote:
>
> In Autotest, [xyz] just expands to xyz.  To get [xyz] in output, we
> need [[xyz]] in input.
>
> I spotted this based on "expr" reporting an error in testsuite output.
>
> Signed-off-by: Ben Pfaff <blp at ovn.org>
> ---
>  tests/ovsdb-server.at | 4 ++--
>  1 file changed, 2 insertions(+), 2 deletions(-)
>
> diff --git a/tests/ovsdb-server.at b/tests/ovsdb-server.at
> index 926abce3a359..0badafa3291d 100644
> --- a/tests/ovsdb-server.at
> +++ b/tests/ovsdb-server.at
> @@ -548,8 +548,8 @@ AT_SKIP_IF([test "$HAVE_OPENSSL" = no])
>  # msys on Windows does not convert the path style automatically.
>  # So, do that forcefully with a 'pwd -W' (called through pwd() function).
>  PKIDIR="$(cd $abs_top_builddir/tests && pwd)"
> -AT_SKIP_IF([expr "$PKIDIR" : ".*[       '\"
> -\\]"])
> +AT_SKIP_IF([expr "$PKIDIR" : ".*[[       '\"
> +\\]]"])
>  AT_DATA([schema],
>    [[{"name": "mydb",
>       "tables": {
> --
> 2.31.1
>
> _______________________________________________
> dev mailing list
> dev at openvswitch.org
> https://mail.openvswitch.org/mailman/listinfo/ovs-dev

LGTM for both patches with the same commit message (are they made the same
intentionally?)

Acked-by: Han Zhou <hzhou at ovn.org>


More information about the dev mailing list