I support Sven here. And I also think that ugly patches like this should not try fix client side deficiencies.
BTW: S60 works just fine with IMAP I am for tagging this as wontfix. Ondrej. On Thu, Apr 9, 2009 at 16:12, Sven Mueller <[email protected]> wrote: > 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. > > Debian Bug report logs: Bugs submitted by [email protected] > > Note that people may use different email accounts for different bugs, so > there may be other reports filed under different addresses. > > - Outstanding bugs -- Normal bugs; Will Not > Fix<#1208b3f4f0c91770__0_4_0>(1 bug) > - Outstanding bugs -- Wishlist items; Patch > Available<#1208b3f4f0c91770__0_6_2>(1 bug) > - Outstanding bugs -- Wishlist items; > Unclassified<#1208b3f4f0c91770__0_6_4>(3 bugs) > - Resolved bugs -- Wishlist items <#1208b3f4f0c91770__4_6_5> (1 bug) > > Outstanding bugs -- Normal bugs; Will Not Fix (1 bug) > > - #242428 <http://bugreport.cgi?bug=242428> [n|☹| ] > [cyrus21-common<http://pkgreport.cgi?package=cyrus21-common>] > cyrus21-common: "mbpath" utility should not need write access to > mailboxes.db <http://bugreport.cgi?bug=242428> Reported by: Tor > Slettnes <[email protected]><http://pkgreport.cgi?submitter=tor%40slett.net> > ; Date: Tue, 6 Apr 2004 17:33:04 UTC; Severity: normal; Tags: wontfix; > Found > in version 2.1.16-6; > > <http://version.cgi?info=1;absolute=0;collapse=1;found=2.1.16-6;package=cyrus21-common> > *Filed 5 years and 3 days ago*; *Modified 2 years and 203 days ago*; > > Outstanding bugs -- Wishlist items; Patch Available (1 bug) > > - #522737 <http://bugreport.cgi?bug=522737> [w|+|┣] > [cyrus-pop3d-2.2<http://pkgreport.cgi?package=cyrus-pop3d-2.2>] > kolab-cyrus-pop3d: Patch to limit POP3 mailbox size (e.g. for mobile > clients) via username hack <http://bugreport.cgi?bug=522737> Reported > by: "Tor Slettnes" > <[email protected]><http://pkgreport.cgi?submitter=tor%40slett.net> > ; Date: Mon, 6 Apr 2009 04:45:02 UTC; Severity: wishlist; Tags: patch; > Blocking > fix for 522715 <http://bugreport.cgi?bug=522715>: kolab-cyrus-pop3d: > Patch to limit POP3 mailbox size (e.g. for mobile clients) via username > hack; > > Outstanding bugs -- Wishlist items; Unclassified (3 bugs) > > - #287581 <http://bugreport.cgi?bug=287581> [w| | ] > [exim4-config<http://pkgreport.cgi?package=exim4-config>] > please split ACL files <http://bugreport.cgi?bug=287581> Reported by: Tor > Slettnes <[email protected]><http://pkgreport.cgi?submitter=tor%40slett.net> > ; Date: Tue, 28 Dec 2004 22:18:02 UTC; Severity: wishlist; Found in > version 4.34-7; > > <http://version.cgi?info=1;absolute=0;collapse=1;found=4.34-7;package=exim4-config> > *Filed 4 years and 102 days ago*; *Modified 2 years and 257 days ago*; > - #392705 <http://bugreport.cgi?bug=392705> [w|u|=] > [mol<http://pkgreport.cgi?package=mol>] > mol-modules-source: kmod/_fault.c does not compile for powerpc64 > (G5)<http://bugreport.cgi?bug=392705> Reported > by: Tor Slettnes > <[email protected]><http://pkgreport.cgi?submitter=tor%40slett.net> > ; Date: Thu, 12 Oct 2006 22:13:12 UTC; Severity: wishlist; Tags: > upstream; Merged with 392709 <http://bugreport.cgi?bug=392709>; *Filed > 2 years and 179 days ago*; *Modified 2 years and 173 days ago*; > - #522715 <http://bugreport.cgi?bug=522715> [w| |┫] [ > kolab-cyrus-pop3d <http://pkgreport.cgi?package=kolab-cyrus-pop3d>] > kolab-cyrus-pop3d: > Patch to limit POP3 mailbox size (e.g. for mobile clients) via username > hack<http://bugreport.cgi?bug=522715> Reported > by: "Tor Slettnes" > <[email protected]><http://pkgreport.cgi?submitter=tor%40slett.net> > ; Date: Mon, 6 Apr 2009 04:45:02 UTC; Severity: wishlist; Found in > version 2.2.13-5.1; > > <http://version.cgi?info=1;absolute=0;collapse=1;found=2.2.13-5.1;package=kolab-cyrus-pop3d>Fix > blocked by 522737 <http://bugreport.cgi?bug=522737>: kolab-cyrus-pop3d: > Patch to limit POP3 mailbox size (e.g. for mobile clients) via username > hack; > > Resolved bugs -- Wishlist items (1 bug) > > - #274470 <http://bugreport.cgi?bug=274470> [w| |☺↝] > [wireless-tools<http://pkgreport.cgi?package=wireless-tools>] > wireless-tools: Sample mapping script for multiple SSIDs/APs: > ssidscan<http://bugreport.cgi?bug=274470> Reported > by: Tor Slettnes > <[email protected]><http://pkgreport.cgi?submitter=tor%40slett.net> > ; Date: Fri, 1 Oct 2004 23:18:01 UTC; Severity: wishlist; Found in > version 26+27pre25-2; Fixed in version ifscheme/1.5-1; > > <http://version.cgi?info=1;absolute=0;fixed=ifscheme%2F1.5-1;collapse=1;found=26%2B27pre25-2;package=wireless-tools> > *Forwarded* to Jean Tourrilhes ; *Done:* Guus Sliepen <[email protected] > >; *Filed 4 years and 190 days ago*; *Modified 3 years and 190 days ago > *; > > Summary > > - Status > - 5 Outstanding > - 1 Resolved > - Severity > - 1 Normal bugs > - 5 Wishlist items > - Classification > - 1 Patch Available > - 3 Unclassified > - 1 Will Not Fix > > Options Select bugs in package tagged with severity in source package in > packages maintained by submitted by owned by with status which affect > package with mail from newest bugs in bug > in package tagged with severity in source package in packages maintained > by submitted by owned by with status which affect package with mail from > newest > bugs in bug > > The same search fields are ORed, different fields are ANDed. > > Valid severities are critical, grave, serious, important, normal, minor, > wishlist, fixed > > Valid tags are patch, wontfix, moreinfo, unreproducible, fixed, potato, > woody, sid, help, security, upstream, pending, sarge, sarge-ignore, > experimental, d-i, confirmed, ipv6, lfs, fixed-in-experimental, > fixed-upstream, l10n, etch, etch-ignore, lenny, lenny-ignore, squeeze, > squeeze-ignore > Include Bugs with subject containing tagged with severity with pending > state with submitter containing with forwarded containing with owner > containing with package Exclude Bugs with subject containing tagged with > severity with pending state with submitter containing with forwarded > containing with owner containing with package Categorize using Order by > normal oldview raw age Misc options Repeat Merged > Reverse Bugs > Reverse Pending > Reverse Severity > None testing oldstable experimental unstable stable > Unarchived Archived Archived and Unarchived > Toggle all extra information Submit > ------------------------------ > Debian bug tracking system administrator <[email protected]>. Last > modified: Thu Apr 9 14:12:29 2009; Machine Name: rietz.debian.org > > Debian Bug tracking system <http://www.debian.org/Bugs/> > Copyright (C) 1999 Darren O. Benham, 1997,2003 nCipher Corporation Ltd, > 1994-97 Ian Jackson. > > _______________________________________________ > Pkg-Cyrus-imapd-Debian-devel mailing list > [email protected] > > http://lists.alioth.debian.org/mailman/listinfo/pkg-cyrus-imapd-debian-devel > -- Ondřej Surý <[email protected]> http://blog.rfc1925.org/
