From owner-freebsd-ports@FreeBSD.ORG Wed Sep 2 13:19:45 2009 Return-Path: Delivered-To: freebsd-ports@freebsd.org Received: from mx1.freebsd.org (mx1.freebsd.org [IPv6:2001:4f8:fff6::34]) by hub.freebsd.org (Postfix) with ESMTP id 8358D1065676 for ; Wed, 2 Sep 2009 13:19:45 +0000 (UTC) (envelope-from mel.flynn+fbsd.ports@mailing.thruhere.net) Received: from mailhub.rachie.is-a-geek.net (rachie.is-a-geek.net [66.230.99.27]) by mx1.freebsd.org (Postfix) with ESMTP id 46B698FC0C for ; Wed, 2 Sep 2009 13:19:44 +0000 (UTC) Received: from smoochies.rachie.is-a-geek.net (mailhub.lan.rachie.is-a-geek.net [192.168.2.11]) by mailhub.rachie.is-a-geek.net (Postfix) with ESMTP id 639487E818; Wed, 2 Sep 2009 05:19:55 -0800 (AKDT) From: Mel Flynn To: freebsd-ports@freebsd.org Date: Wed, 2 Sep 2009 15:19:41 +0200 User-Agent: KMail/1.11.4 (FreeBSD/8.0-BETA2; KDE/4.2.4; i386; ; ) References: <20090827131800.191378ee@gumby.homeunix.com> <4A982DC9.7050608@CoolRat.org> <20090829181122.GA22669@atarininja.org> In-Reply-To: <20090829181122.GA22669@atarininja.org> MIME-Version: 1.0 Content-Type: Text/Plain; charset="us-ascii" Content-Transfer-Encoding: 7bit Content-Disposition: inline Message-Id: <200909021519.41950.mel.flynn+fbsd.ports@mailing.thruhere.net> Cc: Wesley Shields Subject: Re: Dovecot Sieve port switched from CMU Sieve to Dovecot X-BeenThere: freebsd-ports@freebsd.org X-Mailman-Version: 2.1.5 Precedence: list List-Id: Porting software to FreeBSD List-Unsubscribe: , List-Archive: List-Post: List-Help: List-Subscribe: , X-List-Received-Date: Wed, 02 Sep 2009 13:19:45 -0000 On Saturday 29 August 2009 20:11:22 Wesley Shields wrote: > On Fri, Aug 28, 2009 at 03:19:37PM -0400, Yarema wrote: > > I was previously overruled by a committer when I filed a PR to default > > ManageSieve to ON. IIRC, POLA was sited as the reason. I'm still of > > the opinion that the ManageSieve patch to the main dovecot port should > > default to ON for the following reasons: > > > > - with the ManageSieve patch built into the package it becomes possible > > for users of binary packages to just install the dovecot-sieve and > > dovecot-managesieve ports and have them work. As it stands now anyone > > who wants to use ManageSieve has to build the dovecot port from source. > > So it doesn't even make sense to have a binary package of > > dovecot-managesieve unless the ManageSieve patch is built into the > > dovecot package by default as well. > > > > - the ManageSieve patch does not add much bulk to the package. Those > > who do not use ManageSieve can simply ignore it or if they build from > > source can disable it. Either way from the perspective of those who do > > not use ManageSieve nothing really changes (thus POLA is not violated). > > > > - and finally there would be fewer broken PRs filed without the distinfo > > for the ManageSieve patch included. > > > > In my opinion it seems not having the binary dovecot-managesieve package > > "just work" is more of a POLA violation than having an extra > > README.managesieve and related dovecot.conf sections installed by > > default in the main dovecot port. > > I have no problems marking that option as on by default since it will > mean that the managesieve port can be usefully packaged, while not > bloating the port at all. To further this issue in the "right" direction, I've investigated the bloat, using a slave port: PORTNAME= dovecot PKGNAMESUFFIX= -withsieve CATEGORIES= mail ipv6 MASTERDIR= ${.CURDIR}/../../mail/dovecot CONFLICTS= dovecot-1* .include "${MASTERDIR}/Makefile" .if defined(WITHOUT_MANAGESIEVE) .undef WITHOUT_MANAGESIEVE .endif WITH_MANAGESIEVE= yes Result: -rw-r--r-- 1 root wheel 2626479 Sep 2 05:05 dovecot-1.2.4.tbz -rw-r--r-- 1 root wheel 2626719 Sep 2 05:04 dovecot-withsieve-1.2.4.tbz I think more bytes have been wasted on discussing this, then it adds to the port. Also, I've left it off, thinking "I'll add this later or just add the package", because the OPTION framework does not really have enough room to specify "You have to tick this option to ON if you want to be able to add dovecot-managesieve port later", so yes, POLA was violated by not having it on by default and the description should probably read something like "Set to off if you never want managesieve support". -- Mel