mirror/archivemail
1
0
Fork 0
mirror of https://git.code.sf.net/p/archivemail/code synced 2024-12-22 07:42:55 +00:00
Code Issues Releases Wiki Activity

TODO: drop more obsolete items

Browse source 
In particular:
* we no longer use shutil.copy{,2} to write back a changed mbox
* having temporary mbox files in the same directory as the originals doesn't
  make sense anymore since we no longer commit them with rename(2)
* the --archive-name option is now implemented
This commit is contained in:
Nikolaus Schulz 2010-08-08 18:53:11 +02:00
parent ca80e185e6
commit 910b507a2d
1 changed files with 1 additions and 20 deletions
Show stats Download patch file Download diff file Expand all files Collapse all files

21
TODO
View file

@ -6,10 +6,7 @@ Gracefully close IMAP connection upon unexptected error (currently archivemail
just terminates). just terminates).
LOCKING & Co: LOCKING & Co:
* Block signals while writing changed mailbox back. Also, we probably shouldn't * Block signals while writing changed mailbox back.
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
be an example how to write back a changed mailbox.
* Double-check the entire locking code. * Double-check the entire locking code.
Seems like existing archives are not read or validated in any way. New archive Seems like existing archives are not read or validated in any way. New archive
@ -45,10 +42,6 @@ are mapped like this:
Implement --include-draft. But before, think about it again. (This is feature Implement --include-draft. But before, think about it again. (This is feature
request #1569305.) request #1569305.)
Create temporary archive mbox in /tmp only if we don't have write permissions in
the mbox directory. Currently, if /tmp resides on another filesystem, we have
to copy the entire box to its destination.
Implement a fallback if an IMAP server doesn't support SEARCH. (Ouch!) Implement a fallback if an IMAP server doesn't support SEARCH. (Ouch!)
Add IMAP tests to the testsuite (upload test messages with IMAP "APPEND Add IMAP tests to the testsuite (upload test messages with IMAP "APPEND
@ -77,15 +70,6 @@ Perhaps add some more nice stuff like printing of subject, sender...
See tracker #868714 "added stats option to archivemail", which has a point. See tracker #868714 "added stats option to archivemail", which has a point.
Message-Ids are useful for diagnosis, but not very nice to read for humans. Message-Ids are useful for diagnosis, but not very nice to read for humans.
Regarding the --archive-name option:
* Do we want this? Probably, it adds flexibility.
* I think we should expand date format strings like we do with --suffix
* Hmm, --output-dir overrides os.dirname(archive_name)...
If no output_dir is given, use $PWD like we do for IMAP, or require -o?
* Provide short option -a? Not sure.
* The patch in #905657 is not bad. The Debian package also has a custom
--archive-name option, but with a worse implementation.
Be a nicer citizen with respect to mailbox locking. Be a nicer citizen with respect to mailbox locking.
Perhaps prune/shorten IMAP mailbox URLs in messages? Perhaps prune/shorten IMAP mailbox URLs in messages?
@ -103,9 +87,6 @@ Check all items below, which are from the original author. :-)
When you get a file-not-found in the 6th mailbox of 10, it aborts the whole When you get a file-not-found in the 6th mailbox of 10, it aborts the whole
run. Better to fail gracefully and keep going. run. Better to fail gracefully and keep going.
Think about the best way to specify the names of archives created with
possibly an --archive-name option.
Add more tests (see top of test_archivemail) Add more tests (see top of test_archivemail)
We need some better checking to see if we are really looking at a valid We need some better checking to see if we are really looking at a valid