doc patch - archive/restore_command on windows

ITAGAKI Takahiro <[email protected]>
Newsgroups gmane.comp.db.postgresql.devel.patches
Message-ID <[email protected]>
I found the examples of documentation about archive_command and
restore_command for Windows are incorrect or improper.

 - "copy" doesn't accept / (slash) as a path separator.
   We should use \\ (double backslash) for the purpose.
 - Windows path is typically starts with a drive letter (C:\\).
 - We'd better to quote a whole path, not only the last filename
   with double-quotes. It can work, but is not a windows manner.


Index: doc/src/sgml/backup.sgml
===================================================================
--- doc/src/sgml/backup.sgml	(HEAD)
+++ doc/src/sgml/backup.sgml	(working copy)
@@ -1122,7 +1122,7 @@
         when so asked.  Examples:
 <programlisting>
 restore_command = 'cp /mnt/server/archivedir/%f "%p"'
-restore_command = 'copy /mnt/server/archivedir/%f "%p"'  # Windows
+restore_command = 'copy "C:\\server\\archivedir/%f" "%p"'  # Windows
 </programlisting>
        </para>
       </listitem>
Index: doc/src/sgml/config.sgml
===================================================================
--- doc/src/sgml/config.sgml	(HEAD)
+++ doc/src/sgml/config.sgml	(working copy)
@@ -1698,7 +1698,7 @@
         and only if it succeeds.  Examples:
 <programlisting>
 archive_command = 'cp "%p" /mnt/server/archivedir/"%f"'
-archive_command = 'copy "%p" /mnt/server/archivedir/"%f"'  # Windows
+archive_command = 'copy "%p" "C:\\server\\archivedir\\%f"'  # Windows
 </programlisting>
        </para>
       </listitem>



Regards,
---
ITAGAKI Takahiro
NTT Open Source Software Center


-- 
Sent via pgsql-patches mailing list ([email protected])
To make changes to your subscription:
http://www.postgresql.org/mailpref/pgsql-patches
lmpx.com only provides a reader for public news (NNTP) servers. It is not affiliated with the servers or forums shown here and is not responsible for the content of articles, which is written by their respective authors.