"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