Make pg_dump dump ACL's by default, print warning on use of -z, and add

new -x option to skip acl dump.

diff --git a/doc/src/sgml/install.sgml b/doc/src/sgml/install.sgml
index 907bbfe0352c26a4a2f4c902542a9cf5c002b6bf..9d6d8a2cc68a06acccaf7fb1328b9b450f986cf7 100644 (file)
--- a/doc/src/sgml/install.sgml
+++ b/doc/src/sgml/install.sgml
@@ -259,7 +259,7 @@ script from v6.0 or everything
 To dump your fairly recent post-v6.0 database installation, type
 
 
-$ pg_dumpall -z > db.out
+$ pg_dumpall > db.out
 
 
 
@@ -273,7 +273,7 @@ $ cd
 $ gunzip -c postgresql-v6.5.tar.gz \
     | tar xvf - src/bin/pg_dump/pg_dumpall
 $ chmod a+x src/bin/pg_dump/pg_dumpall
-$ src/bin/pg_dump/pg_dumpall -z > db.out
+$ src/bin/pg_dump/pg_dumpall > db.out
 $ rm -rf src
 
 

diff --git a/doc/src/sgml/ref/pg_dump.sgml b/doc/src/sgml/ref/pg_dump.sgml
index 8fe0bf478973db7db6d9752340b1ffb4da1529a5..2345d34c84105010984b5c6f838059275d9cbca4 100644 (file)
--- a/doc/src/sgml/ref/pg_dump.sgml
+++ b/doc/src/sgml/ref/pg_dump.sgml
@@ -22,7 +22,7 @@ pg_dump [ dbname ]
 pg_dump [ -h host ] [ -p port ]
     [ -t table ]
     [ -f outputfile ]
-    [ -a ] [ -c ] [ -d ] [ -D ] [ -n ] [ -N ] [ -o ] [ -s ] [ -u ] [ -v ] [ -z ]
+    [ -a ] [ -c ] [ -d ] [ -D ] [ -n ] [ -N ] [ -o ] [ -s ] [ -u ] [ -v ] [ -x ]
     [ dbname ]
 
 
@@ -190,11 +190,11 @@ pg_dump [ -h host ] [ -p 
      
      
        
-         -z
+         -x
        
        
          
-       Include ACLs (grant/revoke commands) and table ownership information.
+       Prevent dumping of ACLs (grant/revoke commands) and table ownership information.
          
        
      

diff --git a/doc/src/sgml/ref/pg_dumpall.sgml b/doc/src/sgml/ref/pg_dumpall.sgml
index 3be7440720c5bc792afd1c47f0975cb68d3b2cfa..9447ff8942623745e9c6fc15d7a9c46a5c0a163c 100644 (file)
--- a/doc/src/sgml/ref/pg_dumpall.sgml
+++ b/doc/src/sgml/ref/pg_dumpall.sgml
@@ -20,7 +20,7 @@ Extract all Postgres databases into a script file
 
 pg_dumpall
 pg_dumpall [ -h host ] [ -p port ]
-    [ -a ] [ -d ] [ -D ] [ -o ] [ -s ] [ -u ] [ -v ] [ -z ]
+    [ -a ] [ -d ] [ -D ] [ -o ] [ -s ] [ -u ] [ -v ] [ -x ]
 
 
     
@@ -125,11 +125,11 @@ pg_dumpall [ -h host ] [ -p 
      
      
        
-         -z
+         -x
        
        
          
-       Include ACLs (grant/revoke commands) and table ownership information.
+       Prevent dumping ACLs (grant/revoke commands) and table ownership information.
          
        
      

diff --git a/doc/src/sgml/ref/pg_upgrade.sgml b/doc/src/sgml/ref/pg_upgrade.sgml
index 2f1a0c87f797d7b7d7019d7af22b9729ec5e84a3..7ee84fa0d3cbc1898a6857b9b80eaf76e2f3e474 100644 (file)
--- a/doc/src/sgml/ref/pg_upgrade.sgml
+++ b/doc/src/sgml/ref/pg_upgrade.sgml
@@ -34,7 +34,7 @@ Description
  PostgreSQL release without reloading all the data.  First,
  to be safe, back up your data directory.  Then, use:
 
-% pg_dumpall -s -z >db.out
+% pg_dumpall -s >db.out
 
 to dump out your old  database definitions without any
 data.  Stop the postmaster and all backends.

diff --git a/src/bin/pg_dump/common.c b/src/bin/pg_dump/common.c
index cf5bf3d0b1eea15cba4e68f642c714d971c32211..d0737b4b114bede376fc4fec0ca59961a681a809 100644 (file)
--- a/src/bin/pg_dump/common.c
+++ b/src/bin/pg_dump/common.c
@@ -7,7 +7,7 @@
  *
  *
  * IDENTIFICATION
- *   $Header: /cvsroot/pgsql/src/bin/pg_dump/common.c,v 1.31 1999/05/26 21:51:13 tgl Exp $
+ *   $Header: /cvsroot/pgsql/src/bin/pg_dump/common.c,v 1.32 1999/05/27 16:29:03 momjian Exp $
  *
  * Modifications - 6/12/96 - [email protected] - version 1.13.dhb.2
  *
@@ -232,7 +232,7 @@ TableInfo  *
 dumpSchema(FILE *fout,
           int *numTablesPtr,
           const char *tablename,
-          const bool acls)
+          const bool aclsSkip)
 {
    int         numTypes;
    int         numFuncs;
@@ -301,7 +301,7 @@ dumpSchema(FILE *fout,
            fprintf(stderr, "%s dumping out tables %s\n",
                    g_comment_start, g_comment_end);
        dumpTables(fout, tblinfo, numTables, inhinfo, numInherits,
-                  tinfo, numTypes, tablename, acls);
+                  tinfo, numTypes, tablename, aclsSkip);
    }
 
    if (!tablename && fout)

diff --git a/src/bin/pg_dump/pg_dump.c b/src/bin/pg_dump/pg_dump.c
index e94faf35b160414b63ec07f0bc5cc35dded3d73c..12cf2d63afa50e108dfad90bec1a7364f2804e6f 100644 (file)
--- a/src/bin/pg_dump/pg_dump.c
+++ b/src/bin/pg_dump/pg_dump.c
@@ -21,7 +21,7 @@
  *
  *
  * IDENTIFICATION
- *   $Header: /cvsroot/pgsql/src/bin/pg_dump/pg_dump.c,v 1.112 1999/05/26 21:51:12 tgl Exp $
+ *   $Header: /cvsroot/pgsql/src/bin/pg_dump/pg_dump.c,v 1.113 1999/05/27 16:29:03 momjian Exp $
  *
  * Modifications - 6/10/96 - [email protected] - version 1.13.dhb
  *
@@ -116,7 +116,7 @@ bool        dumpData;           /* dump data using proper insert strings */
 bool       attrNames;          /* put attr names into insert strings */
 bool       schemaOnly;
 bool       dataOnly;
-bool       aclsOption;
+bool       aclsSkip;
 bool       dropSchema;
 
 char       g_opaque_type[10];  /* name for the opaque type */
@@ -549,7 +549,7 @@ main(int argc, char **argv)
    char        tmp_string[128];
    char        username[100];
    char        password[100];
-   int         use_password = 0;
+   bool        use_password = false;
 
    g_verbose = false;
    force_quotes = true;
@@ -563,7 +563,7 @@ main(int argc, char **argv)
 
    progname = *argv;
 
-   while ((c = getopt(argc, argv, "acdDf:h:nNop:st:vzu")) != EOF)
+   while ((c = getopt(argc, argv, "acdDf:h:nNop:st:uvxz")) != EOF)
    {
        switch (c)
        {
@@ -630,14 +630,19 @@ main(int argc, char **argv)
                    }
                }
                break;
+           case 'u':
+               use_password = true;
+               break;
            case 'v':           /* verbose */
                g_verbose = true;
                break;
-           case 'z':           /* Dump ACLs and table ownership info */
-               aclsOption = true;
+           case 'x':           /* skip ACL dump */
+               aclsSkip = true;
                break;
-           case 'u':
-               use_password = 1;
+           case 'z':           /* Old ACL option bjm 1999/05/27 */
+               fprintf(stderr,
+                "%s: The -z option(dump ACLs) is now the default, continuing.\n",
+                   progname);
                break;
            default:
                usage(progname);
@@ -726,10 +731,10 @@ main(int argc, char **argv)
        if (g_verbose)
            fprintf(stderr, "%s last builtin oid is %u %s\n",
                    g_comment_start, g_last_builtin_oid, g_comment_end);
-       tblinfo = dumpSchema(g_fout, &numTables, tablename, aclsOption);
+       tblinfo = dumpSchema(g_fout, &numTables, tablename, aclsSkip);
    }
    else
-       tblinfo = dumpSchema(NULL, &numTables, tablename, aclsOption);
+       tblinfo = dumpSchema(NULL, &numTables, tablename, aclsSkip);
 
    if (!schemaOnly)
        dumpClasses(tblinfo, numTables, g_fout, tablename, oids);
@@ -2689,7 +2694,7 @@ void
 dumpTables(FILE *fout, TableInfo *tblinfo, int numTables,
           InhInfo *inhinfo, int numInherits,
           TypeInfo *tinfo, int numTypes, const char *tablename,
-          const bool acls)
+          const bool aclsSkip)
 {
    int         i,
                j,
@@ -2723,7 +2728,7 @@ dumpTables(FILE *fout, TableInfo *tblinfo, int numTables,
        {
            becomeUser(fout, tblinfo[i].usename);
            dumpSequence(fout, tblinfo[i]);
-           if (acls)
+           if (!aclsSkip)
                dumpACL(fout, tblinfo[i]);
        }
    }
@@ -2847,7 +2852,7 @@ dumpTables(FILE *fout, TableInfo *tblinfo, int numTables,
 
            strcat(q, ";\n");
            fputs(q, fout);
-           if (acls)
+           if (!aclsSkip)
                dumpACL(fout, tblinfo[i]);
 
        }
@@ -3380,7 +3385,7 @@ becomeUser(FILE *fout, const char *username)
 {
    static const char *lastusername = "";
 
-   if (!aclsOption)
+   if (aclsSkip)
        return;
 
    if (strcmp(lastusername, username) == 0)

diff --git a/src/man/pg_dump.1 b/src/man/pg_dump.1
index ec369b0b4bdf1b67bafa3680bd4ff3287e17f55c..a0930ee7bce9cd79d23bd56a5988e1cc97b167cc 100644 (file)
--- a/src/man/pg_dump.1
+++ b/src/man/pg_dump.1
@@ -1,6 +1,6 @@
 .\" This is -*-nroff-*-
 .\" XXX standard disclaimer belongs here....
-.\" $Header: /cvsroot/pgsql/src/man/Attic/pg_dump.1,v 1.16 1999/01/21 22:53:37 momjian Exp $
+.\" $Header: /cvsroot/pgsql/src/man/Attic/pg_dump.1,v 1.17 1999/05/27 16:29:05 momjian Exp $
 .TH PG_DUMP UNIX 7/15/98 PostgreSQL PostgreSQL
 .SH NAME
 pg_dump - dumps out a Postgres database into a script file
@@ -48,7 +48,7 @@ table]
 .BR "-v"
 ]
 [\c
-.BR "-z"
+.BR "-x"
 ]
 dbname
 .in -5n
@@ -113,8 +113,8 @@ Use password authentication. Prompts for username and password
 .BR "-v" ""
 Specifies verbose mode
 .TP
-.BR "-z" ""
-Include ACLs (grant/revoke commands) and table ownership information
+.BR "-x" ""
+Prevent dumping of ACLs (grant/revoke commands) and table ownership information
 .PP
 If dbname is not supplied, then the DATABASE environment variable value is used.
 .SH "CAVEATS AND LIMITATIONS"