Bug#522737: #kolab-cyrus-pop3d: Patch to limit POP3 mailbox size (e.g. for mobile

[Sven Mueller]

This patch (thanks for that!) seems to implement a useful feature at
first glance.

However, I find that it is (in its current form) unacceptable for
inclusion in the cyrus-pop3d-2.2 package (or the -2.3 package that is
currently worked on) for two reasons:

1) It implements a new behaviour/feature and should thus be discussed
with upstream.
2) It is really a hack, especially since it can't be disabled. This
means it would conflict with otherwise perfectly legal behaviour of
having logins named like user-001 user-002 etc. So if you would want
this to be included, you (or someone else) would have to make sure the
feature could be disabled by a configuration file switch.

I suspect that the latter would also be asked for by upstream if they
were to include the patch.

The pkg-cyrus-imapd team is in general willing to include patches if
they (in no particular order, especially not importance):
1) are as non-intrusive as possible (which is almost true for this patch)
2) Don't change behaviour in non-compatible ways (which this one does,
unfortunately) or can be enabled (i.e. default is disabled) by a
configuration switch (which might be relatively easy for this patch)
3) Have proven to work for production environments at two (at the very
least) different sites.
Exceptions can be made if there is good reason to of course.

Regards,
Sven

PS: This is just my personal opinion, not the official position of the
cyrus-imapd packaging team.
-------------- next part --------------
An HTML attachment was scrubbed...
URL: <http://lists.alioth.debian.org/pipermail/pkg-cyrus-imapd-debian-devel/attachments/20090409/0cfb3280/attachment.htm>
-------------- next part --------------
A non-text attachment was scrubbed...
Name: signature.asc
Type: application/pgp-signature
Size: 249 bytes
Desc: OpenPGP digital signature
URL: <http://lists.alioth.debian.org/pipermail/pkg-cyrus-imapd-debian-devel/attachments/20090409/0cfb3280/attachment.pgp>