2004-12-03 Andrew J. Schorr <ajschorr@alumni.princeton.edu>

	* bgp_main.c: (sigint) Use zlog_notice for termination message.
	  (main) Use zlog_notice for startup announcement.
	* isis_main.c: (sigint,sigterm) Use zlog_notice for termination message.
	  (terminate) This function should be static, not global.
	  (main) Use zlog_notice for startup announcement, and remove
	  ifdef ZEBRA_VERSION.
	* version.h.in: Remove declaration for pid_output_lock, this function
	  is now static, not global.
	* pid_output.c: (pid_output_lock) This function should be static, not
	  global.  And remove "old umask" error message, since it was really
	  an unimportant debug message, not an error.
	  (pid_output) Need to declare static function pid_output_lock.
	* ospf6_main.c: (sigint,sigterm) Use zlog_notice for termination
	  message.
	  (main) Remove commented-out call to pid_output_lock (which should
	  never be called other than from inside pid_output).  And use
	  zlog_notice to print the startup message, which now includes
	  the vty port.
	* ospf_main.c: (sigint) Use zlog_notice for termination message.
	  (main) Issue a startup announcement using zlog_notice.
	* rip_main.c: (sigint) Use zlog_notice for termination message.
	  (main) Add a startup announcement using zlog_notice.
	* ripng_main.c: (sighup) Remove spurious terminating message.
	  (sigint) Use zlog_notice for termination message.
	  (main) Issue a startup announcement using zlog_notice.
	* main.c: (sigint) Use zlog_notice for termination message.
	  (main) Add a startup announcement using zlog_notice.
diff --git a/bgpd/ChangeLog b/bgpd/ChangeLog
index 64b229c..c76627b 100644
--- a/bgpd/ChangeLog
+++ b/bgpd/ChangeLog
@@ -1,3 +1,8 @@
+2004-12-03 Andrew J. Schorr <ajschorr@alumni.princeton.edu>
+
+	* bgp_main.c: (sigint) Use zlog_notice for termination message.
+	  (main) Use zlog_notice for startup announcement.
+
 2004-11-25 Hasso Tepper <hasso at quagga.net>
 
 	* bgp_main.c: Make group to run as configurable.
diff --git a/bgpd/bgp_main.c b/bgpd/bgp_main.c
index 1d6710f..ee498b0 100644
--- a/bgpd/bgp_main.c
+++ b/bgpd/bgp_main.c
@@ -173,7 +173,7 @@
 void
 sigint (void)
 {
-  zlog (NULL, LOG_INFO, "Terminating on signal");
+  zlog_notice ("Terminating on signal");
 
   if (! retain_mode)
     bgp_terminate ();
@@ -305,8 +305,8 @@
   vty_serv_sock (vty_addr, vty_port, BGP_VTYSH_PATH);
 
   /* Print banner. */
-  zlog_info ("BGPd %s starting: vty@%d, bgp@%d", QUAGGA_VERSION,
-	     vty_port, bm->port);
+  zlog_notice ("BGPd %s starting: vty@%d, bgp@%d", QUAGGA_VERSION,
+	       vty_port, bm->port);
 
   /* Start finite state machine, here we go! */
   while (thread_fetch (master, &thread))
diff --git a/isisd/ChangeLog b/isisd/ChangeLog
index 9ba86f1..e241d54 100644
--- a/isisd/ChangeLog
+++ b/isisd/ChangeLog
@@ -1,3 +1,10 @@
+2004-12-03 Andrew J. Schorr <ajschorr@alumni.princeton.edu>
+
+	* isis_main.c: (sigint,sigterm) Use zlog_notice for termination message.
+	  (terminate) This function should be static, not global.
+	  (main) Use zlog_notice for startup announcement, and remove
+	  ifdef ZEBRA_VERSION.
+
 2004-11-25 Hasso Tepper <hasso at quagga.net>
 
 	* isis_main.c: Make group to run as configurable.
diff --git a/isisd/isis_main.c b/isisd/isis_main.c
index 1bf93f2..9aa487a 100644
--- a/isisd/isis_main.c
+++ b/isisd/isis_main.c
@@ -143,7 +143,7 @@
   execve (_progpath, _argv, _envp);
 }
 
-void
+static void
 terminate (int i)
 {
   exit (i);
@@ -165,16 +165,14 @@
 void
 sigint (void)
 {
-  zlog_info ("SIGINT received");
+  zlog_notice ("Terminating on signal SIGINT");
   terminate (0);
-
-  return;
 }
 
 void
 sigterm (void)
 {
-  zlog_info ("SIGTERM received");
+  zlog_notice ("Terminating on signal SIGTERM");
   terminate (0);
 }
 
@@ -326,11 +324,7 @@
   vty_serv_sock (vty_addr, vty_port, ISIS_VTYSH_PATH);
 
   /* Print banner. */
-#if defined(ZEBRA_VERSION)
-  zlog_info ("ISISd %s starting: vty@%d", ZEBRA_VERSION, vty_port);
-#elif defined(QUAGGA_VERSION)
-  zlog_info ("Quagga-ISISd %s starting: vty@%d", QUAGGA_VERSION, vty_port);
-#endif
+  zlog_notice ("Quagga-ISISd %s starting: vty@%d", QUAGGA_VERSION, vty_port);
 #ifdef HAVE_IPV6
   zlog_info ("IPv6 enabled");
 #endif
diff --git a/lib/ChangeLog b/lib/ChangeLog
index 8dde0eb..84a578d 100644
--- a/lib/ChangeLog
+++ b/lib/ChangeLog
@@ -1,3 +1,12 @@
+2004-12-03 Andrew J. Schorr <ajschorr@alumni.princeton.edu>
+
+	* version.h.in: Remove declaration for pid_output_lock, this function
+	  is now static, not global.
+	* pid_output.c: (pid_output_lock) This function should be static, not
+	  global.  And remove "old umask" error message, since it was really
+	  an unimportant debug message, not an error.
+	  (pid_output) Need to declare static function pid_output_lock.
+
 2004-11-28 Andrew J. Schorr <ajschorr@alumni.princeton.edu>
 
 	* log.h: Remove several unused fields from struct zlog.  Add comments
diff --git a/lib/pid_output.c b/lib/pid_output.c
index ae3393c..11e1243 100644
--- a/lib/pid_output.c
+++ b/lib/pid_output.c
@@ -47,12 +47,14 @@
   umask(oldumask);
   return pid;
 #else
+  static pid_t pid_output_lock (const char *);
+
   return pid_output_lock(path);
 #endif /* HAVE_FCNTL */
 }
 
 #ifdef HAVE_FCNTL
-pid_t
+static pid_t
 pid_output_lock (const char *path)
 {
   int tmp;
@@ -65,7 +67,6 @@
   pid = getpid ();
 
   oldumask = umask(0777 & ~LOGFILE_MASK);
-  zlog_err( "old umask %d %d", oldumask, 0777 & ~LOGFILE_MASK);
   fd = open (path, O_RDWR | O_CREAT, LOGFILE_MASK);
       if (fd < 0)
         {
diff --git a/lib/version.h.in b/lib/version.h.in
index 9a707d4..cc7c6ca 100644
--- a/lib/version.h.in
+++ b/lib/version.h.in
@@ -35,7 +35,6 @@
 #define QUAGGA_COPYRIGHT "Copyright 1996-2004 Kunihiro Ishiguro, et al."
 
 pid_t pid_output (const char *);
-pid_t pid_output_lock (const char *);
 
 #ifndef HAVE_DAEMON
 int daemon(int, int);
diff --git a/ospf6d/ChangeLog b/ospf6d/ChangeLog
index b5b5294..7bb6b4b 100644
--- a/ospf6d/ChangeLog
+++ b/ospf6d/ChangeLog
@@ -1,3 +1,12 @@
+2004-12-03 Andrew J. Schorr <ajschorr@alumni.princeton.edu>
+
+	* ospf6_main.c: (sigint,sigterm) Use zlog_notice for termination
+	  message.
+	  (main) Remove commented-out call to pid_output_lock (which should
+	  never be called other than from inside pid_output).  And use
+	  zlog_notice to print the startup message, which now includes
+	  the vty port.
+
 2004-11-26 Hasso Tepper <hasso at quagga.net>
 
 	* ospf6_snmp.c: Fix compile: listnode -> struct listnode *.
diff --git a/ospf6d/ospf6_main.c b/ospf6d/ospf6_main.c
index 4563ece..ba266d9 100644
--- a/ospf6d/ospf6_main.c
+++ b/ospf6d/ospf6_main.c
@@ -133,7 +133,7 @@
 void
 sigint (void)
 {
-  zlog_info ("SIGINT received");
+  zlog_notice ("Terminating on signal SIGINT");
   exit (0);
 }
 
@@ -141,7 +141,7 @@
 void
 sigterm (void)
 {
-  zlog_info ("SIGTERM received");
+  zlog_notice ("Terminating on signal SIGTERM");
   exit (0);
 }
 
@@ -281,19 +281,16 @@
     daemon (0, 0);
 
   /* pid file create */
-#if 0
-  pid_output_lock (pid_file);
-#else
   pid_output (pid_file);
-#endif
 
   /* Make ospf6 vty socket. */
-  vty_serv_sock (vty_addr,
-                 vty_port ? vty_port : OSPF6_VTY_PORT, OSPF6_VTYSH_PATH);
+  if (!vty_port)
+    vty_port = OSPF6_VTY_PORT;
+  vty_serv_sock (vty_addr, vty_port, OSPF6_VTYSH_PATH);
 
   /* Print start message */
-  zlog_notice ("OSPF6d (Quagga-%s ospf6d-%s) starts",
-               QUAGGA_VERSION, OSPF6_DAEMON_VERSION);
+  zlog_notice ("OSPF6d (Quagga-%s ospf6d-%s) starts: vty@%d",
+               QUAGGA_VERSION, OSPF6_DAEMON_VERSION,vty_port);
 
   /* Start finite state machine, here we go! */
   while (thread_fetch (master, &thread))
diff --git a/ospfd/ChangeLog b/ospfd/ChangeLog
index 92651f2..ff54a50 100644
--- a/ospfd/ChangeLog
+++ b/ospfd/ChangeLog
@@ -1,3 +1,8 @@
+2004-12-03 Andrew J. Schorr <ajschorr@alumni.princeton.edu>
+
+	* ospf_main.c: (sigint) Use zlog_notice for termination message.
+	  (main) Issue a startup announcement using zlog_notice.
+
 2004-11-30 Andrew J. Schorr <ajschorr@alumni.princeton.edu>
 
 	* ospf_packet.c: (ospf_db_desc_proc) Fix spelling of packet in warning
diff --git a/ospfd/ospf_main.c b/ospfd/ospf_main.c
index 09ff197..d155c98 100644
--- a/ospfd/ospf_main.c
+++ b/ospfd/ospf_main.c
@@ -141,7 +141,7 @@
 void
 sigint (void)
 {
-  zlog (NULL, LOG_INFO, "Terminating on signal");
+  zlog_notice ("Terminating on signal");
 
   ospf_terminate ();
 
@@ -317,10 +317,8 @@
   /* Create VTY socket */
   vty_serv_sock (vty_addr, vty_port, OSPF_VTYSH_PATH);
 
-#ifdef DEBUG
   /* Print banner. */
-  zlog (NULL, LOG_INFO, "OSPFd (%s) starts", QUAGGA_VERSION);
-#endif
+  zlog_notice ("OSPFd %s starting: vty@%d", QUAGGA_VERSION, vty_port);
 
   /* Fetch next active thread. */
   while (thread_fetch (master, &thread))
diff --git a/ripd/ChangeLog b/ripd/ChangeLog
index 3ca7d18..1c703df 100644
--- a/ripd/ChangeLog
+++ b/ripd/ChangeLog
@@ -1,3 +1,8 @@
+2004-12-03 Andrew J. Schorr <ajschorr@alumni.princeton.edu>
+
+	* rip_main.c: (sigint) Use zlog_notice for termination message.
+	  (main) Add a startup announcement using zlog_notice.
+
 2004-11-25 Hasso Tepper <hasso at quagga.net>
 
 	* rip_main.c: Make group to run as configurable.
diff --git a/ripd/rip_main.c b/ripd/rip_main.c
index d545a8e..2b2939d 100644
--- a/ripd/rip_main.c
+++ b/ripd/rip_main.c
@@ -144,7 +144,7 @@
 void
 sigint (void)
 {
-  zlog (NULL, LOG_INFO, "Terminating on signal");
+  zlog_notice ("Terminating on signal");
 
   if (! retain_mode)
     rip_clean ();
@@ -290,6 +290,9 @@
   /* Create VTY's socket */
   vty_serv_sock (vty_addr, vty_port, RIP_VTYSH_PATH);
 
+  /* Print banner. */
+  zlog_notice ("RIPd %s starting: vty@%d", QUAGGA_VERSION, vty_port);
+
   /* Execute each thread. */
   while (thread_fetch (master, &thread))
     thread_call (&thread);
diff --git a/ripngd/ChangeLog b/ripngd/ChangeLog
index 1abea56..840f413 100644
--- a/ripngd/ChangeLog
+++ b/ripngd/ChangeLog
@@ -1,3 +1,9 @@
+2004-12-03 Andrew J. Schorr <ajschorr@alumni.princeton.edu>
+
+	* ripng_main.c: (sighup) Remove spurious terminating message.
+	  (sigint) Use zlog_notice for termination message.
+	  (main) Issue a startup announcement using zlog_notice.
+
 2004-11-25 Hasso Tepper <hasso at quagga.net>
 
 	* ripng_main.c: Make group to run as configurable.
diff --git a/ripngd/ripng_main.c b/ripngd/ripng_main.c
index df609a5..ac63abb 100644
--- a/ripngd/ripng_main.c
+++ b/ripngd/ripng_main.c
@@ -133,7 +133,6 @@
   zlog_info ("SIGHUP received");
   ripng_clean ();
   ripng_reset ();
-  zlog_info ("Terminating on signal");
 
   /* Reload config file. */
   vty_read_config (config_file, config_default);
@@ -147,7 +146,7 @@
 void
 sigint (void)
 {
-  zlog_info ("Terminating on signal");
+  zlog_notice ("Terminating on signal");
 
   if (! retain_mode)
     ripng_clean ();
@@ -292,6 +291,9 @@
   /* Process id file create. */
   pid_output (pid_file);
 
+  /* Print banner. */
+  zlog_notice ("RIPNGd %s starting: vty@%d", QUAGGA_VERSION, vty_port);
+
   /* Fetch next active thread. */
   while (thread_fetch (master, &thread))
     thread_call (&thread);
diff --git a/zebra/ChangeLog b/zebra/ChangeLog
index 7193231..c38b47d 100644
--- a/zebra/ChangeLog
+++ b/zebra/ChangeLog
@@ -1,3 +1,8 @@
+2004-12-03 Andrew J. Schorr <ajschorr@alumni.princeton.edu>
+
+	* main.c: (sigint) Use zlog_notice for termination message.
+	  (main) Add a startup announcement using zlog_notice.
+
 2004-11-25 Hasso Tepper <hasso at quagga.net>
 
 	* irdp_interface.c: Remove comment from DEFUN, it breaks vtysh because
diff --git a/zebra/main.c b/zebra/main.c
index 42e6bd1..96420da 100644
--- a/zebra/main.c
+++ b/zebra/main.c
@@ -165,7 +165,7 @@
   /* Decrared in rib.c */
   void rib_close ();
 
-  zlog_info ("Terminating on signal");
+  zlog_notice ("Terminating on signal");
 
   if (!retain_mode)
     rib_close ();
@@ -372,6 +372,9 @@
   /* Make vty server socket. */
   vty_serv_sock (vty_addr, vty_port, ZEBRA_VTYSH_PATH);
 
+  /* Print banner. */
+  zlog_notice ("Zebra %s starting: vty@%d", QUAGGA_VERSION, vty_port);
+
   while (thread_fetch (zebrad.master, &thread))
     thread_call (&thread);