Set recovery_config_directory for EXEC_BACKEND.

simonat2ndQuadrant · simonat2ndQuadrant · commit 7a5a59d378e0 · 2013-03-27T16:35:38.000Z
Remove comment questioning whether this is necessary for DataDir.
From buildfarm failures on Windows.
diff --git a/src/backend/utils/misc/guc.c b/src/backend/utils/misc/guc.c
@@ -4187,9 +4187,7 @@ SelectConfigFiles(const char *userDoption, const char *progname)
 	 * Reflect the final DataDir value back into the data_directory GUC var.
 	 * (If you are wondering why we don't just make them a single variable,
 	 * it's because the EXEC_BACKEND case needs DataDir to be transmitted to
-	 * child backends specially.  XXX is that still true?  Given that we now
-	 * chdir to DataDir, EXEC_BACKEND can read the config file without knowing
-	 * DataDir in advance.)
+	 * child backends specially.
 	 */
 	SetConfigOption("data_directory", DataDir, PGC_POSTMASTER, PGC_S_OVERRIDE);
 
@@ -4205,6 +4203,11 @@ SelectConfigFiles(const char *userDoption, const char *progname)
 	else
 		SetRecoveryConfDir(DataDir);
 
+	/*
+	 * Reflect the final RecoveryConfDir value back into the GUC var, as above.
+	 */
+	SetConfigOption("recovery_config_directory", RecoveryConfDir, PGC_POSTMASTER, PGC_S_OVERRIDE);
+
 	/*
 	 * If timezone_abbreviations wasn't set in the configuration file, install
 	 * the default value.  We do it this way because we can't safely install a
