[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