10 files changed, 12 insertions, 14 deletions

diff --git a/Mailman/Cgi/admin.py b/Mailman/Cgi/admin.py
index f91339a7a..ca2d0f08f 100644
--- a/Mailman/Cgi/admin.py
+++ b/Mailman/Cgi/admin.py
@@ -77,8 +77,8 @@ def main():
         mlist = MailList.MailList(listname, lock=0)
     except Errors.MMListError, e:
         admin_overview(_('No such list <em>%(listname)s</em>'))
-        syslog('error', 'Someone tried to access the admin interface for a '
-               'non-existent list: %s' % listname)
+        syslog('error', 'admin.py access for non-existent list: %s',
+               listname)
         return
     # Now that we know what list has been requested, all subsequent admin
     # pages are shown in that list's preferred language.
diff --git a/Mailman/Cgi/admindb.py b/Mailman/Cgi/admindb.py
index 4952b32ee..c476e08c1 100644
--- a/Mailman/Cgi/admindb.py
+++ b/Mailman/Cgi/admindb.py
@@ -58,7 +58,7 @@ def main():
         mlist = MailList.MailList(listname, lock=0)
     except Errors.MMListError, e:
         handle_no_list(doc, _('No such list <em>%(listname)s</em>'))
-        syslog('error', 'No such list "%s": %s\n' % (listname, e))
+        syslog('error', 'No such list "%s": %s\n', listname, e)
         return
 
     # Now that we know which list to use, set the system's language to it.
diff --git a/Mailman/Cgi/confirm.py b/Mailman/Cgi/confirm.py
index 84e1acc03..927331285 100644
--- a/Mailman/Cgi/confirm.py
+++ b/Mailman/Cgi/confirm.py
@@ -50,7 +50,7 @@ def main():
         bad_confirmation(doc, _('No such list <em>%(listname)s</em>'))
         doc.AddItem(MailmanLogo())
         print doc.Format()
-        syslog('error', 'No such list "%s": %s' % (listname, e))
+        syslog('error', 'No such list "%s": %s', listname, e)
         return
 
     # Set the language for the list
diff --git a/Mailman/Cgi/create.py b/Mailman/Cgi/create.py
index f344e3ab5..70c0d75d1 100644
--- a/Mailman/Cgi/create.py
+++ b/Mailman/Cgi/create.py
@@ -48,7 +48,7 @@ def main():
         doc.SetTitle(title)
         doc.AddItem(
             Header(3, Bold(FontAttr(title, color='#ff0000', size='+2'))))
-        syslog('error', 'Bad URL specification: %s' % parts)
+        syslog('error', 'Bad URL specification: %s', parts)
     elif cgidata.has_key('doit'):
         # We must be processing the list creation request
         process_request(doc, cgidata)
diff --git a/Mailman/Cgi/edithtml.py b/Mailman/Cgi/edithtml.py
index 1a4266b70..7268d899b 100644
--- a/Mailman/Cgi/edithtml.py
+++ b/Mailman/Cgi/edithtml.py
@@ -18,7 +18,6 @@
 
 import os
 import cgi
-import gettext
 
 from Mailman import Utils
 from Mailman import MailList
@@ -63,7 +62,7 @@ def main():
     except Errors.MMListError, e:
         doc.AddItem(Header(2, _('No such list <em>%(listname)s</em>')))
         print doc.Format()
-        syslog('error', _('No such list "%(listname)s": %(e)s\n'))
+        syslog('error', 'No such list "%s": %s', listname, e)
         return
 
     # Now that we have a valid list, set the language to its default
diff --git a/Mailman/Cgi/listinfo.py b/Mailman/Cgi/listinfo.py
index 7dca72cc1..835f05eac 100644
--- a/Mailman/Cgi/listinfo.py
+++ b/Mailman/Cgi/listinfo.py
@@ -47,7 +47,7 @@ def main():
         mlist = MailList.MailList(listname, lock=0)
     except Errors.MMListError, e:
         listinfo_overview(_('No such list <em>%(listname)s</em>'))
-        syslog('error', 'No such list "%s": %s' % (listname, e))
+        syslog('error', 'No such list "%s": %s', listname, e)
         return
 
     # See if the user want to see this page in other language
diff --git a/Mailman/Cgi/options.py b/Mailman/Cgi/options.py
index 5626deafd..30abc4a58 100644
--- a/Mailman/Cgi/options.py
+++ b/Mailman/Cgi/options.py
@@ -65,7 +65,7 @@ def main():
         doc.AddItem('<hr>')
         doc.AddItem(MailmanLogo())
         print doc.Format()
-        syslog('error', 'No such list "%s": %s\n' % (listname, e))
+        syslog('error', 'No such list "%s": %s\n', listname, e)
         return
 
     # Now we know which list is requested, so we can set the language to the
diff --git a/Mailman/Cgi/private.py b/Mailman/Cgi/private.py
index 07de99201..c2c18cc84 100644
--- a/Mailman/Cgi/private.py
+++ b/Mailman/Cgi/private.py
@@ -97,7 +97,7 @@ def main():
         doc.SetTitle(_("Private Archive Error - %(msg)s"))
         doc.AddItem(Header(2, msg))
         print doc.Format()
-        syslog('error', 'No such list "%s": %s\n' % (listname, e))
+        syslog('error', 'No such list "%s": %s\n', listname, e)
         return
 
     i18n.set_language(mlist.preferred_language)
diff --git a/Mailman/Cgi/roster.py b/Mailman/Cgi/roster.py
index 17260986f..1d72ab470 100644
--- a/Mailman/Cgi/roster.py
+++ b/Mailman/Cgi/roster.py
@@ -53,7 +53,7 @@ def main():
         mlist = MailList.MailList(listname, lock=0)
     except Errors.MMListError, e:
         error_page(_('No such list <em>%(listname)s</em>'))
-        syslog('error', 'roster: no such list "%s": %s' % (listname, e))
+        syslog('error', 'roster: no such list "%s": %s', listname, e)
         return
 
     cgidata = cgi.FieldStorage()
diff --git a/Mailman/Cgi/subscribe.py b/Mailman/Cgi/subscribe.py
index c004e1cda..52630f768 100644
--- a/Mailman/Cgi/subscribe.py
+++ b/Mailman/Cgi/subscribe.py
@@ -55,7 +55,7 @@ def main():
         doc.AddItem(Header(2, _("Error")))
         doc.AddItem(Bold(_('No such list <em>%(listname)s</em>')))
         print doc.Format()
-        syslog('error', 'No such list "%s": %s\n' % (listname, e))
+        syslog('error', 'No such list "%s": %s\n', listname, e)
         return
 
     # See if the form data has a preferred language set, in which case, use it
@@ -159,8 +159,7 @@ def process_form(mlist, doc, cgidata, lang):
         else:
             remote = _("unidentified origin")
         badremote = "\n\tfrom " + remote
-        syslog("mischief", "Attempt to self subscribe %s:%s"
-               % (email, badremote))
+        syslog('mischief', 'Attempt to self subscribe %s:%s', email, badremote)
         results += _('You must not subscribe a list to itself!<br>')
 
     # If the user did not supply a password, generate one for him