[master] 21d3e8e85 test vmod_path for -C

Dridi Boukelmoune dridi at varni.sh
Wed Jan 15 15:53:37 UTC 2020


On Mon, Jan 13, 2020 at 3:47 PM Nils Goroll <nils.goroll at uplex.de> wrote:
>
>
> commit 21d3e8e8556b0e29fb803d1b4b37089b38a279fc
> Author: Nils Goroll <nils.goroll at uplex.de>
> Date:   Mon Jan 13 16:45:59 2020 +0100
>
>     test vmod_path for -C
>
> diff --git a/bin/varnishtest/tests/m00003.vtc b/bin/varnishtest/tests/m00003.vtc
> index 4e5a60ff4..ff9f514ce 100644
> --- a/bin/varnishtest/tests/m00003.vtc
> +++ b/bin/varnishtest/tests/m00003.vtc
> @@ -7,6 +7,15 @@ server s1 {
>         txresp
>  } -start
>
> +shell {
> +       echo "vcl 4.1; import std; backend dummy None;" >${tmpdir}/test.vcl
> +       varnishd -pvmod_path=${topbuild}/lib/libvmod_std/.libs -C -f ${tmpdir}/test.vcl 2>/dev/null
> +}
> +
> +shell -err -expect {Could not find VMOD std} {
> +       varnishd -C -f ${tmpdir}/test.vcl 2>&1
> +}
> +

I'm going to remove the second shell introduced by this patch, because
in the absence of a vmod_path change it might find one and on my
system it does, failing with an ABI mismatch instead.

>  varnish v1 -arg "-pvmod_path=${topbuild}/lib/libvmod_std/.libs/" \
>      -vcl+backend {
>         import std;
> _______________________________________________
> varnish-commit mailing list
> varnish-commit at varnish-cache.org
> https://www.varnish-cache.org/lists/mailman/listinfo/varnish-commit


More information about the varnish-commit mailing list