Re: doc patch - archive/restore_command on windows - Mailing list pgsql-patches

From Bruce Momjian
Subject Re: doc patch - archive/restore_command on windows
Date
Msg-id 200807181733.m6IHXHs26224@momjian.us
Whole thread Raw
In response to doc patch - archive/restore_command on windows  (ITAGAKI Takahiro <itagaki.takahiro@oss.ntt.co.jp>)
List pgsql-patches
Patch applied to CVS HEAD and 8.3.X.  Not sure how this was overlooked
in the past.  Thanks.

---------------------------------------------------------------------------

ITAGAKI Takahiro wrote:
> 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 (pgsql-patches@postgresql.org)
> To make changes to your subscription:
> http://www.postgresql.org/mailpref/pgsql-patches

--
  Bruce Momjian  <bruce@momjian.us>        http://momjian.us
  EnterpriseDB                             http://enterprisedb.com

  + If your life is a hard drive, Christ can be your backup. +

pgsql-patches by date:

Previous
From: "Erik"
Date:
Subject: Re: [HACKERS] WITH RECUSIVE patches 0717
Next
From: Bruce Momjian
Date:
Subject: Re: doc patch - archive/restore_command on windows