-----BEGIN PGP SIGNED MESSAGE-----
Hash: SHA1

Mark Sapiro wrote:

| What I suggest for the moment is to just delete lines 594-603 of
| Mailman/MailList.py which includes the following 4 lines and the
| preceding comment.
|
|>>             mtime = os.path.getmtime(dbfile)
|>>             if mtime <= self.__timestamp:
|>>                 # File is not newer
|>>                 return None, None


Ooops!. That won't quite do it. you can't just remove the

~             mtime = os.path.getmtime(dbfile)

line as mtime is referenced later and will be undefined. A patch is
attached which has been applied and tested. I ran the test script twice
with this patch installed with no lost subscribers. It isn't the final
patch which should remove __timestamp completely and won't remove all
the comment, but it will do as a workaround.

- --
Mark Sapiro <[EMAIL PROTECTED]>        The highway is for gamblers,
San Francisco Bay Area, California    better use your sense - B. Dylan

-----BEGIN PGP SIGNATURE-----
Version: GnuPG v1.4.7 (MingW32)

iD8DBQFImSKDVVuXXpU7hpMRArrKAJ9K2fA8MzI7Iarp0WfnNClwtwVbdACfaxCO
Xsx7ru3fK2/v5ZRw0YHYU9s=
=BdZw
-----END PGP SIGNATURE-----
--- Mailman/MailList.py 2008-06-30 09:29:46.000000000 -0700
+++ Mailman/MailList.py 2008-08-05 20:43:19.000000000 -0700
@@ -591,16 +591,7 @@
         else:
             assert 0, 'Bad database file name'
         try:
-            # Check the mod time of the file first.  If it matches our
-            # timestamp, then the state hasn't change since the last time we
-            # loaded it.  Otherwise open the file for loading, below.  If the
-            # file doesn't exist, we'll get an EnvironmentError with errno set
-            # to ENOENT (EnvironmentError is the base class of IOError and
-            # OSError).
             mtime = os.path.getmtime(dbfile)
-            if mtime <= self.__timestamp:
-                # File is not newer
-                return None, None
             fp = open(dbfile)
         except EnvironmentError, e:
             if e.errno <> errno.ENOENT: raise

Attachment: MailList.patch.txt.sig
Description: Binary data

_______________________________________________
Mailman-Developers mailing list
[email protected]
http://mail.python.org/mailman/listinfo/mailman-developers
Mailman FAQ: http://wiki.list.org/x/AgA3
Searchable Archives: 
http://www.mail-archive.com/mailman-developers%40python.org/
Unsubscribe: 
http://mail.python.org/mailman/options/mailman-developers/archive%40jab.org

Security Policy: http://wiki.list.org/x/QIA9

Reply via email to