From owner-freebsd-ports@FreeBSD.ORG Thu Apr 21 03:25:00 2005 Return-Path: Delivered-To: freebsd-ports@freebsd.org Received: from mx1.FreeBSD.org (mx1.freebsd.org [216.136.204.125]) by hub.freebsd.org (Postfix) with ESMTP id 9B81B16A4D1 for ; Thu, 21 Apr 2005 03:25:00 +0000 (GMT) Received: from dppl.com (sapas.dppl.com [216.182.10.231]) by mx1.FreeBSD.org (Postfix) with ESMTP id E98F943D39 for ; Thu, 21 Apr 2005 03:24:59 +0000 (GMT) (envelope-from yds@CoolRat.org) Received: from [192.168.1.69] (pcp0011284575pcs.union01.nj.comcast.net [69.248.144.22]) (AUTH: PLAIN yds, TLS: TLSv1/SSLv3,256bits,AES256-SHA) by dppl.com with esmtp; Wed, 20 Apr 2005 23:24:59 -0400 Date: Wed, 20 Apr 2005 23:24:58 -0400 From: Yarema To: Oliver Lehmann , ports@FreeBSD.org Message-ID: <13A599297289B8FA39CA73DB@tuber.coolrat.org> In-Reply-To: References: <20050414111426.775f6afd.lehmann@ans-netz.de> <200504191258.53327.freebsd@redesjm.local> <20050419173039.4725638a.lehmann@ans-netz.de> <200504191827.05896.ports@dino.sk> <20050420121254.34c59e53.lehmann@ans-netz.de> <9F0C7E4FBA18BFD0B3946DFE@tuber.coolrat.org> <20050420205646.5c7b72df.lehmann@ans-netz.de> X-Mailer: Mulberry/3.1.6 (Linux/x86) MIME-Version: 1.0 Content-Type: text/plain; charset=us-ascii; format=flowed Content-Transfer-Encoding: 7bit Content-Disposition: inline cc: Milan Obuch cc: Jose M Rodriguez Subject: Re: splitting courier-authlib into master+slave ports X-BeenThere: freebsd-ports@freebsd.org X-Mailman-Version: 2.1.1 Precedence: list List-Id: Porting software to FreeBSD List-Unsubscribe: , List-Archive: List-Post: List-Help: List-Subscribe: , X-List-Received-Date: Thu, 21 Apr 2005 03:25:00 -0000 Hello again Oliver, Another rewrite, putting some code where my mouth is... ;) This one is based on the latest version you provided. I moved your Makefile to Makefile.opt and made it so that a port which RUN_DEPENDS on courier-authlib can do: .include "${PORTSDIR}/security/courier-authlib/Makefile.opt instead of the usual: BUILD_DEPENDS+= courierauthconfig:${PORTSDIR}/security/courier-authlib RUN_DEPENDS+= courierauthconfig:${PORTSDIR}/security/courier-authlib As I proposed in my previous email (quoted below) this solves the OPTIONS/meta-port issue for dependent ports and keeps courier-authlib itself from having to deal with it. I then moved Makefile.ext back to Makefile and very liberally sprinkled it with lots of comments. Many of which in the form "# Oliver, ...." to indicate that they are there to point out my changes and are meant to be deleted. Otherwise I tried to keep to the style of the original rather than imposing my style on this edit. I left 'files/courier-authdaemond.sh' as is and for easy comparison included my version as 'files/courier-authdaemond.sh.in' Note that "USE_RCORDER= courier-authdaemond.sh" will process 'files/courier-authdaemond.sh.in' and not 'files/courier-authdaemond.sh', that's just how deals with it. I also did a bunch of cleanup to make portlint happier. Most of the remaining portlint warnings are caused by my comments which are meant to be deleted anyhow. -- Yarema http://yds.CoolRat.org --On Wednesday, April 20, 2005 20:11:20 -0400 Yarema wrote: > Hi Oliver, > > --On Wednesday, April 20, 2005 20:56:46 +0200 Oliver Lehmann > wrote: > >> Yarema wrote: >> >>> And that >>> post-install: >>> @${GREP} '^@exec ' ${TMPPLIST} \ >>> | ${SED} -e 's:^@exec ::' -e 's:%D:${PREFIX}:g' \ >>> > ${WRKDIR}/.PLIST.exec \ >>> && ${SH} ${WRKDIR}/.PLIST.exec >> >> I don't like it because it is not the "standard" way of doing that kind >> of thing. And until bsd.port.mk will do it, I don't want to use it. Even >> if that means to have some things duplicated. > > That's cool. I was just explaining why I did it that way. It's your > port to maintain so it's your choice to use whatever code you're most > comfortable with. > >> Further I like that meta-port idea and I'll keep it. With that you still >> have your seperate authpam metaport which is "on" per default in the >> meta- port so it will be installed, and it is easy to pik the module you >> need and you even won't forget it. And think of dependencies. If I want >> to install sqwebmail but I'm using vpopmail - I have to remember >> installing the -vchkpw port later. I won't see the pkg-message since >> sqwebmail will continue configuring when courier-authlib-base is >> installed. > > See my reply to Jose M Rodriguez explaining > how the "libauthpwd.so.0" happens to get installed in the currently > committed version of mail/courier-authlib BEFORE I ever touched it. > > Another idea of solving the meta port issue is to create an include > Makefile.opt in the base port which is included by every port which needs > courier-authlib. courier-authlib itself would not use this file. But > sqwebmail, courier-imap and the MTA courier ports could all suck it in > and give the user a menu to select which auth-methods to include. Kinda > like a meta port without an actual port. > >> I'll look in the other points later (rc.d isues) this evening or >> tomorrow. The script will remain called courier-authlib.sh since that is >> how it is now, and I see no point in changing it once more. We even have >> courier-imapd.sh, courier-pop3d.sh and so on... courier- authdaemond.sh >> would be more consistent to that than authdaemond.sh > > OK, I'm not all that particular about the name. Again just presenting > my thoughts on the matter. In either case take a look at how I coded it. > The name change is the most insignificant part of the rewrite. You can > just rename my version to courier-authdaemond.sh if you like. > > -- > Yarema > http://yds.CoolRat.org