[ archivemail-Patches-1918937 ] recursively archive all mails from an imap server

[SourceForge.net <no...@so...>]

Patches item #1918937, was opened at 2008-03-19 00:36
Message generated for change (Comment added) made by nikosch
You can respond by visiting: 
https://sourceforge.net/tracker/?func=detail&atid=456912&aid=1918937&group_id=49630

Please note that this message will contain a full copy of the comment thread,
including the initial issue submission, for this request,
not just the latest update.
Category: None
Group: None
Status: Open
Resolution: None
Priority: 5
Private: No
Submitted By: Christian (chrisbra)
Assigned to: Nobody/Anonymous (nobody)
Summary: recursively archive all mails from an imap server

Initial Comment:
Hi,
I thought, it would be nice, if archivemail could automatically archive messages from all folders in an imap server. This could for example be used for migrating all mails from one imap-server to another server.

Attached is my first try to achieve this. I am not sure, how stable it is, especially the naming of the final archive could probably be improved and I am not sure how well this works with other imap servers beside cyrus, with which I've tested it.

Greetings,
C.Brabandt

----------------------------------------------------------------------

>Comment By: Nikolaus Schulz (nikosch)
Date: 2008-03-19 21:02

Message:
Logged In: YES 
user_id=1594781
Originator: NO

Hm, okay, I see that this can be convenient. I'm not convinced that it's
worth adding more complexity and yet another option to archivemail,
though... I would expect that typically, mailboxes are pretty static
objects; you don't create them often or move them around a lot, so the
benefit of recursive archiving seems pretty small. 

By the way, this has already been requested a long time ago before I was
involved with archivemail, but the tracker was closed by the maintainer for
some reason, see Feature request #649607. 

Finally, can you please explain the following line from your patch? 

final_archive_name = "INBOX" + folder.replace("INBOX","") +
o_fan.replace("INBOX","")

First, this looks like you're assuming that all folders are below INBOX,
which need not be the case. Also, one would have to catch if 'folder' in
the assignment contains slashes. But leaving that aside, I don't understand
it. :-) 



----------------------------------------------------------------------

You can respond by visiting: 
https://sourceforge.net/tracker/?func=detail&atid=456912&aid=1918937&group_id=49630