From d706409c595a5994371d2b238b2a5e77be171f13 Mon Sep 17 00:00:00 2001 From: Nikolaus Schulz Date: Tue, 3 Mar 2009 04:33:09 +0100 Subject: Switch mbox locking from flock(2) to posix lockf(2) flock() locks aren't portable; lockf() locks are. --- TODO | 6 ------ 1 file changed, 6 deletions(-) (limited to 'TODO') diff --git a/TODO b/TODO index fe8ee64..02af4e8 100644 --- a/TODO +++ b/TODO @@ -6,12 +6,6 @@ Gracefully close IMAP connection upon unexptected error (currently archivemail just terminates). LOCKING & Co: -* switch to or add fcntl() locking; when combining with flock, be careful not to - break on Solaris/FreeBSD, where flock() is just fcntl() so using both would - block. -* Ensure that we don't accidently lose fcntl locks when closing some file - descriptor; this applies even for flock, since again, it might be emulated - with fcntl * Block signals while writing changed mailbox back. Also, we probably shouldn't use shutil.copy2() for this; at least we have to handle symlinked targets in a sane way, see Debian bug #349068. mbox_sync_mailbox() in the mutt code might -- cgit v1.2.3