Julien Danjou wrote: > Package: archivemail > Version: 0.7.2-2 > Severity: serious > > Hi, > > First of all, I'm making this bug serious since it breaks the software > for me, but feel free to downgrade.
That's a very weak reason for making a bug RC, especally when the bug is
that archivemail fails to move mail out of an already empty mailbox. And
I think you know it..
My general experience with over-inflating severities is that it either leads
to an overreaction by the maintainer in the other direction and/or encourages
the maintainer to waste time thinking about the severity, and writing text
like this about it, rather than spending time thinking about the bug.
Perhaps you've had some other experience?
> Here's the problem:
>
> % mkdir -p mailbox-test/{cur,new,tmp}
> % archivemail -d 3 mailbox-test
> Traceback (most recent call last):
> File "/usr/bin/archivemail", line 1611, in <module>
> main()
> File "/usr/bin/archivemail", line 708, in main
> archive(mailbox_path)
> File "/usr/bin/archivemail", line 1162, in archive
> _archive_dir(mailbox_name, final_archive_name, "maildir")
> File "/usr/bin/archivemail", line 1282, in _archive_dir
> assert(original)
> AssertionError
>
> Well, looking at the code I guess that mailbox.Maildir(mailbox_name) is
> just giving nothing. I do not know if it's a bug or a change (maybe from
> python2.5), but this used to work for years for me.
The bug seems to be that under python 2.5, a mailbox.Maildir object that
is empty evaluates to a false value, while in previous versions of python,
the same object would evaluate to true:
if type == "maildir":
original = mailbox.Maildir(mailbox_name)
if original:
print "set"
else:
print "unset"
[EMAIL PROTECTED]:~>python2.4 =archivemail -d 3 mailbox-test
set
[EMAIL PROTECTED]:~>python2.5 =archivemail -d 3 mailbox-test
unset
I don't know enough python to say, but this looks rather like a bug or
incompatible change in python 2.5.
If so, this kind of change in python would certianly seem likely to break
lots of other stuff, so perhaps your severity isn't so inflated after all.
--
see shy jo
signature.asc
Description: Digital signature
