logfile must be specified if pidfile is (sf bug 772820)

git-svn-id: http://svn.roundup-tracker.org/svnroot/roundup/trunk@1804 57a73879-2fb5-44c3-a270-3262357dd7e2

diff --git a/CHANGES.txt b/CHANGES.txt
index abfaf8c3c15a40a0dca86a645b9c43b2722aef9e..96595286fc00198728173cd9a92d0b6d985f49ae 100644 (file)
--- a/CHANGES.txt
+++ b/CHANGES.txt
@@ -5,6 +5,7 @@ are given with the most recent entry first.
 - Fixed editing attributes on FileClass nodes.
 - Query editing now works correctly (sf bug 621248)
 - roundup-server now logs IP addresses by default (sf bug 778795)
+- logfile must be specified if pidfile is (sf bug 772820)
 
 2003-07-29 0.6.0b4
 - plugged cross-site-scripting hole (thanks Jeff Epler)

diff --git a/roundup/scripts/roundup_server.py b/roundup/scripts/roundup_server.py
index e6dd1240b57cc7448778046651ebd4d537d312f9..1575f418f94522a2978be811a5675c073b6dc778 100644 (file)
--- a/roundup/scripts/roundup_server.py
+++ b/roundup/scripts/roundup_server.py
@@ -16,7 +16,7 @@
 # 
 """ HTTP Server that serves roundup.
 
-$Id: roundup_server.py,v 1.25 2003-08-12 01:11:43 richard Exp $
+$Id: roundup_server.py,v 1.26 2003-08-12 01:14:11 richard Exp $
 """
 
 # python version check
@@ -316,7 +316,7 @@ options:
  -l: sets a filename to log to (instead of stdout)
  -d: run the server in the background and on UN*X write the server's PID
      to the nominated file. Note: on Windows the PID argument is needed,
-     but ignored.
+     but ignored. The -l option *must* be specified if this option is.
  -N: log client machine names in access log instead of IP addresses (much
      slower)
 
@@ -405,6 +405,9 @@ def run():
             elif opt == '-h': usage()
             elif opt == '-N': RoundupRequestHandler.LOG_IPADDRESS = 0
 
+        if pidfile and not logfile:
+            raise ValueError, _("logfile *must* be specified if pidfile is")
+
         if hasattr(os, 'getuid'):
             # if root, setuid to the running user
             if not os.getuid() and user is not None: