[Python-checkins] r76056 - in python/branches/release31-maint: Lib/mailbox.py Lib/test/test_mailbox.py Misc/NEWS
antoine.pitrou
python-checkins at python.org
Mon Nov 2 12:40:19 CET 2009
Author: antoine.pitrou
Date: Mon Nov 2 12:40:19 2009
New Revision: 76056
Log:
Merged revisions 76055 via svnmerge from
svn+ssh://pythondev@svn.python.org/python/branches/py3k
................
r76055 | antoine.pitrou | 2009-11-02 12:36:51 +0100 (lun., 02 nov. 2009) | 13 lines
Merged revisions 76034,76054 via svnmerge from
svn+ssh://pythondev@svn.python.org/python/trunk
........
r76034 | antoine.pitrou | 2009-11-01 22:29:33 +0100 (dim., 01 nov. 2009) | 3 lines
This should finally fix #6896. Let's watch the buildbots.
........
r76054 | antoine.pitrou | 2009-11-02 12:34:27 +0100 (lun., 02 nov. 2009) | 3 lines
Since r76034 was successful, add a NEWS entry for it.
........
................
Modified:
python/branches/release31-maint/ (props changed)
python/branches/release31-maint/Lib/mailbox.py
python/branches/release31-maint/Lib/test/test_mailbox.py
python/branches/release31-maint/Misc/NEWS
Modified: python/branches/release31-maint/Lib/mailbox.py
==============================================================================
--- python/branches/release31-maint/Lib/mailbox.py (original)
+++ python/branches/release31-maint/Lib/mailbox.py Mon Nov 2 12:40:19 2009
@@ -234,6 +234,9 @@
raise NoSuchMailboxError(self._path)
self._toc = {}
self._last_read = None # Records last time we read cur/new
+ # NOTE: we manually invalidate _last_read each time we do any
+ # modifications ourselves, otherwise we might get tripped up by
+ # bogus mtime behaviour on some systems (see issue #6896).
def add(self, message):
"""Add message and return assigned key."""
@@ -267,11 +270,15 @@
raise
if isinstance(message, MaildirMessage):
os.utime(dest, (os.path.getatime(dest), message.get_date()))
+ # Invalidate cached toc
+ self._last_read = None
return uniq
def remove(self, key):
"""Remove the keyed message; raise KeyError if it doesn't exist."""
os.remove(os.path.join(self._path, self._lookup(key)))
+ # Invalidate cached toc (only on success)
+ self._last_read = None
def discard(self, key):
"""If the keyed message exists, remove it."""
@@ -306,6 +313,8 @@
if isinstance(message, MaildirMessage):
os.utime(new_path, (os.path.getatime(new_path),
message.get_date()))
+ # Invalidate cached toc
+ self._last_read = None
def get_message(self, key):
"""Return a Message representation or raise a KeyError."""
@@ -360,7 +369,9 @@
def flush(self):
"""Write any pending changes to disk."""
- return # Maildir changes are always written immediately.
+ # Maildir changes are always written immediately, so there's nothing
+ # to do except invalidate our cached toc.
+ self._last_read = None
def lock(self):
"""Lock the mailbox."""
Modified: python/branches/release31-maint/Lib/test/test_mailbox.py
==============================================================================
--- python/branches/release31-maint/Lib/test/test_mailbox.py (original)
+++ python/branches/release31-maint/Lib/test/test_mailbox.py Mon Nov 2 12:40:19 2009
@@ -673,6 +673,9 @@
self.assertEqual(self._box._lookup(key0), os.path.join('new', key0))
os.remove(os.path.join(self._path, 'new', key0))
self.assertEqual(self._box._toc, {key0: os.path.join('new', key0)})
+ # Be sure that the TOC is read back from disk (see issue #6896
+ # about bad mtime behaviour on some systems).
+ self._box.flush()
self.assertRaises(KeyError, lambda: self._box._lookup(key0))
self.assertEqual(self._box._toc, {})
Modified: python/branches/release31-maint/Misc/NEWS
==============================================================================
--- python/branches/release31-maint/Misc/NEWS (original)
+++ python/branches/release31-maint/Misc/NEWS Mon Nov 2 12:40:19 2009
@@ -40,6 +40,10 @@
Library
-------
+- Issue #6896: mailbox.Maildir now invalidates its internal cache each time
+ a modification is done through it. This fixes inconsistencies and test
+ failures on systems with slightly bogus mtime behaviour.
+
- Issue #6665: Fix fnmatch to properly match filenames with newlines in them.
- Issue #7246 & Issue #7208: getpass now properly flushes input before
More information about the Python-checkins
mailing list