Remove -o/--oids from pg_dumpall

This has been forgotten in 578b229, which has removed support for WITH
OIDS.

Discussion: https://postgr.es/m/CALAY4q99FcFCoG6ddke0V-AksGe82L_+bhDWgEfgZBakB840zA@mail.gmail.com
Author: Surafel Temesgen

diff --git a/doc/src/sgml/ref/pg_dumpall.sgml b/doc/src/sgml/ref/pg_dumpall.sgml
index 24c8c031d64ac1190016bec7a994152827b4a44b..b35c702f99c5f364a54f22fa029c3f27f2d63112 100644 (file)
--- a/doc/src/sgml/ref/pg_dumpall.sgml
+++ b/doc/src/sgml/ref/pg_dumpall.sgml
@@ -130,20 +130,6 @@ PostgreSQL documentation
       </listitem>
      </varlistentry>
 
-     <varlistentry>
-      <term><option>-o</option></term>
-      <term><option>--oids</option></term>
-      <listitem>
-       <para>
-        Dump object identifiers (<acronym>OID</acronym>s) as part of the
-        data for every table.  Use this option if your application references
-        the <acronym>OID</acronym>
-        columns in some way (e.g., in a foreign key constraint).
-        Otherwise, this option should not be used.
-       </para>
-      </listitem>
-     </varlistentry>
-
      <varlistentry>
       <term><option>-O</option></term>
       <term><option>--no-owner</option></term>

diff --git a/src/bin/pg_dump/pg_dumpall.c b/src/bin/pg_dump/pg_dumpall.c
index 7623fc8b4ea56084dfdbb5f2ccd53ed65f2a9273..c8eb9de9fb0c286baefc4331d5f461da02c0229d 100644 (file)
--- a/src/bin/pg_dump/pg_dumpall.c
+++ b/src/bin/pg_dump/pg_dumpall.c
@@ -107,7 +107,6 @@ main(int argc, char *argv[])
                {"host", required_argument, NULL, 'h'},
                {"dbname", required_argument, NULL, 'd'},
                {"database", required_argument, NULL, 'l'},
-               {"oids", no_argument, NULL, 'o'},
                {"no-owner", no_argument, NULL, 'O'},
                {"port", required_argument, NULL, 'p'},
                {"roles-only", no_argument, NULL, 'r'},
@@ -211,7 +210,7 @@ main(int argc, char *argv[])
 
        pgdumpopts = createPQExpBuffer();
 
-       while ((c = getopt_long(argc, argv, "acd:E:f:gh:l:oOp:rsS:tU:vwWx", long_options, &optindex)) != -1)
+       while ((c = getopt_long(argc, argv, "acd:E:f:gh:l:Op:rsS:tU:vwWx", long_options, &optindex)) != -1)
        {
                switch (c)
                {
@@ -252,10 +251,6 @@ main(int argc, char *argv[])
                                pgdb = pg_strdup(optarg);
                                break;
 
-                       case 'o':
-                               appendPQExpBufferStr(pgdumpopts, " -o");
-                               break;
-
                        case 'O':
                                appendPQExpBufferStr(pgdumpopts, " -O");
                                break;
@@ -628,7 +623,6 @@ help(void)
        printf(_("  -c, --clean                  clean (drop) databases before recreating\n"));
        printf(_("  -E, --encoding=ENCODING      dump the data in encoding ENCODING\n"));
        printf(_("  -g, --globals-only           dump only global objects, no databases\n"));
-       printf(_("  -o, --oids                   include OIDs in dump\n"));
        printf(_("  -O, --no-owner               skip restoration of object ownership\n"));
        printf(_("  -r, --roles-only             dump only roles, no databases or tablespaces\n"));
        printf(_("  -s, --schema-only            dump only the schema, no data\n"));