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.

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.

bgpd/bgp_main.c

@@ -173,7 +173,7 @@ sighup (void)
 void
 sigint (void)
 {
-  zlog (NULL, LOG_INFO, "Terminating on signal");
+  zlog_notice ("Terminating on signal");
 
   if (! retain_mode)
     bgp_terminate ();
@@ -305,8 +305,8 @@ main (int argc, char **argv)
   vty_serv_sock (vty_addr, vty_port, BGP_VTYSH_PATH);
 
   /* Print banner. */
-  zlog_info ("BGPd %s starting: vty@%d, bgp@%d", QUAGGA_VERSION,
+  zlog_notice ("BGPd %s starting: vty@%d, bgp@%d", QUAGGA_VERSION,
-	     vty_port, bm->port);
+	       vty_port, bm->port);
 
   /* Start finite state machine, here we go! */
   while (thread_fetch (master, &thread))

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.

isisd/isis_main.c

@@ -143,7 +143,7 @@ reload ()
   execve (_progpath, _argv, _envp);
 }
 
-void
+static void
 terminate (int i)
 {
   exit (i);
@@ -165,16 +165,14 @@ sighup (void)
 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 @@ main (int argc, char **argv, char **envp)
   vty_serv_sock (vty_addr, vty_port, ISIS_VTYSH_PATH);
 
   /* Print banner. */
-#if defined(ZEBRA_VERSION)
+  zlog_notice ("Quagga-ISISd %s starting: vty@%d", QUAGGA_VERSION, vty_port);
-  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
 #ifdef HAVE_IPV6
   zlog_info ("IPv6 enabled");
 #endif

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

lib/pid_output.c

@@ -47,12 +47,14 @@ pid_output (const char *path)
   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_output_lock (const char *path)
   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)
         {

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);

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 *.

ospf6d/ospf6_main.c

@@ -133,7 +133,7 @@ sighup (void)
 void
 sigint (void)
 {
-  zlog_info ("SIGINT received");
+  zlog_notice ("Terminating on signal SIGINT");
   exit (0);
 }
 
@@ -141,7 +141,7 @@ sigint (void)
 void
 sigterm (void)
 {
-  zlog_info ("SIGTERM received");
+  zlog_notice ("Terminating on signal SIGTERM");
   exit (0);
 }
 
@@ -281,19 +281,16 @@ main (int argc, char *argv[], char *envp[])
     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,
+  if (!vty_port)
-                 vty_port ? vty_port : OSPF6_VTY_PORT, OSPF6_VTYSH_PATH);
+    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",
+  zlog_notice ("OSPF6d (Quagga-%s ospf6d-%s) starts: vty@%d",
-               QUAGGA_VERSION, OSPF6_DAEMON_VERSION);
+               QUAGGA_VERSION, OSPF6_DAEMON_VERSION,vty_port);
 
   /* Start finite state machine, here we go! */
   while (thread_fetch (master, &thread))

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

ospfd/ospf_main.c

@@ -141,7 +141,7 @@ sighup (void)
 void
 sigint (void)
 {
-  zlog (NULL, LOG_INFO, "Terminating on signal");
+  zlog_notice ("Terminating on signal");
 
   ospf_terminate ();
 
@@ -317,10 +317,8 @@ main (int argc, char **argv)
   /* 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);
+  zlog_notice ("OSPFd %s starting: vty@%d", QUAGGA_VERSION, vty_port);
-#endif
 
   /* Fetch next active thread. */
   while (thread_fetch (master, &thread))

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.

ripd/rip_main.c

@@ -144,7 +144,7 @@ sighup (void)
 void
 sigint (void)
 {
-  zlog (NULL, LOG_INFO, "Terminating on signal");
+  zlog_notice ("Terminating on signal");
 
   if (! retain_mode)
     rip_clean ();
@@ -290,6 +290,9 @@ main (int argc, char **argv)
   /* 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);

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.

ripngd/ripng_main.c

@@ -133,7 +133,6 @@ sighup (void)
   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 @@ sighup (void)
 void
 sigint (void)
 {
-  zlog_info ("Terminating on signal");
+  zlog_notice ("Terminating on signal");
 
   if (! retain_mode)
     ripng_clean ();
@@ -292,6 +291,9 @@ main (int argc, char **argv)
   /* 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);

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

zebra/main.c

@@ -165,7 +165,7 @@ sigint (void)
   /* 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 @@ main (int argc, char **argv)
   /* 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);