Buchan Milne wrote:

Luca (Olivetti), I have just added this patch (or actually the more

I hope you did it to the latest (2.1.14-2mdk) rpm, not to 2.1.14-1mdk.

up-to-date one from Simon Matter's SRPM), which at present I have done
as another conditional (--with autocreate), even though I think it would
be safe to apply the patch (I will test both with and without the
directives in /etc/imapd.conf to see) by default.

I find this patch *extremely* dangerous, but I won't complain too loudly if the option is left disabled in the default config file. Note that it doesn't support murder (see http://email.uoa.gr/autocreate/)


Another issue is that there are some scripts (cpmsg, bsd2cyrus,folderxfer,batchreconstruct,inboxfer) in Simon Matter's cyrus-imapd for converting from mbox (ie UW-IMAP) format, which you don't have. Since we need to convert from mbox anyway, I would like to add this. Any objection? Should I make a subpackage (they seem to depend on formail, thus will depend on procmail, which would be a bit of a waste ;-)).

I'd prefer a *separate* package, for a couple of reasons, one because these are separate, stand alone utilities, and could have a different release cycle than cyrus (note that they aren't released as a tarball so it's difficult to track for new versions). The other is that there are a bazillion other utilities to convert from one mail format to another, mbox to imap or imap to imap, (e.g. I used something from UW imap utils), and it doesn't make sense to package all of these utilities in the same SRPM. Maybe batchreconstruct is the only one really cyrus specific, but I cannot comment since I didn't need it so far (in ~4 years of cyrus use).


Finally, there are a few other patches you don't have, any particular reason, or can they be added as well? cyrus-imapd-2.1.11-snmpargs.patch

the comment and the spec file say that it's only needed for redhat 6.2. Anyway snmp is disabled in Simon's (and my) package. If someone needs/is using snmp with cyrus he could give us some advice (on needed buildrequires, patches, etc.).

cyrus-imapd-2.1.14-ipv6-20030703.diff.gz

Don't use ipv6 myself so I cannot check if it's working correctly (I don't doubt the author's knowledge, but mine and/or strange interactions with linux and/or mandrake). I could test if it builds but nothing more. If it doesn't break anything (i.e. someone *really* test its functionality with the mandrake package) I'll have no problem in including it. But the same problem would arise with the next version of cyrus-imapd+ipv6 patches.

cyrus-imapd-2.1.3-flock.patch

IIRC (but I may be wrong) it wasn't the correct solution for the problem at hand and CMU folks were skeptical about this patch:

http://asg.web.cmu.edu/archive/message.php?mailbox=archive.info-cyrus&msg=15115

note that there's also cricisism about the munge8bit patch, and I'm not too happy to have it (but if the option is disabled it won't do too much harm).
A followup message says that recent kernels don't have this problem (and this was a year ago).

Bye
--
Que les importa a las viudas, a los hu�rfanos, a los desvalidos
si las masacres se hacen en nombre del totalitarismo o en el
sagrado nombre de la libertad y la democracia.
    Mahatma Gandhi (1869 - 1948)

Attachment: pgp00000.pgp
Description: PGP signature



Reply via email to