]> granicus.if.org Git - postgresql/commit
Find postgresql.auto.conf in PGDATA even when postgresql.conf is elsewhere.
authorTom Lane <tgl@sss.pgh.pa.us>
Sun, 11 May 2014 19:13:30 +0000 (15:13 -0400)
committerTom Lane <tgl@sss.pgh.pa.us>
Sun, 11 May 2014 19:13:30 +0000 (15:13 -0400)
commit195e81aff5d63da1e1f9dd0314a13859ec15c7c0
treef86798f710ea33894e3acf76e22bf4220c223142
parent12e611d43e6efbf0e36014a3055ed47366facee7
Find postgresql.auto.conf in PGDATA even when postgresql.conf is elsewhere.

The original coding for ALTER SYSTEM made a fundamentally bogus assumption
that postgresql.auto.conf could be sought relative to the main config file
if we hadn't yet determined the value of data_directory.  This fails for
common arrangements with the config file elsewhere, as reported by
Christoph Berg.

The simplest fix is to not try to read postgresql.auto.conf until after
SelectConfigFiles has chosen (and locked down) the data_directory setting.

Because of the logic in ProcessConfigFile for handling resetting of GUCs
that've been removed from the config file, we cannot easily read the main
and auto config files separately; so this patch adopts a brute force
approach of reading the main config file twice during postmaster startup.
That's a tad ugly, but the actual time cost is likely to be negligible,
and there's no time for a more invasive redesign before beta.

With this patch, any attempt to set data_directory via ALTER SYSTEM
will be silently ignored.  It would probably be better to throw an
error, but that can be dealt with later.  This bug, however, would
prevent any testing of ALTER SYSTEM by a significant fraction of the
userbase, so it seems important to get it fixed before beta.
src/backend/utils/misc/guc-file.l
src/backend/utils/misc/guc.c