pg_upgrade: Message style fixes

diff --git a/src/bin/pg_upgrade/check.c b/src/bin/pg_upgrade/check.c
index 86225eaa4c25e6204702c9a25bd3649c1b5efd9a..b7e1e4be190177095d5056bfd20094fd01304cf2 100644 (file)
--- a/src/bin/pg_upgrade/check.c
+++ b/src/bin/pg_upgrade/check.c
@@ -987,7 +987,7 @@ check_for_jsonb_9_4_usage(ClusterInfo *cluster)
    bool        found = false;
    char        output_path[MAXPGPATH];
 
-   prep_status("Checking for incompatible jsonb data type");
+   prep_status("Checking for incompatible \"jsonb\" data type");
 
    snprintf(output_path, sizeof(output_path), "tables_using_jsonb.txt");
 

diff --git a/src/bin/pg_upgrade/exec.c b/src/bin/pg_upgrade/exec.c
index cb8e29b17c8253afe0060cb8bb9759608b819a54..1cf64e1a4509501e32a32b68951b4ee387d84a70 100644 (file)
--- a/src/bin/pg_upgrade/exec.c
+++ b/src/bin/pg_upgrade/exec.c
@@ -51,7 +51,7 @@ get_bin_version(ClusterInfo *cluster)
        *strchr(cmd_output, '\n') = '\0';
 
    if (sscanf(cmd_output, "%*s %*s %d.%d", &pre_dot, &post_dot) < 1)
-       pg_fatal("could not get version from %s\n", cmd);
+       pg_fatal("could not get pg_ctl version output from %s\n", cmd);
 
    cluster->bin_version = (pre_dot * 100 + post_dot) * 100;
 }
@@ -143,7 +143,7 @@ exec_prog(const char *log_file, const char *opt_log_file,
 #endif
 
    if (log == NULL)
-       pg_fatal("cannot write to log file %s\n", log_file);
+       pg_fatal("could not write to log file \"%s\"\n", log_file);
 
 #ifdef WIN32
    /* Are we printing "command:" before its output? */
@@ -198,7 +198,7 @@ exec_prog(const char *log_file, const char *opt_log_file,
     * log these commands to a third file, but that just adds complexity.
     */
    if ((log = fopen(log_file, "a")) == NULL)
-       pg_fatal("cannot write to log file %s\n", log_file);
+       pg_fatal("could not write to log file \"%s\"\n", log_file);
    fprintf(log, "\n\n");
    fclose(log);
 #endif
@@ -426,7 +426,7 @@ validate_exec(const char *dir, const char *cmdName)
        pg_fatal("check for \"%s\" failed: %s\n",
                 path, strerror(errno));
    else if (!S_ISREG(buf.st_mode))
-       pg_fatal("check for \"%s\" failed: not an executable file\n",
+       pg_fatal("check for \"%s\" failed: not a regular file\n",
                 path);
 
    /*

diff --git a/src/bin/pg_upgrade/file.c b/src/bin/pg_upgrade/file.c
index eb925d1e0fcf36b8841686d731434724965c94d7..ae8d89fb66ba1618d914adf54ff8c4f1fa1b6e73 100644 (file)
--- a/src/bin/pg_upgrade/file.c
+++ b/src/bin/pg_upgrade/file.c
@@ -290,7 +290,7 @@ check_hard_link(void)
 
    if (pg_link_file(existing_file, new_link_file) < 0)
        pg_fatal("could not create hard link between old and new data directories: %s\n"
-                "In link mode the old and new data directories must be on the same file system volume.\n",
+                "In link mode the old and new data directories must be on the same file system.\n",
                 strerror(errno));
 
    unlink(new_link_file);

diff --git a/src/bin/pg_upgrade/option.c b/src/bin/pg_upgrade/option.c
index bbe364741cebfb4953a912db7c5b7d57b3a452ce..c74eb25e18ce29d896b6e341e7a56a16596ab05d 100644 (file)
--- a/src/bin/pg_upgrade/option.c
+++ b/src/bin/pg_upgrade/option.c
@@ -98,7 +98,7 @@ parseCommandLine(int argc, char *argv[])
        pg_fatal("%s: cannot be run as root\n", os_info.progname);
 
    if ((log_opts.internal = fopen_priv(INTERNAL_LOG_FILE, "a")) == NULL)
-       pg_fatal("cannot write to log file %s\n", INTERNAL_LOG_FILE);
+       pg_fatal("could not write to log file \"%s\"\n", INTERNAL_LOG_FILE);
 
    while ((option = getopt_long(argc, argv, "d:D:b:B:cj:ko:O:p:P:rU:v",
                                 long_options, &optindex)) != -1)
@@ -214,7 +214,7 @@ parseCommandLine(int argc, char *argv[])
    for (filename = output_files; *filename != NULL; filename++)
    {
        if ((fp = fopen_priv(*filename, "a")) == NULL)
-           pg_fatal("cannot write to log file %s\n", *filename);
+           pg_fatal("could not write to log file \"%s\"\n", *filename);
 
        /* Start with newline because we might be appending to a file. */
        fprintf(fp, "\n"
@@ -262,7 +262,7 @@ parseCommandLine(int argc, char *argv[])
        canonicalize_path(new_cluster_pgdata);
 
        if (!getcwd(cwd, MAXPGPATH))
-           pg_fatal("cannot find current directory\n");
+           pg_fatal("could not determine current directory\n");
        canonicalize_path(cwd);
        if (path_is_prefix_of_path(new_cluster_pgdata, cwd))
            pg_fatal("cannot run pg_upgrade from inside the new cluster data directory on Windows\n");
@@ -459,7 +459,7 @@ get_sock_dir(ClusterInfo *cluster, bool live_check)
            /* Use the current directory for the socket */
            cluster->sockdir = pg_malloc(MAXPGPATH);
            if (!getcwd(cluster->sockdir, MAXPGPATH))
-               pg_fatal("cannot find current directory\n");
+               pg_fatal("could not determine current directory\n");
        }
        else
        {
@@ -477,14 +477,16 @@ get_sock_dir(ClusterInfo *cluster, bool live_check)
            snprintf(filename, sizeof(filename), "%s/postmaster.pid",
                     cluster->pgdata);
            if ((fp = fopen(filename, "r")) == NULL)
-               pg_fatal("Cannot open file %s: %m\n", filename);
+               pg_fatal("could not open file \"%s\": %s\n",
+                        filename, strerror(errno));
 
            for (lineno = 1;
                 lineno <= Max(LOCK_FILE_LINE_PORT, LOCK_FILE_LINE_SOCKET_DIR);
                 lineno++)
            {
                if (fgets(line, sizeof(line), fp) == NULL)
-                   pg_fatal("Cannot read line %d from %s: %m\n", lineno, filename);
+                   pg_fatal("could not read line %d from file \"%s\": %s\n",
+                            lineno, filename, strerror(errno));
 
                /* potentially overwrite user-supplied value */
                if (lineno == LOCK_FILE_LINE_PORT)
@@ -501,7 +503,7 @@ get_sock_dir(ClusterInfo *cluster, bool live_check)
 
            /* warn of port number correction */
            if (orig_port != DEF_PGUPORT && old_cluster.port != orig_port)
-               pg_log(PG_WARNING, "User-supplied old port number %hu corrected to %hu\n",
+               pg_log(PG_WARNING, "user-supplied old port number %hu corrected to %hu\n",
                       orig_port, cluster->port);
        }
    }

diff --git a/src/bin/pg_upgrade/pg_upgrade.c b/src/bin/pg_upgrade/pg_upgrade.c
index 2a68ce6efaaabbe32ff4cc159b8beee1a2c4973a..d44fefb457ba31b33d959ae62190d3a678f62320 100644 (file)
--- a/src/bin/pg_upgrade/pg_upgrade.c
+++ b/src/bin/pg_upgrade/pg_upgrade.c
@@ -261,7 +261,7 @@ prepare_new_cluster(void)
     * datfrozenxid, relfrozenxids, and relminmxid later to match the new xid
     * counter later.
     */
-   prep_status("Freezing all rows on the new cluster");
+   prep_status("Freezing all rows in the new cluster");
    exec_prog(UTILITY_LOG_FILE, NULL, true,
              "\"%s/vacuumdb\" %s --all --freeze %s",
              new_cluster.bindir, cluster_conn_opts(&new_cluster),
@@ -471,7 +471,7 @@ copy_xact_xlog_xid(void)
         */
        remove_new_subdir("pg_multixact/offsets", false);
 
-       prep_status("Setting oldest multixact ID on new cluster");
+       prep_status("Setting oldest multixact ID in new cluster");
 
        /*
         * We don't preserve files in this case, but it's important that the

diff --git a/src/bin/pg_upgrade/server.c b/src/bin/pg_upgrade/server.c
index 26e60fab469dda5f77978544491cd0d68fcde579..3e3323a6e8a4a6aacfc0b9056a908540d8b6ca10 100644 (file)
--- a/src/bin/pg_upgrade/server.c
+++ b/src/bin/pg_upgrade/server.c
@@ -167,7 +167,7 @@ get_major_server_version(ClusterInfo *cluster)
    if (fscanf(version_fd, "%63s", cluster->major_version_str) == 0 ||
        sscanf(cluster->major_version_str, "%d.%d", &integer_version,
               &fractional_version) < 1)
-       pg_fatal("could not get version from %s\n", cluster->pgdata);
+       pg_fatal("could not parse PG_VERSION file from %s\n", cluster->pgdata);
 
    fclose(version_fd);