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

"Hiroshi Inoue" <Inoue(at)tpf(dot)co(dot)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