Skip site navigation (1)Skip section navigation (2)
Date:      Mon, 25 Apr 2005 13:34:50 -0400
From:      Yarema <yds@CoolRat.org>
To:        Oliver Lehmann <lehmann@ans-netz.de>, Jose M Rodriguez <josemi@freebsd.jazztel.es>
Cc:        ports@dino.sk
Subject:   Re: splitting courier-authlib into master+slave ports
Message-ID:  <04814815DE2B181F5CC5F5A9@tuber.coolrat.org>
In-Reply-To: <20050425185726.14c7facf.lehmann@ans-netz.de>
References:  <20050414111426.775f6afd.lehmann@ans-netz.de> <895A5ED96F5ECDED176975AC@[192.168.1.73]> <7FED6BD2EB2A5924B2448D7B@tuber.coolrat.org> <200504251809.19713.josemi@redesjm.local> <20050425185726.14c7facf.lehmann@ans-netz.de>

next in thread | previous in thread | raw e-mail | index | archive | help
--==========81D0E70A83DAD2D35B44==========
Content-Type: text/plain; charset=us-ascii; format=flowed
Content-Transfer-Encoding: 7bit
Content-Disposition: inline

--On Monday, April 25, 2005 18:57:26 +0200 Oliver Lehmann 
<lehmann@ans-netz.de> wrote:

> Jose M Rodriguez wrote:
>
>> I think Makefile.opt.diff is not acceptable
>>
>> Remember that we are working to import Makefile.opt from foreing ports
>> and for the metaport.
>>
>> Work with a WITH RCORDER Knob and, if you really need and option in the
>> metaport, place it in Makefile, before importing Makefile.opt
>
> hmm the thing is, I don't think it'll work that way. If you think it is
> important Yarema, OPTIONS= has to be in courier-authlib-base/Makefile
> (since no port depends on security/courier-authlib, and defining it in
> courier, sqwebmail would nothing change for courier-authlib-base)
>
> I've uploaded an updated version of the tar.gz where I added OPTIONS= to
> -base besides other changes Jose provided me as pm.

OK, I see what you mean..  attached is a small diff to the Makefile moving 
the MASTER_SITES declaration to the top where portlint likes it.  And a 
couple of blank space at end of line fixes pointed out by portlint.

... and there's another typo in Makefile.dep: s/tist/this/ ;)

-- 
Yarema
http://yds.CoolRat.org
--==========81D0E70A83DAD2D35B44==========
Content-Type: text/plain; charset=us-ascii; name="Makefile.diff"
Content-Transfer-Encoding: 7bit
Content-Disposition: attachment; filename="Makefile.diff"; size=1169

--- Makefile.orig	Mon Apr 25 12:56:13 2005
+++ Makefile	Mon Apr 25 13:28:18 2005
@@ -9,7 +9,11 @@
 PORTVERSION=	0.55
 PORTREVISION=	2
 CATEGORIES?=	security mail
-.if !defined(PKGNAMESUFFIX)
+
+.if defined(PKGNAMESUFFIX)
+MASTER_SITES=	${MASTER_SITE_SOURCEFORGE}
+MASTER_SITE_SUBDIR=	courier
+.else
 MASTER_SITES=	# empty
 DISTFILES=	# empty
 EXTRACT_ONLY=	# empty
@@ -26,15 +30,12 @@
 NO_BUILD=	yes
 PATCHDIR=	/dev/null
 
-OPTIONS+=	GDBM	"Use gdbm db instead of system bdb"		off
+OPTIONS+=	GDBM	"Use gdbm instead of system bdb"	off
 
 .include "${.CURDIR}/Makefile.opt"
 
 .else	# !defined(PKGNAMESUFFIX)
 
-MASTER_SITES=	${MASTER_SITE_SOURCEFORGE}
-MASTER_SITE_SUBDIR=	courier
-
 USE_BZIP2=	yes
 USE_GMAKE=	yes
 USE_LIBLTDL=	yes
@@ -88,7 +89,7 @@
 
 MAN3=		authlib.3 auth_generic.3 auth_login.3 auth_getuserinfo.3 \
 		auth_enumerate.3 auth_passwd.3 auth_getoption.3 auth_sasl.3
- 
+
 DOCS=		AUTHORS COPYING ChangeLog INSTALL NEWS README \
 		README.authdebug.html README_authlib.html
 
@@ -169,7 +170,7 @@
 .if ${AUTHMOD} == authbase
 .if defined(WITH_RCORDER)
 USE_RCORDER=	courier-authdaemond.sh
-.else 
+.else
 USE_RC_SUBR=	courier-authdaemond.sh
 .endif
 .endif

--==========81D0E70A83DAD2D35B44==========--



Want to link to this message? Use this URL: <https://mail-archive.FreeBSD.org/cgi/mid.cgi?04814815DE2B181F5CC5F5A9>