pg_ctl, pg_upgrade: allow multiple -o/-O options, append them

Report by Pavel Raiskup

diff --git a/contrib/pg_upgrade/option.c b/contrib/pg_upgrade/option.c
index bb594dd2f6938c575c776514017ec1ed3d05cdee..cfc88ec03be63daf30480e0fb6e08ed20fa426a3 100644 (file)
--- a/contrib/pg_upgrade/option.c
+++ b/contrib/pg_upgrade/option.c
@@ -137,17 +137,35 @@ parseCommandLine(int argc, char *argv[])
                break;
 
            case 'o':
-               old_cluster.pgopts = pg_strdup(optarg);
+               /* append option? */
+               if (!old_cluster.pgopts)
+                   old_cluster.pgopts = pg_strdup(optarg);
+               else
+               {
+                   char *old_pgopts = old_cluster.pgopts;
+
+                   old_cluster.pgopts = psprintf("%s %s", old_pgopts, optarg);
+                   free(old_pgopts);
+               }
                break;
 
            case 'O':
-               new_cluster.pgopts = pg_strdup(optarg);
+               /* append option? */
+               if (!new_cluster.pgopts)
+                   new_cluster.pgopts = pg_strdup(optarg);
+               else
+               {
+                   char *new_pgopts = new_cluster.pgopts;
+
+                   new_cluster.pgopts = psprintf("%s %s", new_pgopts, optarg);
+                   free(new_pgopts);
+               }
                break;
 
                /*
                 * Someday, the port number option could be removed and passed
                 * using -o/-O, but that requires postmaster -C to be
-                * supported on all old/new versions.
+                * supported on all old/new versions (added in PG 9.2).
                 */
            case 'p':
                if ((old_cluster.port = atoi(optarg)) <= 0)

diff --git a/doc/src/sgml/pgupgrade.sgml b/doc/src/sgml/pgupgrade.sgml
index b79f0db7de4d862880cad16440ff9bb00a6bba97..dd57c5ca5ad43fe3975cac3baef27d419c4d1191 100644 (file)
--- a/doc/src/sgml/pgupgrade.sgml
+++ b/doc/src/sgml/pgupgrade.sgml
@@ -130,14 +130,16 @@
       -o options
       --old-options options
       options to be passed directly to the
-      old postgres command
+      old postgres command;  multiple
+      option invocations are appended
      
 
      
       -O options
       --new-options options
       options to be passed directly to the
-      new postgres command
+      new postgres command;  multiple
+      option invocations are appended
      
 
      

diff --git a/doc/src/sgml/ref/pg_ctl-ref.sgml b/doc/src/sgml/ref/pg_ctl-ref.sgml
index 23681294b87d767f5d085936ac482bec743c26d5..29f882bd76e2dfe21257809d80a4324edb98928d 100644 (file)
--- a/doc/src/sgml/ref/pg_ctl-ref.sgml
+++ b/doc/src/sgml/ref/pg_ctl-ref.sgml
@@ -302,7 +302,8 @@ PostgreSQL documentation
       
        
         Specifies options to be passed directly to the
-        postgres command.
+        postgres command;  multiple
+        option invocations are appended.
        
        
         The options should usually be surrounded by single or double

diff --git a/doc/src/sgml/ref/postgres-ref.sgml b/doc/src/sgml/ref/postgres-ref.sgml
index cdfdaa0b397ade839cf9274403f7002e5659ff0a..845d96940b045c8bce562862b1f5edf5d99368be 100644 (file)
--- a/doc/src/sgml/ref/postgres-ref.sgml
+++ b/doc/src/sgml/ref/postgres-ref.sgml
@@ -288,7 +288,8 @@ PostgreSQL documentation
         class="parameter">extra-options are passed to
         all server processes started by this
         postgres process.  If the option string contains
-        any spaces, the entire string must be quoted.
+        any spaces, the entire string must be quoted;  multiple
+        option invocations are appended.
        
 
        

diff --git a/src/bin/pg_ctl/pg_ctl.c b/src/bin/pg_ctl/pg_ctl.c
index ad7f36cdd2bed355848507607cb9758445287a01..a46ca53ba6e16b3916a63b6225077694ee84563c 100644 (file)
--- a/src/bin/pg_ctl/pg_ctl.c
+++ b/src/bin/pg_ctl/pg_ctl.c
@@ -2184,7 +2184,16 @@ main(int argc, char **argv)
                    register_servicename = pg_strdup(optarg);
                    break;
                case 'o':
-                   post_opts = pg_strdup(optarg);
+                   /* append option? */
+                   if (!post_opts)
+                       post_opts = pg_strdup(optarg);
+                   else
+                   {
+                       char *old_post_opts = post_opts;
+
+                       post_opts = psprintf("%s %s", old_post_opts, optarg);
+                       free(old_post_opts);
+                   }
                    break;
                case 'p':
                    exec_path = pg_strdup(optarg);