"undefined" is the same as not having this option at all. Not needed for
body common control, but I wanted it to be just like the copy_from option,
where undefined makes sense (i.e. falls back to common control option)


On Thu, Mar 13, 2014 at 3:28 PM, Ted Zlatanov <notifications@github.com>wrote:

> In libpromises/mod_common.c:
>
> > @@ -258,7 +258,7 @@ static bool ClassesParseTreeCheck(const Promise *pp, Seq *errors)
> >      ConstraintSyntaxNewInt("syslog_port", CF_VALRANGE, "The port number of a UDP syslog service", SYNTAX_STATUS_NORMAL),
> >      ConstraintSyntaxNewBool("fips_mode", "Activate full FIPS mode restrictions. Default value: false", SYNTAX_STATUS_NORMAL),
> >      ConstraintSyntaxNewBool("cache_system_functions", "Cache the result of system functions. Default value: true", SYNTAX_STATUS_NORMAL),
> > -    ConstraintSyntaxNewString("protocol_version", "1,classic,2,latest", "CFEngine protocol version to use when connecting to the server. Default: latest", SYNTAX_STATUS_NORMAL),
> > +    ConstraintSyntaxNewOption("protocol_version", "0,undefined,1,classic,2,latest", "CFEngine protocol version to use when connecting to the server. Default: classic", SYNTAX_STATUS_NORMAL),
>
> undefined?!?!
>
> --
> Reply to this email directly or view it on GitHub<https://github.com/cfengine/core/pull/1551/files#r10566236>
> .
>