Small polishing of syslog facility and ident settings. Only allow setting

at postmaster start, rename syslog_progid to syslog_ident, since syslog
itself uses that term, fix doc markup.

diff --git a/doc/src/sgml/runtime.sgml b/doc/src/sgml/runtime.sgml
index f7eff6da82194397c281474ba12dfb1a9c1eff14..a8117cafa0ab182098f8ba612b09b0bb69092623 100644 (file)
--- a/doc/src/sgml/runtime.sgml
+++ b/doc/src/sgml/runtime.sgml
@@ -1,5 +1,5 @@
 <!--
-$Header: /cvsroot/pgsql/doc/src/sgml/runtime.sgml,v 1.35 2000/11/14 18:11:30 petere Exp $
+$Header: /cvsroot/pgsql/doc/src/sgml/runtime.sgml,v 1.36 2000/11/14 19:13:26 petere Exp $
 -->
 
 <Chapter Id="runtime">
@@ -825,22 +825,25 @@ env PGOPTIONS='-c geqo=off' psql
       <term>SYSLOG_FACILITY (<type>string</type>)</term>
        <listitem>
         <para>
-          If the SYSLOG option is set to 1 or greater, this option determines
-          the <application>syslog</application> facility used.  You may choose
-          from LOCAL0, LOCAL1, LOCAL2, LOCAL3, LOCAL4, LOCAL5, LOCAL6, LOCAL7.
-          the default is LOCAL0
+          This option determines the <application>syslog</application>
+          <quote>facility</quote> to be used when syslog is enabled.
+          You may choose from LOCAL0, LOCAL1, LOCAL2, LOCAL3, LOCAL4,
+          LOCAL5, LOCAL6, LOCAL7; the default is LOCAL0.  See also the
+          documentation of your system's
+          <application>syslog</application>.
         </para>
        </listitem>
      </varlistentry>
      
      <varlistentry>
-      <term>SYSLOG_PROGID (<type>string</type>)</term>
+      <term>SYSLOG_IDENT (<type>string</type>)</term>
        <listitem>
         <para>
-         If the SYSLOG option is set to 1 or greater, this option determines
-         the program id used to identify <product>PostgreSQL</product> messages
-         in <application>syslog</application> log messages.  The default is
-         postgres.
+         If logging to syslog is enabled, this option determines the
+         program name used to identify
+         <productname>PostgreSQL</productname> messages in
+         <application>syslog</application> log messages.  The default
+         is <quote>postgres</quote>.
         </para>
        </listitem>
       </varlistentry>

diff --git a/src/backend/utils/error/elog.c b/src/backend/utils/error/elog.c
index 89aeacb5ffea7e0d454fd4e1805797a45dba1f24..52020544c36be97e52aba33b037abd86c8885eec 100644 (file)
--- a/src/backend/utils/error/elog.c
+++ b/src/backend/utils/error/elog.c
@@ -8,7 +8,7 @@
  *
  *
  * IDENTIFICATION
- *       $Header: /cvsroot/pgsql/src/backend/utils/error/elog.c,v 1.66 2000/11/13 21:35:02 momjian Exp $
+ *       $Header: /cvsroot/pgsql/src/backend/utils/error/elog.c,v 1.67 2000/11/14 19:13:27 petere Exp $
  *
  *-------------------------------------------------------------------------
  */
@@ -58,8 +58,8 @@ extern CommandDest whereToSendOutput;
  * ... in theory anyway
  */
 int Use_syslog = 0;
-char *Syslog_facility = "LOCAL0";
-char *Syslog_progid = "postgres";
+char *Syslog_facility;
+char *Syslog_ident;
 
 static void write_syslog(int level, const char *line);
 
@@ -646,7 +646,7 @@ write_syslog(int level, const char *line)
                        syslog_fac = LOG_LOCAL6;
                if (strcasecmp(Syslog_facility,"LOCAL7") == 0)
                        syslog_fac = LOG_LOCAL7;
-               openlog(Syslog_progid, LOG_PID | LOG_NDELAY, syslog_fac);
+               openlog(Syslog_ident, LOG_PID | LOG_NDELAY, syslog_fac);
                openlog_done = true;
        }
 

diff --git a/src/backend/utils/misc/guc.c b/src/backend/utils/misc/guc.c
index 1ce8186b7cb44600260e52c523e052caafc6ee07..8598051c6c7777ab5108745f0e514e02fc2de063 100644 (file)
--- a/src/backend/utils/misc/guc.c
+++ b/src/backend/utils/misc/guc.c
@@ -4,7 +4,7 @@
  * Support for grand unified configuration scheme, including SET
  * command, configuration file, and command line options.
  *
- * $Header: /cvsroot/pgsql/src/backend/utils/misc/guc.c,v 1.19 2000/11/14 01:15:02 momjian Exp $
+ * $Header: /cvsroot/pgsql/src/backend/utils/misc/guc.c,v 1.20 2000/11/14 19:13:27 petere Exp $
  *
  * Copyright 2000 by PostgreSQL Global Development Group
  * Written by Peter Eisentraut <peter_e@gmx.net>.
@@ -41,8 +41,8 @@ extern int XLOGbuffers;
 extern int XLOG_DEBUG;
 #ifdef ENABLE_SYSLOG
 extern char *Syslog_facility;
-extern char *Syslog_progid;
-       bool check_facility(const char *facility);
+extern char *Syslog_ident;
+bool check_facility(const char *facility);
 #endif
 
 /*
@@ -308,10 +308,11 @@ ConfigureNamesString[] =
 
        {"unix_socket_group",         PGC_POSTMASTER,       &Unix_socket_group,
         "", NULL},
+
 #ifdef ENABLE_SYSLOG
-       {"syslog_facility",           PGC_SIGHUP,           &Syslog_facility, 
+       {"syslog_facility",           PGC_POSTMASTER,       &Syslog_facility, 
        "LOCAL0", check_facility},       
-       {"syslog_progid",             PGC_SIGHUP,           &Syslog_progid, 
+       {"syslog_ident",              PGC_POSTMASTER,       &Syslog_ident, 
        "postgres", NULL},       
 #endif
 
@@ -608,7 +609,7 @@ set_config_option(const char * name, const char * value, GucContext
                                bool boolval;
                 if (!parse_bool(value, &boolval))
                                {
-                                       elog(elevel, "Option '%s' requires a boolean value", name);
+                                       elog(elevel, "option '%s' requires a boolean value", name);
                                        return false;
                                }
                                if (DoIt)
@@ -629,12 +630,12 @@ set_config_option(const char * name, const char * value, GucContext
 
                 if (!parse_int(value, &intval))
                                {
-                    elog(elevel, "Option '%s' expects an integer value", name);
+                    elog(elevel, "option '%s' expects an integer value", name);
                                        return false;
                                }
                 if (intval < conf->min || intval > conf->max)
                                {
-                    elog(elevel, "Option '%s' value %d is outside"
+                    elog(elevel, "option '%s' value %d is outside"
                                                 " of permissible range [%d .. %d]",
                                                 name, intval, conf->min, conf->max);
                                        return false;
@@ -657,12 +658,12 @@ set_config_option(const char * name, const char * value, GucContext
 
                 if (!parse_real(value, &dval))
                                {
-                    elog(elevel, "Option '%s' expects a real number", name);
+                    elog(elevel, "option '%s' expects a real number", name);
                                        return false;
                                }
                 if (dval < conf->min || dval > conf->max)
                                {
-                    elog(elevel, "Option '%s' value %g is outside"
+                    elog(elevel, "option '%s' value %g is outside"
                                                 " of permissible range [%g .. %g]",
                                                 name, dval, conf->min, conf->max);
                                        return false;
@@ -683,7 +684,7 @@ set_config_option(const char * name, const char * value, GucContext
                        {
                                if (conf->parse_hook && !(conf->parse_hook)(value))
                                {
-                                       elog(elevel, "Option '%s' rejects value '%s'", name, value);
+                                       elog(elevel, "invalid value for option '%s': '%s'", name, value);
                                        return false;
                                }
                                if (DoIt)
@@ -824,6 +825,9 @@ ParseLongOption(const char * string, char ** name, char ** value)
                if (*cp == '-')
                        *cp = '_';
 }
+
+
+
 #ifdef ENABLE_SYSLOG
 bool 
 check_facility(const char *facility)