Re: pgsql/src/backend/postmaster (postmaster.c)

"Hiroshi Inoue" <Inoue@tpf.co.jp> writes:
> Seems you have changed the position of ResetAllOptions() as follows.

> +    /*
> +     * Options setup
> +     */
>      if (getenv("PGDATA"))
>          DataDir = strdup(getenv("PGDATA")); /* default value */

>      if (getenv("PGPORT"))
>          PostPortName = atoi(getenv("PGPORT"));

> +    ResetAllOptions();

Hmm, actually I was trying to move it to after the memory-context
creation code, but I guess your point is that ResetAllOptions() is
ignoring these env-var-based settings.  Seems like the correct fix
is that guc.c ought to be handling this logic, not postmaster.c.
Peter, what do you think?

            regards, tom lane