[HACKERS] pg_ctl init doc bug

2010-11-09 Thread Euler Taveira de Oliveira
Hi,

While executing the following command I got:

$ pg_ctl init -D /tmp/foo -o -N 11
/home/euler/pg/bin/initdb: invalid option -- N
Try initdb --help for more information.
pg_ctl: database system initialization failed

I tried -N 11 (postgres option) after looking at the manual but the -o
option only accept initdb options so I think there is a documentation bug.
Patch is attached.

While at it, is it worth adding a pg_ctl init example?


-- 
  Euler Taveira de Oliveira
  http://www.timbira.com/
diff --git a/doc/src/sgml/ref/pg_ctl-ref.sgml b/doc/src/sgml/ref/pg_ctl-ref.sgml
index 29bea24..0cc82cd 100644
*** a/doc/src/sgml/ref/pg_ctl-ref.sgml
--- b/doc/src/sgml/ref/pg_ctl-ref.sgml
*** PostgreSQL documentation
*** 25,31 
 arg choice=plaininit[db]/arg
 arg-s/arg
 arg-D replaceabledatadir/replaceable/arg
!arg-o replaceableoptions/replaceable/arg
/cmdsynopsis
  
cmdsynopsis
--- 25,31 
 arg choice=plaininit[db]/arg
 arg-s/arg
 arg-D replaceabledatadir/replaceable/arg
!arg-o replaceableinit-options/replaceable/arg
/cmdsynopsis
  
cmdsynopsis
*** PostgreSQL documentation
*** 263,268 
--- 263,282 
   /varlistentry
  
   varlistentry
+   termoption-o replaceable class=parameterinit-options/replaceable/option/term
+   listitem
+para
+ Specifies options to be passed directly to the
+ commandinitdb/command command.
+/para
+para
+ The options are usually surrounded by single or double
+ quotes to ensure that they are passed through as a group.
+/para
+   /listitem
+  /varlistentry
+ 
+  varlistentry
termoption-m replaceable class=parametermode/replaceable/option/term
listitem
 para

-- 
Sent via pgsql-hackers mailing list (pgsql-hackers@postgresql.org)
To make changes to your subscription:
http://www.postgresql.org/mailpref/pgsql-hackers


Re: [HACKERS] pg_ctl init doc bug

2010-11-09 Thread Tom Lane
Euler Taveira de Oliveira eu...@timbira.com writes:
 I tried -N 11 (postgres option) after looking at the manual but the -o
 option only accept initdb options so I think there is a documentation bug.
 Patch is attached.

Yeah, the init option was squeezed in later with only minimal attention
to documentation.  Agree with your idea of treating its -o option as
taking a different kind of argument.  Will apply.

regards, tom lane

-- 
Sent via pgsql-hackers mailing list (pgsql-hackers@postgresql.org)
To make changes to your subscription:
http://www.postgresql.org/mailpref/pgsql-hackers