* postmaster.c: cleanup pmdaemonize under win32; missed failure message

in CreateOptsFile
* s_lock.c: minor comment fix
* findbe.c: variables not used under win32 moved within #ifndef WIN32
case

Claudio Natoli

diff --git a/src/backend/postmaster/postmaster.c b/src/backend/postmaster/postmaster.c
index d7f26516d4a9de84e34db46eb40baf47c2f9eb6f..2e13838e70e1b34763e6d7bfbd727f69b23bb5b6 100644 (file)
--- a/src/backend/postmaster/postmaster.c
+++ b/src/backend/postmaster/postmaster.c
@@ -37,7 +37,7 @@
  *
  *
  * IDENTIFICATION
- *       $PostgreSQL: pgsql/src/backend/postmaster/postmaster.c,v 1.374 2004/03/15 15:56:21 momjian Exp $
+ *       $PostgreSQL: pgsql/src/backend/postmaster/postmaster.c,v 1.375 2004/03/15 16:18:42 momjian Exp $
  *
  * NOTES
  *
@@ -946,6 +946,10 @@ PostmasterMain(int argc, char *argv[])
 static void
 pmdaemonize(int argc, char *argv[])
 {
+#ifdef WIN32
+       /* not supported */
+       elog(FATAL,"SilentMode not supported under WIN32");
+#else
        int                     i;
        pid_t           pid;
 
@@ -958,12 +962,7 @@ pmdaemonize(int argc, char *argv[])
        getitimer(ITIMER_PROF, &prof_itimer);
 #endif
 
-#ifdef WIN32
-       /* FIXME: [fork/exec] to be implemented? */
-       abort();
-#else
        pid = fork();
-#endif
        if (pid == (pid_t) -1)
        {
                postmaster_error("could not fork background process: %s",
@@ -998,6 +997,7 @@ pmdaemonize(int argc, char *argv[])
        dup2(i, 1);
        dup2(i, 2);
        close(i);
+#endif
 }
 
 
@@ -3230,7 +3230,10 @@ CreateOptsFile(int argc, char *argv[])
        int                     i;
 
        if (FindExec(fullprogname, argv[0], "postmaster") < 0)
+       {
+               elog(LOG, "could not locate postmaster");
                return false;
+       }
 
        snprintf(filename, sizeof(filename), "%s/postmaster.opts", DataDir);
 

diff --git a/src/backend/storage/lmgr/s_lock.c b/src/backend/storage/lmgr/s_lock.c
index dcbf210519b8a7fe76065bab8255b335a95a446e..052cb0ce16789fca550c9d1a8c14c6da2dee2586 100644 (file)
--- a/src/backend/storage/lmgr/s_lock.c
+++ b/src/backend/storage/lmgr/s_lock.c
@@ -9,7 +9,7 @@
  *
  *
  * IDENTIFICATION
- *       $PostgreSQL: pgsql/src/backend/storage/lmgr/s_lock.c,v 1.25 2004/02/10 03:42:45 tgl Exp $
+ *       $PostgreSQL: pgsql/src/backend/storage/lmgr/s_lock.c,v 1.26 2004/03/15 16:18:43 momjian Exp $
  *
  *-------------------------------------------------------------------------
  */
@@ -72,7 +72,7 @@ s_lock(volatile slock_t *lock, const char *file, int line)
         * (and thus the probability of unintended failure) than to fix the
         * total time spent.
         *
-        * The select() delays are measured in centiseconds (0.01 sec) because 10
+        * The pg_usleep() delays are measured in centiseconds (0.01 sec) because 10
         * msec is a common resolution limit at the OS level.
         */
 #define SPINS_PER_DELAY                100

diff --git a/src/backend/utils/init/findbe.c b/src/backend/utils/init/findbe.c
index 2ef63872fa13aee8cb2164b50b5bbafdef25eb5e..035ff0553aa8033386e8132a4d35df68f61d40a3 100644 (file)
--- a/src/backend/utils/init/findbe.c
+++ b/src/backend/utils/init/findbe.c
@@ -7,7 +7,7 @@
  *
  *
  * IDENTIFICATION
- *       $PostgreSQL: pgsql/src/backend/utils/init/findbe.c,v 1.41 2004/02/02 00:11:31 momjian Exp $
+ *       $PostgreSQL: pgsql/src/backend/utils/init/findbe.c,v 1.42 2004/03/15 16:18:43 momjian Exp $
  *
  *-------------------------------------------------------------------------
  */
@@ -49,13 +49,13 @@ ValidateBinary(char *path)
        uid_t           euid;
        struct group *gp;
        struct passwd *pwp;
+       int                     i;
+       int                     in_grp = 0;
 #else
        char            path_exe[MAXPGPATH + 2 + strlen(".exe")];
 #endif
-       int                     i;
        int                     is_r = 0;
        int                     is_x = 0;
-       int                     in_grp = 0;
 
 #ifdef WIN32
        /* Win32 requires a .exe suffix for stat() */