[master] 169e162 Enforce that VCL names must be C-language identifiers ([A-Za-z][A-Za-z0-9_]*)

Dridi Boukelmoune dridi at varni.sh
Thu Sep 8 14:35:33 CEST 2016


On Thu, Sep 8, 2016 at 2:28 PM, Poul-Henning Kamp <phk at phk.freebsd.dk> wrote:
> --------
> In message <CAJV_h0ZrTPpdf0u0PnjS1Vb1gnxW0Auk7SGKuCReGwTEQW9Ajg at mail.gmail.com>
> , Federico Schwindt writes:
>
>>Isn't dash (-) usage quite widespread?
>>
>>I'd expect it to be the most common character outside digits and numbers.
>
> The problem is that VCL label names need to be C-idents, and I'm
> not terribly keen on having different namespace for VCL labels and
> "real" VCLs.
>
> The alternative would be to encode a freer form into C-idents, but that
> turns really ugly really fast...

I can fix packaging scripts once we reach consensus, but this probably
deserve a proper entry in upgrading.rst since this change may break
things downstream (like the varnish agent for instance).

Dridi



More information about the varnish-commit mailing list