Added to TODO: o Allow recovery.conf to allow the same syntax as postgresql.conf, including quoting
http://archives.postgresql.org/pgsql-hackers/2006-12/msg00497.php --------------------------------------------------------------------------- Simon Riggs wrote: > On Wed, 2006-12-13 at 19:28 +0000, Simon Riggs wrote: > > On Wed, 2006-12-13 at 04:23 +0000, Andrew - Supernews wrote: > > > While testing a PITR recovery, I discovered that recovery.conf doesn't > > > seem to allow specifying ' in the command string, making it hard to > > > protect the restore_command against problematic filenames (whitespace > > > etc.). This doesn't seem to be a problem for archive_command, which > > > allows \' (e.g. archive_command = '/path/to/script \'%f\' \'%p\''). > > Immediate workaround is to use a script to encapsulate the actual > desired request. > > > > Should this be fixed? > > > > Yes, I'll look into that. > > OK, I would propose to extend the guc-file.l to include sufficient code > to allow the parsing of the conf files to be identical between the > postgresql.conf and the recovery.conf (it isn't the same yet). > > There'll be care taken to ensure that the various options are not > settable in the wrong file. > > Any requests for specific implementation details? I'll be looking to > remove code from xlog.c, if possible. > > Implementation won't be immediate because of other current work. > > -- > Simon Riggs > EnterpriseDB http://www.enterprisedb.com > > > > ---------------------------(end of broadcast)--------------------------- > TIP 6: explain analyze is your friend -- Bruce Momjian [EMAIL PROTECTED] EnterpriseDB http://www.enterprisedb.com + If your life is a hard drive, Christ can be your backup. + ---------------------------(end of broadcast)--------------------------- TIP 3: Have you checked our extensive FAQ? http://www.postgresql.org/docs/faq