From 6bf931d43b7241a93f71a925586b5f634e32b1bd Mon Sep 17 00:00:00 2001 From: Bruce Momjian Date: Wed, 26 Mar 2008 14:32:22 +0000 Subject: [PATCH] Strengthen warnings about using pg_dump's -i option. --- doc/src/sgml/ref/pg_dump.sgml | 15 +++++++-------- src/bin/pg_dump/pg_backup_db.c | 5 +++-- src/bin/pg_dump/pg_dump.c | 3 +-- src/bin/pg_dump/pg_dumpall.c | 8 ++++---- src/bin/pg_dump/pg_restore.c | 2 +- 5 files changed, 16 insertions(+), 17 deletions(-) diff --git a/doc/src/sgml/ref/pg_dump.sgml b/doc/src/sgml/ref/pg_dump.sgml index 8c84d6920d..30528402cc 100644 --- a/doc/src/sgml/ref/pg_dump.sgml +++ b/doc/src/sgml/ref/pg_dump.sgml @@ -296,19 +296,18 @@ PostgreSQL documentation - Ignore version mismatch between + Ignore incompatible version check between pg_dump and the database server. - pg_dump can dump from servers running - previous releases of PostgreSQL, but very old - versions are not supported anymore (currently, those prior to 7.0). Dumping from a server newer than pg_dump - is likely not to work at all. - Use this option if you need to override the version check (and - if pg_dump then fails, don't say - you weren't warned). + is likely fail and is disabled by default. + Also, while pg_dump can dump from servers running + previous releases of PostgreSQL, some very old + versions are not supported (currently, pre-7.0). + Use this option if you need to override the version check, but + be prepared for pg_dump to fail. diff --git a/src/bin/pg_dump/pg_backup_db.c b/src/bin/pg_dump/pg_backup_db.c index d61d7dd7b6..8f2de9739c 100644 --- a/src/bin/pg_dump/pg_backup_db.c +++ b/src/bin/pg_dump/pg_backup_db.c @@ -72,9 +72,10 @@ _check_database_version(ArchiveHandle *AH, bool ignoreVersion) write_msg(NULL, "server version: %s; %s version: %s\n", remoteversion_str, progname, PG_VERSION); if (ignoreVersion) - write_msg(NULL, "proceeding despite version mismatch\n"); + write_msg(NULL, "ignoring server version mismatch\n"); else - die_horribly(AH, NULL, "aborting because of version mismatch (Use the -i option to proceed anyway.)\n"); + die_horribly(AH, NULL, "aborting because of server version mismatch\n" + "Use the -i option to bypass server version check, but be prepared for failure.\n"); } } diff --git a/src/bin/pg_dump/pg_dump.c b/src/bin/pg_dump/pg_dump.c index 27affc3baa..28421849f8 100644 --- a/src/bin/pg_dump/pg_dump.c +++ b/src/bin/pg_dump/pg_dump.c @@ -745,8 +745,7 @@ help(const char *progname) printf(_("\nGeneral options:\n")); printf(_(" -f, --file=FILENAME output file name\n")); printf(_(" -F, --format=c|t|p output file format (custom, tar, plain text)\n")); - printf(_(" -i, --ignore-version proceed even when server version mismatches\n" - " pg_dump version\n")); + printf(_(" -i, --ignore-version ignore server version mismatch\n")); printf(_(" -v, --verbose verbose mode\n")); printf(_(" -Z, --compress=0-9 compression level for compressed formats\n")); printf(_(" --help show this help, then exit\n")); diff --git a/src/bin/pg_dump/pg_dumpall.c b/src/bin/pg_dump/pg_dumpall.c index 9a187e6374..90cd0f08e0 100644 --- a/src/bin/pg_dump/pg_dumpall.c +++ b/src/bin/pg_dump/pg_dumpall.c @@ -488,8 +488,7 @@ help(void) printf(_("\nGeneral options:\n")); printf(_(" -f, --file=FILENAME output file name\n")); - printf(_(" -i, --ignore-version proceed even when server version mismatches\n" - " pg_dumpall version\n")); + printf(_(" -i, --ignore-version ignore server version mismatch\n")); printf(_(" --help show this help, then exit\n")); printf(_(" --version output version information, then exit\n")); printf(_("\nOptions controlling the output content:\n")); @@ -1399,10 +1398,11 @@ connectDatabase(const char *dbname, const char *pghost, const char *pgport, fprintf(stderr, _("server version: %s; %s version: %s\n"), remoteversion_str, progname, PG_VERSION); if (ignoreVersion) - fprintf(stderr, _("proceeding despite version mismatch\n")); + fprintf(stderr, _("ignoring server version mismatch\n")); else { - fprintf(stderr, _("aborting because of version mismatch (Use the -i option to proceed anyway.)\n")); + fprintf(stderr, _("aborting because of server version mismatch\n" + "Use the -i option to bypass server version check, but be prepared for failure.\n")); exit(1); } } diff --git a/src/bin/pg_dump/pg_restore.c b/src/bin/pg_dump/pg_restore.c index 7d9e0b9220..6ab32bd18e 100644 --- a/src/bin/pg_dump/pg_restore.c +++ b/src/bin/pg_dump/pg_restore.c @@ -378,7 +378,7 @@ usage(const char *progname) printf(_(" -d, --dbname=NAME connect to database name\n")); printf(_(" -f, --file=FILENAME output file name\n")); printf(_(" -F, --format=c|t specify backup file format\n")); - printf(_(" -i, --ignore-version proceed even when server version mismatches\n")); + printf(_(" -i, --ignore-version ignore server version mismatch\n")); printf(_(" -l, --list print summarized TOC of the archive\n")); printf(_(" -v, --verbose verbose mode\n")); printf(_(" --help show this help, then exit\n")); -- 2.39.5