@@ -4187,7 +4187,9 @@ SelectConfigFiles(const char *userDoption, const char *progname)
41874187 * Reflect the final DataDir value back into the data_directory GUC var.
41884188 * (If you are wondering why we don't just make them a single variable,
41894189 * it's because the EXEC_BACKEND case needs DataDir to be transmitted to
4190- * child backends specially.
4190+ * child backends specially. XXX is that still true? Given that we now
4191+ * chdir to DataDir, EXEC_BACKEND can read the config file without knowing
4192+ * DataDir in advance.)
41914193 */
41924194SetConfigOption ("data_directory" ,DataDir ,PGC_POSTMASTER ,PGC_S_OVERRIDE );
41934195
@@ -4203,11 +4205,6 @@ SelectConfigFiles(const char *userDoption, const char *progname)
42034205else
42044206SetRecoveryConfDir (DataDir );
42054207
4206- /*
4207- * Reflect the final RecoveryConfDir value back into the GUC var, as above.
4208- */
4209- SetConfigOption ("recovery_config_directory" ,RecoveryConfDir ,PGC_POSTMASTER ,PGC_S_OVERRIDE );
4210-
42114208/*
42124209 * If timezone_abbreviations wasn't set in the configuration file, install
42134210 * the default value. We do it this way because we can't safely install a