Viktor Dukhovni:
> On Sat, Sep 27, 2014 at 02:23:12PM -0400, Wietse Venema wrote:
>
> > > + if ((config_dir = safe_getenv(CONF_ENV_PATH)) != 0 &&
> > > + strcmp(config_dir, DEF_CONFIG_DIR) != 0) {
> >
> > I prefer not to lie in software. Thus, config_dir will show up in
> > "postconf -n" output when there is any override, regardless of
> > whether it comes from main.cf, MAIL_CONF, or -c options, even
> > if inherited from a multi-instane manager.
>
> In other respects, is the rest of the patch sound (correct and
> useful)? I am not advocating that the patch be adopted, just using
postconf does not suppress parameters based on string comparison.
Instead, suppression is based on the origin of their definition.
Making config_directory behavior different in this respect would
be a mistake.
Wietse
