[ archivemail-Bugs-1783369 ] archivemail fails to see all messages

SourceForge Headquarters 225 Broadway Suite 1600 San Diego, CA 92101 +1 (858) 422-6466

Bugs item #1783369, was opened at 2007-08-28 16:12
Message generated for change (Comment added) made by nikosch
You can respond by visiting: 
https://sourceforge.net/tracker/?func=detail&atid=456910&aid=1783369&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: Closed
>Resolution: Fixed
Priority: 5
Private: No
Submitted By: Elan Ruusamäe (ahmake)
>Assigned to: Nikolaus Schulz (nikosch)
Summary: archivemail fails to see all messages

Initial Comment:
# archivemail -n Maildir/.Trash
 Maildir/.Trash:
 I would have archived 10 of 92 message(s) (55.0kB of 1.7MB) in 0.3 seconds

# find Maildir/.Trash/{cur,new} -type f | wc -l
 44777

# archivemail --version
archivemail v0.7.0

Copyright (C) 2002  Paul Rodger <pa...@pa...>
          (C) 2006  Peter Poeml <po...@su...>,
          (C) 2006  Nikolaus Schulz <mic...@we...>
This is free software; see the source for copying conditions. There is NO
warranty; not even for MERCHANTABILITY or FITNESS FOR A PARTICULAR PURPOSE.

using -v doesn't reveal any much more information why it happens so i'm not pasting it here.

please ask for more information as i'm not feeling comfortable of sharing my Trash folder contents for testing :)

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

>Comment By: Nikolaus Schulz (nikosch)
Date: 2007-10-24 04:35

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

Thanks for reporting this, and even more for tracking it down and for your
patch. I wonder how such invalid sprint into existance, but anyway, I can
find three bug reports which seem to be about this issue, so it affects
people in real life. :-) 
Your patch looks good to me, I've only slightly modified it. The fix is
now in subversion. 

Cheers, 
Nikolaus

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

Comment By: Elan Ruusamäe (ahmake)
Date: 2007-09-18 12:51

Message:
Logged In: YES 
user_id=289546
Originator: YES

looks like the loop was ended because this code (from mailbox.py) returned
False when message contained just \n (has it's header and body).

          fn = self.boxes.pop() 
          fp = open(fn) 
          return self.factory(fp)

http://cvs.pld-linux.org/cgi-bin/cvsweb.cgi/SOURCES/archivemail-invalid-message.patch
File Added: archivemail-invalid-message.patch

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

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