Updated version of initdb with "rmdir/del" call on Win32.
authorBruce Momjian <bruce@momjian.us>
Mon, 10 Nov 2003 22:52:10 +0000 (22:52 +0000)
committerBruce Momjian <bruce@momjian.us>
Mon, 10 Nov 2003 22:52:10 +0000 (22:52 +0000)
Other cleanups as discussed.

src/bin/initdb/initdb.c

index 13522be5a3f237323138f76114795ef7cc948845..66aaa8107abd426868ed350f934ce24533582e11 100644 (file)
@@ -1,11 +1,9 @@
-
 /*-------------------------------------------------------------------------
  *
  * initdb
  *
  * author: Andrew Dunstan     mailto:andrew@dunslane.net
  *
- * Copyright (C) 2003 Andrew Dunstan
  * Portions Copyright (c) 1996-2003, PostgreSQL Global Development Group
  * Portions Copyright (c) 1994, Regents of the University of California
  *
@@ -14,7 +12,7 @@
  * This is a C implementation of the previous shell script for setting up a
  * PostgreSQL cluster location, and should be highly compatible with it.
  *
- * $Header: /cvsroot/pgsql/src/bin/initdb/initdb.c,v 1.1 2003/11/10 22:51:16 momjian Exp $
+ * $Header: /cvsroot/pgsql/src/bin/initdb/initdb.c,v 1.2 2003/11/10 22:52:10 momjian Exp $
  *
  * TODO:
  *  - clean up find_postgres code and return values
@@ -53,7 +51,6 @@ char     *bindir = PGBINDIR;
 char      *datadir = PGDATADIR;
 
 /* values to be obtained from arguments */
-
 char      *pg_data = "";
 char      *encoding = "";
 char      *locale = "";
@@ -93,8 +90,8 @@ char      infoversion[100];
 bool       not_ok = false;
 
 /* defaults */
-int            n_buffers = 50;
 int            n_connections = 10;
+int            n_buffers = 50;
 
 
 /* platform specific path stuff */
@@ -104,13 +101,13 @@ int           n_connections = 10;
 #else
 #define EXE ""
 #define DEVNULL "/dev/null"
-#endif   /* defined(__CYGWIN__) || defined(WIN32) */
+#endif
 
 #ifdef WIN32
 #define PATHSEP ';'
 #else
 #define PATHSEP ':'
-#endif   /* WIN32 */
+#endif
 
 /* detected path to postgres and (we assume) friends */
 char      *pgpath;
@@ -118,23 +115,20 @@ char     *pgpath;
 /* forward declare all our functions */
 static bool rmtree(char *, bool);
 static void exit_nicely(void);
-static void canonicalise_path(char *);
+static void canonicalize_path(char *);
 
 #ifdef WIN32
 static char *expanded_path(char *);
-static int init_unlink(const char *);
-
 #else
-#define expanded_path(x) ( x )
-#define init_unlink(x) unlink( (x) )
-#endif   /* WIN32 */
+#define expanded_path(x) (x)
+#endif
 
 static char **readfile(char *);
 static void writefile(char *, char **);
 static char *get_id(void);
 static char *get_encoding_id(char *);
 static char *get_short_version(void);
-static int build_path(char *, mode_t);
+static int mkdir_p(char *, mode_t);
 static bool check_data_dir(void);
 static bool mkdatadir(char *);
 static bool chklocale(const char *);
@@ -151,7 +145,7 @@ static void test_connections(void);
 static void setup_config(void);
 static void bootstrap_template1(char *);
 static void setup_shadow(void);
-static void get_set_pw(void);
+static void get_set_pwd(void);
 static void unlimit_systables(void);
 static void setup_depend(void);
 static void setup_sysviews(void);
@@ -171,60 +165,41 @@ static void *xmalloc(size_t);
 /*
  * macros for running pipes to postgres
  */
-
 #define PG_CMD_DECL        char cmd[MAXPGPATH]; char ** line ; FILE * pg
 #define PG_CMD_DECL_NOLINE        char cmd[MAXPGPATH]; FILE * pg
-#define PG_CMD_OPEN        \
-   do {\
-         pg = popen(cmd,PG_BINARY_W);\
-         if (pg == NULL) \
-           exit_nicely();\
-   } while (0)
-#define PG_CMD_CLOSE   \
-   do {\
-        if(pclose(pg)>>8 &0xff)\
-           exit_nicely();\
-   } while (0)
-#define PG_CMD_PUTLINE \
-   do {\
+
+#define PG_CMD_OPEN \
+do { \
+         pg = popen(cmd,PG_BINARY_W); \
+         if (pg == NULL)  \
+           exit_nicely(); \
+} while (0)
+
+#define PG_CMD_CLOSE \
+do { \
+        if(pclose(pg) >> 8 & 0xff) \
+           exit_nicely(); \
+} while (0)
+
+#define PG_CMD_PUTLINE \
+do { \
         if (fputs(*line, pg) < 0) \
           exit_nicely(); \
-        fflush(pg);\
-   } while (0)
-
-
+        fflush(pg); \
+} while (0)
 
-
-#ifdef WIN32
-
-/* workaround for win32 unlink bug, not using logging like in port/dirmod.c */
-
-/* make sure we call the real unlink from MSVCRT */
-
-#ifdef unlink
-#undef unlink
+#ifndef WIN32
+#define QUOTE_PATH ""
+#else
+#define QUOTE_PATH "\""
 #endif
 
-static int
-init_unlink(const char *path)
-{
-   while (unlink(path))
-   {
-       if (errno != EACCES)
-           return -1;
-       Sleep(100);             /* ms */
-   }
-   return 0;
-}
-#endif   /* WIN32 */
-
 /*
- * routines to check mem allocations and fail noisily
+ * routines to check mem allocations and fail noisily.
  * Note that we can't call exit_nicely() on a memory failure, as it calls
  * rmtree() which needs memory allocation. So we just exit with a bang.
  *
  */
-
 static void *
 xmalloc(size_t size)
 {
@@ -263,84 +238,31 @@ xstrdup(const char *s)
 static bool
 rmtree(char *path, bool rmtopdir)
 {
-   char        filepath[MAXPGPATH];
-   DIR        *dir;
-   struct dirent *file;
-   char      **filenames;
-   char      **filename;
-   int         numnames = 0;
-   struct stat statbuf;
+   char        buf[MAXPGPATH + 64];
 
-   /*
-    * we copy all the names out of the directory before we start
-    * modifying it.
-    *
-    */
-
-   dir = opendir(path);
-   if (dir == NULL)
-       return false;
-
-   while ((file = readdir(dir)) != NULL)
-   {
-       if (strcmp(file->d_name, ".") != 0 && strcmp(file->d_name, "..") != 0)
-           numnames++;
-   }
-
-   rewinddir(dir);
-
-   filenames = xmalloc((numnames + 2) * sizeof(char *));
-   numnames = 0;
-
-   while ((file = readdir(dir)) != NULL)
-   {
-       if (strcmp(file->d_name, ".") != 0 && strcmp(file->d_name, "..") != 0)
-           filenames[numnames++] = xstrdup(file->d_name);
-   }
-
-   filenames[numnames] = NULL;
-
-   closedir(dir);
-
-   /* now we have the names we can start removing things */
-
-   for (filename = filenames; *filename; filename++)
-   {
-       snprintf(filepath, MAXPGPATH, "%s/%s", path, *filename);
-
-       if (stat(filepath, &statbuf) != 0)
-           return false;
-
-       if (S_ISDIR(statbuf.st_mode))
-       {
-           /* call ourselves recursively for a directory */
-           if (!rmtree(filepath, true))
-               return false;
-       }
-       else
-       {
-           if (init_unlink(filepath) != 0)
-               return false;
-       }
-   }
-
-   if (rmtopdir)
-   {
-       if (rmdir(path) != 0)
-           return false;
-   }
+#ifndef WIN32
+   /* doesn't handle .* files */
+   snprintf(buf, sizeof(buf), "rm -rf '%s%s'", path,
+       rmtopdir ? "" : "/*");
+#else
+   snprintf(buf, sizeof(buf), "%s /s /q \"%s\"",
+       rmtopdir ? "rmdir" : "del", path);
+#endif
 
-   return true;
+   return !system(buf);
 }
 
 
 /*
  * make all paths look like unix, with forward slashes
- * also strip any trailing slash
+ * also strip any trailing slash.
+ * The Windows command processor will accept suitably quoted paths
+ * with forward slashes, but barfs badly with mixed forward and back
+ * slashes. Removing the trailing slash on a path means we never get
+ * ugly double slashes.
  */
-
 static void
-canonicalise_path(char *path)
+canonicalize_path(char *path)
 {
    char       *p;
 
@@ -349,7 +271,7 @@ canonicalise_path(char *path)
 #ifdef WIN32
        if (*p == '\\')
            *p = '/';
-#endif   /* WIN32 */
+#endif
    }
    if (p != path && *--p == '/')
        *p = '\0';
@@ -361,7 +283,6 @@ canonicalise_path(char *path)
  * This does most of what sed was used for in the shell script, but
  * doesn't need any regexp stuff.
  */
-
 static char **
 replace_token(char **lines, char *token, char *replacement)
 {
@@ -372,7 +293,6 @@ replace_token(char **lines, char *token, char *replacement)
                replen,
                diff;
 
-
    for (i = 0; lines[i]; i++)
        numlines++;
 
@@ -431,9 +351,7 @@ readfile(char *path)
    char       *buffer;
    int         c;
 
-   infile = fopen(path, "r");
-
-   if (!infile)
+   if ((infile = fopen(path, "r")) == NULL)
    {
        fprintf(stderr, "could not read %s ... ", path);
        exit_nicely();
@@ -491,8 +409,8 @@ writefile(char *path, char **lines)
    FILE       *out_file;
    char      **line;
 
-   out_file = fopen(path, PG_BINARY_W);
-   if (out_file == NULL)
+   ;
+   if ((out_file = fopen(path, PG_BINARY_W)) == NULL)
    {
        fprintf(stderr, "could not write %s ... ", path);
        exit_nicely();
@@ -515,9 +433,8 @@ writefile(char *path, char **lines)
  * we also assume it isn't null.
  *
  */
-
 static int
-build_path(char *path, mode_t omode)
+mkdir_p(char *path, mode_t omode)
 {
    struct stat sb;
    mode_t      numask,
@@ -532,7 +449,6 @@ build_path(char *path, mode_t omode)
    retval = 0;
 
 #ifdef WIN32
-
    /* skip network and drive specifiers for win32 */
    if (strlen(p) >= 2)
    {
@@ -551,7 +467,7 @@ build_path(char *path, mode_t omode)
            p += 2;
        }
    }
-#endif   /* WIN32 */
+#endif
 
    if (p[0] == '/')            /* Skip leading '/'. */
        ++p;
@@ -614,7 +530,7 @@ build_path(char *path, mode_t omode)
    }
    if (!first && !last)
        (void) umask(oumask);
-   return (retval);
+   return retval;
 }
 
 /*
@@ -673,7 +589,7 @@ get_id(void)
                progname);
        exit(1);
    }
-#endif   /* __BEOS__ */
+#endif
 
 #else                          /* the windows code */
 
@@ -687,7 +603,7 @@ get_id(void)
 
    pw->pw_uid = 1;
    GetUserName(pw->pw_name, &pwname_size);
-#endif   /* ! WIN32 */
+#endif
 
    return xstrdup(pw->pw_name);
 }
@@ -785,7 +701,7 @@ check_data_dir(void)
 
    closedir(chkdir);
 
-   return (empty);
+   return empty;
 }
 
 /*
@@ -812,7 +728,7 @@ mkdatadir(char *subdir)
    else if (subdir == NULL || errno != ENOENT)
        return false;
    else
-       return !build_path(path, 0700);
+       return !mkdir_p(path, 0700);
 }
 
 
@@ -853,7 +769,6 @@ check_input(char *path)
  * don't overkill
  *
  */
-
 #define FIND_SUCCESS 0
 #define FIND_NOT_FOUND 1
 #define FIND_STAT_ERR 2
@@ -875,11 +790,9 @@ find_postgres(char *path)
    char        line[100];
 
 #ifndef WIN32
-
    int         permmask = S_IROTH | S_IXOTH;
 #endif
 
-
    struct stat statbuf;
    FILE       *pgver;
    int         plen = strlen(path);
@@ -900,12 +813,13 @@ find_postgres(char *path)
        return FIND_NOT_REGFILE;
 
 #ifndef WIN32
-
-   /* on windows a .exe file should be executable - this is the unix test */
-
+   /*
+    * Only unix requires this test, on WIN32 an .exe file should be
+    * executable
+    */
    if ((statbuf.st_mode & permmask) != permmask)
        return FIND_BAD_PERM;
-#endif   /* ! WIN32 */
+#endif
 
    snprintf(cmd, MAXPGPATH, "\"%s/postgres\" -V 2>%s", path, DEVNULL);
 
@@ -913,11 +827,8 @@ find_postgres(char *path)
        return FIND_EXEC_ERR;
 
    if (fgets(line, sizeof(line), pgver) == NULL)
-   {
        perror("fgets failure");
 
-   }
-
    pclose(pgver);
 
    if (strcmp(line, PG_VERSIONSTR) != 0)
@@ -926,14 +837,12 @@ find_postgres(char *path)
    return FIND_SUCCESS;
 }
 
-#ifdef WIN32
-
 /*
  * Windows doesn't like relative paths to executables (other things work fine)
  * so we call its builtin function to expand them. Elsewhere this is a NOOP
  *
  */
-
+#ifdef WIN32
 static char *
 expanded_path(char *path)
 {
@@ -944,10 +853,10 @@ expanded_path(char *path)
        perror("expanded path");
        return path;
    }
-   canonicalise_path(abspath);
+   canonicalize_path(abspath);
    return xstrdup(abspath);
 }
-#endif   /* WIN32 */
+#endif 
 
 /*
  * set the paths pointing to postgres
@@ -992,7 +901,7 @@ set_paths(void)
                }
                else
                {
-                   canonicalise_path(cursor);
+                   canonicalize_path(cursor);
                    pathbits[pathsegs] = cursor;
                }
                pathsegs++;
@@ -1024,7 +933,7 @@ set_paths(void)
        char       *cbindir;
 
        cbindir = xstrdup(bindir);
-       canonicalise_path(cbindir);
+       canonicalize_path(cbindir);
        res = find_postgres(expanded_path(cbindir));
        if (res == 0)
            pgpath = expanded_path(cbindir);
@@ -1078,63 +987,63 @@ set_null_conf(void)
 }
 
 /*
- * check how many buffers we can run with
+ * check how many connections we can sustain
  *
  */
 static void
-test_buffers(void)
+test_connections(void)
 {
    char       *format =
-   "\"%s/postgres\"  -boot -x 0 -F "
-   "-c shared_buffers=%d -c max_connections=5 template1 <%s >%s 2>&1";
+   "\"%s/postgres\" -boot -x 0 -F "
+   "-c shared_buffers=%d -c max_connections=%d template1 <%s >%s 2>&1";
    char        cmd[MAXPGPATH];
-   int         bufs[] =
-   {1000, 900, 800, 700, 600, 500, 400, 300, 200, 100, 50};
-   int         len = sizeof(bufs) / sizeof(int);
+   int         conns[] = {100, 50, 40, 30, 20, 10};
+   int         len = sizeof(conns) / sizeof(int);
    int         i,
                status;
 
    for (i = 0; i < len; i++)
    {
-       snprintf(cmd, sizeof(cmd), format, pgpath, bufs[i], DEVNULL, DEVNULL);
+       snprintf(cmd, sizeof(cmd), format,
+                pgpath, conns[i] * 5, conns[i], DEVNULL, DEVNULL);
        status = system(cmd);
        if (status == 0)
            break;
    }
    if (i >= len)
        i = len - 1;
-   n_buffers = bufs[i];
-   printf("buffers set to %d\n", n_buffers);
+   n_connections = conns[i];
+   printf("connections set to %d\n", n_connections);
 }
 
 /*
- * check how many connections we can sustain
+ * check how many buffers we can run with
  *
  */
 static void
-test_connections(void)
+test_buffers(void)
 {
    char       *format =
-   "\"%s/postgres\" -boot -x 0 -F "
+   "\"%s/postgres\"  -boot -x 0 -F "
    "-c shared_buffers=%d -c max_connections=%d template1 <%s >%s 2>&1";
    char        cmd[MAXPGPATH];
-   int         conns[] = {100, 50, 40, 30, 20, 10};
-   int         len = sizeof(conns) / sizeof(int);
+   int         bufs[] = {1000, 900, 800, 700, 600, 500, 400, 300, 200, 100, 50};
+   int         len = sizeof(bufs) / sizeof(int);
    int         i,
                status;
 
    for (i = 0; i < len; i++)
    {
-       snprintf(cmd, sizeof(cmd), format,
-                pgpath, n_buffers, conns[i], DEVNULL, DEVNULL);
+       snprintf(cmd, sizeof(cmd), format, pgpath, bufs[i], n_connections,
+           DEVNULL, DEVNULL);
        status = system(cmd);
        if (status == 0)
            break;
    }
    if (i >= len)
        i = len - 1;
-   n_connections = conns[i];
-   printf("connections set to %d\n", n_connections);
+   n_buffers = bufs[i];
+   printf("buffers set to %d\n", n_buffers);
 }
 
 /*
@@ -1155,12 +1064,12 @@ setup_config(void)
 
    conflines = readfile(conf_file);
 
-   snprintf(repltok, sizeof(repltok), "shared_buffers = %d", n_buffers);
-   conflines = replace_token(conflines, "#shared_buffers = 1000", repltok);
-
    snprintf(repltok, sizeof(repltok), "max_connections = %d", n_connections);
    conflines = replace_token(conflines, "#max_connections = 100", repltok);
 
+   snprintf(repltok, sizeof(repltok), "shared_buffers = %d", n_buffers);
+   conflines = replace_token(conflines, "#shared_buffers = 1000", repltok);
+
    snprintf(repltok, sizeof(repltok), "lc_messages = '%s'", lc_messages);
    conflines = replace_token(conflines, "#lc_messages = 'C'", repltok);
 
@@ -1190,7 +1099,7 @@ setup_config(void)
    conflines = replace_token(conflines,
                              "host    all         all         ::1",
                              "#host    all         all         ::1");
-#endif   /* ! HAVE_IPV6 */
+#endif
 
    snprintf(path, MAXPGPATH, "%s/pg_hba.conf", pg_data);
 
@@ -1211,7 +1120,6 @@ setup_config(void)
    free(conflines);
 
    check_ok();
-
 }
 
 
@@ -1258,7 +1166,6 @@ bootstrap_template1(char *short_version)
     * already called setlocale().
     *
     */
-
    snprintf(cmd, MAXPGPATH, "LC_COLLATE=%s", lc_collate);
    putenv(xstrdup(cmd));
 
@@ -1337,23 +1244,23 @@ setup_shadow(void)
  *
  */
 static void
-get_set_pw(void)
+get_set_pwd(void)
 {
    PG_CMD_DECL_NOLINE;
 
-   char       *pw1,
-              *pw2;
-   char        pwpath[MAXPGPATH];
+   char       *pwd1,
+              *pwd2;
+   char        pwdpath[MAXPGPATH];
    struct stat statbuf;
 
-   pw1 = simple_prompt("Enter new superuser password: ", 100, false);
-   pw2 = simple_prompt("Enter it again: ", 100, false);
-   if (strcmp(pw1, pw2) != 0)
+   pwd1 = simple_prompt("Enter new superuser password: ", 100, false);
+   pwd2 = simple_prompt("Enter it again: ", 100, false);
+   if (strcmp(pwd1, pwd2) != 0)
    {
        fprintf(stderr, "Passwords didn't match.\n");
        exit_nicely();
    }
-   free(pw2);
+   free(pwd2);
 
    printf("storing the password ... ");
 
@@ -1365,7 +1272,7 @@ get_set_pw(void)
    PG_CMD_OPEN;
 
    if (fprintf(
-      pg, "ALTER USER \"%s\" WITH PASSWORD '%s';\n", username, pw1) < 0)
+      pg, "ALTER USER \"%s\" WITH PASSWORD '%s';\n", username, pwd1) < 0)
    {
        /* write failure */
        exit_nicely();
@@ -1374,8 +1281,8 @@ get_set_pw(void)
 
    PG_CMD_CLOSE;
 
-   snprintf(pwpath, MAXPGPATH, "%s/global/pg_pwd", pg_data);
-   if (stat(pwpath, &statbuf) != 0 || !S_ISREG(statbuf.st_mode))
+   snprintf(pwdpath, MAXPGPATH, "%s/global/pg_pwd", pg_data);
+   if (stat(pwdpath, &statbuf) != 0 || !S_ISREG(statbuf.st_mode))
    {
        fprintf(stderr,
                "%s: The password file was not generated - "
@@ -1898,7 +1805,6 @@ make_template0(void)
  * So this will need some testing on Windows.
  *
  */
-
 static void
 trapsig(int signum)
 {
@@ -2009,7 +1915,7 @@ setlocales(void)
        /* when not available, get the CTYPE setting */
        lc_messages = xstrdup(setlocale(LC_CTYPE, NULL));
    }
-#endif   /* LC_MESSAGES */
+#endif
 
 }
 
@@ -2017,7 +1923,6 @@ setlocales(void)
  * help text data
  *
  */
-
 char      *usage_text[] = {
    "$CMDNAME initializes a PostgreSQL database cluster.\n",
    "\n",
@@ -2117,12 +2022,12 @@ main(int argc, char *argv[])
 
 #if defined(__CYGWIN__) || defined(WIN32)
    char       *exe;            /* location of exe suffix in progname */
-#endif   /* defined(__CYGWIN__) || defined(WIN32) */
+#endif
 
    setlocale(LC_ALL, "");
 
    carg0 = xstrdup(argv[0]);
-   canonicalise_path(carg0);
+   canonicalize_path(carg0);
 
    lastsep = strrchr(carg0, '/');
    progname = lastsep ? xstrdup(lastsep + 1) : carg0;
@@ -2135,7 +2040,7 @@ main(int argc, char *argv[])
        /* strip .exe suffix, regardless of case */
        *exe = '\0';
    }
-#endif   /* defined(__CYGWIN__) || defined(WIN32) */
+#endif
 
    if (lastsep)
    {
@@ -2253,7 +2158,7 @@ main(int argc, char *argv[])
        }
    }
 
-   canonicalise_path(pg_data);
+   canonicalize_path(pg_data);
 
    /*
     * we have to set PGDATA for postgres rather than pass it on the
@@ -2317,7 +2222,6 @@ main(int argc, char *argv[])
    set_input(&features_file, "sql_features.txt");
    set_input(&system_views_file, "system_views.sql");
 
-
    if (show_setting || debug)
    {
        fprintf(stderr,
@@ -2335,11 +2239,9 @@ main(int argc, char *argv[])
                hba_file, ident_file);
    }
 
-
    if (show_setting)
        exit(0);
 
-
    check_input(bki_file);
    check_input(desc_file);
    check_input(hba_file);
@@ -2388,17 +2290,16 @@ main(int argc, char *argv[])
    /* some of these are not valid on Windows */
 #ifdef SIGHUP
    pqsignal(SIGHUP, trapsig);
-#endif   /* SIGHUP */
+#endif
 #ifdef SIGINT
    pqsignal(SIGINT, trapsig);
-#endif   /* SIGINT */
+#endif
 #ifdef SIGQUIT
    pqsignal(SIGQUIT, trapsig);
-#endif   /* SIGQUIT */
+#endif
 #ifdef SIGTERM
    pqsignal(SIGTERM, trapsig);
-#endif   /* SIGTERM */
-
+#endif
 
    /* clear this we'll use it in a few lines */
    errno = 0;
@@ -2420,7 +2321,6 @@ main(int argc, char *argv[])
     * check_data_dir() called opendir - the errno should still be hanging
     * around
     */
-
    if (errno == ENOENT)
    {
        printf("creating directory \"%s\" ... ", pg_data);
@@ -2447,9 +2347,9 @@ main(int argc, char *argv[])
 
    set_null_conf();
 
-   test_buffers();
-
+   /* test connections first because it has more constraints */
    test_connections();
+   test_buffers();
 
    setup_config();
 
@@ -2458,9 +2358,8 @@ main(int argc, char *argv[])
    set_short_version(short_version, "base/1");
 
    setup_shadow();
-
    if (pwprompt)
-       get_set_pw();
+       get_set_pwd();
 
    unlimit_systables();
 
@@ -2480,14 +2379,12 @@ main(int argc, char *argv[])
 
    make_template0();
 
-   printf("\n"
-          "Success. You can now start the database server using:\n"
-          "\n"
-          "    \"%s/postmaster\" -D \"%s\"\n"
+   printf("\nSuccess. You can now start the database server using:\n\n"
+          "    %s%s%s/postmaster -D %s%s%s\n"
           "or\n"
-          "    \"%s/pg_ctl\" -D \"%s\" -l logfile start\n"
-          "\n",
-          pgpath, pg_data, pgpath, pg_data);
+          "    %s%s%s/pg_ctl -D %s%s%s -l logfile start\n\n",
+           QUOTE_PATH, pgpath, QUOTE_PATH, QUOTE_PATH, pg_data, QUOTE_PATH,
+           QUOTE_PATH, pgpath, QUOTE_PATH, QUOTE_PATH, pg_data, QUOTE_PATH);
 
    return 0;
 }