pg_upgrade; change major version comparisons to use <=, not <

This makes checking for older major versions more consistent.

Backpatch-through: 9.5

diff --git a/src/bin/pg_upgrade/check.c b/src/bin/pg_upgrade/check.c
index 91b12dc90e7b2870baa762f6194eb7268b025cfe..3f85f190b568048ab0ec0df4e2f6dc2c52beeb13 100644 (file)
--- a/src/bin/pg_upgrade/check.c
+++ b/src/bin/pg_upgrade/check.c
@@ -268,7 +268,7 @@ check_cluster_versions(void)
         * upgrades
         */
 
-       if (GET_MAJOR_VERSION(old_cluster.major_version) < 804)
+       if (GET_MAJOR_VERSION(old_cluster.major_version) <= 803)
                pg_fatal("This utility can only upgrade from PostgreSQL version 8.4 and later.\n");
 
        /* Only current PG version is supported as a target */
@@ -305,7 +305,7 @@ check_cluster_compatibility(bool live_check)
        check_control_data(&old_cluster.controldata, &new_cluster.controldata);
 
        /* We read the real port number for PG >= 9.1 */
-       if (live_check && GET_MAJOR_VERSION(old_cluster.major_version) < 901 &&
+       if (live_check && GET_MAJOR_VERSION(old_cluster.major_version) <= 900 &&
                old_cluster.port == DEF_PGUPORT)
                pg_fatal("When checking a pre-PG 9.1 live old server, "
                                 "you must specify the old server's port number.\n");
@@ -513,7 +513,7 @@ create_script_for_cluster_analyze(char **analyze_script_file_name)
        fprintf(script, "\"%s/vacuumdb\" %s--all --analyze-in-stages\n",
                        new_cluster.bindir, user_specification.data);
        /* Did we copy the free space files? */
-       if (GET_MAJOR_VERSION(old_cluster.major_version) < 804)
+       if (GET_MAJOR_VERSION(old_cluster.major_version) <= 803)
                fprintf(script, "\"%s/vacuumdb\" %s--all\n", new_cluster.bindir,
                                user_specification.data);
 

diff --git a/src/bin/pg_upgrade/controldata.c b/src/bin/pg_upgrade/controldata.c
index 38236415bef99f949418f83051215862306dad2a..298ae50757c13d390eb635eb4c381f034dbb2c01 100644 (file)
--- a/src/bin/pg_upgrade/controldata.c
+++ b/src/bin/pg_upgrade/controldata.c
@@ -180,7 +180,7 @@ get_control_data(ClusterInfo *cluster, bool live_check)
        }
 
        /* pg_resetxlog has been renamed to pg_resetwal in version 10 */
-       if (GET_MAJOR_VERSION(cluster->bin_version) < 1000)
+       if (GET_MAJOR_VERSION(cluster->bin_version) <= 906)
                resetwal_bin = "pg_resetxlog\" -n";
        else
                resetwal_bin = "pg_resetwal\" -n";

diff --git a/src/bin/pg_upgrade/exec.c b/src/bin/pg_upgrade/exec.c
index 036330932820ddc17a719cce8fedfce42c7f9bf1..087a80220d93ee32b79b28fa29db90dbbd2f5804 100644 (file)
--- a/src/bin/pg_upgrade/exec.c
+++ b/src/bin/pg_upgrade/exec.c
@@ -341,13 +341,13 @@ check_data_dir(ClusterInfo *cluster)
        check_single_dir(pg_data, "pg_twophase");
 
        /* pg_xlog has been renamed to pg_wal in v10 */
-       if (GET_MAJOR_VERSION(cluster->major_version) < 1000)
+       if (GET_MAJOR_VERSION(cluster->major_version) <= 906)
                check_single_dir(pg_data, "pg_xlog");
        else
                check_single_dir(pg_data, "pg_wal");
 
        /* pg_clog has been renamed to pg_xact in v10 */
-       if (GET_MAJOR_VERSION(cluster->major_version) < 1000)
+       if (GET_MAJOR_VERSION(cluster->major_version) <= 906)
                check_single_dir(pg_data, "pg_clog");
        else
                check_single_dir(pg_data, "pg_xact");
@@ -386,7 +386,7 @@ check_bin_dir(ClusterInfo *cluster)
        get_bin_version(cluster);
 
        /* pg_resetxlog has been renamed to pg_resetwal in version 10 */
-       if (GET_MAJOR_VERSION(cluster->bin_version) < 1000)
+       if (GET_MAJOR_VERSION(cluster->bin_version) <= 906)
                validate_exec(cluster->bindir, "pg_resetxlog");
        else
                validate_exec(cluster->bindir, "pg_resetwal");

diff --git a/src/bin/pg_upgrade/function.c b/src/bin/pg_upgrade/function.c
index 0c66d1c0566092869a0d3ffacffd24e77aeb9ad6..fef7f25895006774d7ff9b791e2d614d5b28842d 100644 (file)
--- a/src/bin/pg_upgrade/function.c
+++ b/src/bin/pg_upgrade/function.c
@@ -92,7 +92,7 @@ get_loadable_libraries(void)
                 * http://archives.postgresql.org/pgsql-hackers/2012-03/msg01101.php
                 * http://archives.postgresql.org/pgsql-bugs/2012-05/msg00206.php
                 */
-               if (GET_MAJOR_VERSION(old_cluster.major_version) < 901)
+               if (GET_MAJOR_VERSION(old_cluster.major_version) <= 900)
                {
                        PGresult   *res;
 
@@ -224,7 +224,7 @@ check_loadable_libraries(void)
                         * works for languages, and does not help with function shared
                         * objects, so we just do a general fix.
                         */
-                       if (GET_MAJOR_VERSION(old_cluster.major_version) < 901 &&
+                       if (GET_MAJOR_VERSION(old_cluster.major_version) <= 900 &&
                                strcmp(lib, "$libdir/plpython") == 0)
                        {
                                lib = "$libdir/plpython2";

diff --git a/src/bin/pg_upgrade/pg_upgrade.c b/src/bin/pg_upgrade/pg_upgrade.c
index ff78929707ef12699a7579274693f6020c54c755..cb0aff3e33a15138992cec76abdc1c0ac2f95cc7 100644 (file)
--- a/src/bin/pg_upgrade/pg_upgrade.c
+++ b/src/bin/pg_upgrade/pg_upgrade.c
@@ -406,7 +406,7 @@ create_new_objects(void)
         * We don't have minmxids for databases or relations in pre-9.3 clusters,
         * so set those after we have restored the schema.
         */
-       if (GET_MAJOR_VERSION(old_cluster.major_version) < 903)
+       if (GET_MAJOR_VERSION(old_cluster.major_version) <= 902)
                set_frozenxids(true);
 
        /* update new_cluster info now that we have objects in the databases */
@@ -465,9 +465,9 @@ copy_xact_xlog_xid(void)
         * Copy old commit logs to new data dir. pg_clog has been renamed to
         * pg_xact in post-10 clusters.
         */
-       copy_subdir_files(GET_MAJOR_VERSION(old_cluster.major_version) < 1000 ?
+       copy_subdir_files(GET_MAJOR_VERSION(old_cluster.major_version) <= 906 ?
                                          "pg_clog" : "pg_xact",
-                                         GET_MAJOR_VERSION(new_cluster.major_version) < 1000 ?
+                                         GET_MAJOR_VERSION(new_cluster.major_version) <= 906 ?
                                          "pg_clog" : "pg_xact");
 
        /* set the next transaction id and epoch of the new cluster */

diff --git a/src/bin/pg_upgrade/server.c b/src/bin/pg_upgrade/server.c
index 9f61d7bdf315e1cc688893376c8052b491815f28..d7d36b2faaefa0ca90446fc3ef4fa35e5c6688a8 100644 (file)
--- a/src/bin/pg_upgrade/server.c
+++ b/src/bin/pg_upgrade/server.c
@@ -221,7 +221,7 @@ start_postmaster(ClusterInfo *cluster, bool report_and_exit_on_error)
                snprintf(socket_string + strlen(socket_string),
                                 sizeof(socket_string) - strlen(socket_string),
                                 " -c %s='%s'",
-                                (GET_MAJOR_VERSION(cluster->major_version) < 903) ?
+                                (GET_MAJOR_VERSION(cluster->major_version) <= 902) ?
                                 "unix_socket_directory" : "unix_socket_directories",
                                 cluster->sockdir);
 #endif