From owner-freebsd-perl@FreeBSD.ORG Mon Jan 19 08:31:28 2004 Return-Path: Delivered-To: freebsd-perl@freebsd.org Received: from mx1.FreeBSD.org (mx1.freebsd.org [216.136.204.125]) by hub.freebsd.org (Postfix) with ESMTP id 8545816A4CE; Mon, 19 Jan 2004 08:31:28 -0800 (PST) Received: from dragon.nuxi.com (trang.nuxi.com [66.93.134.19]) by mx1.FreeBSD.org (Postfix) with ESMTP id CA2DF43D2F; Mon, 19 Jan 2004 08:31:26 -0800 (PST) (envelope-from obrien@dragon.nuxi.com) Received: from dragon.nuxi.com (obrien@localhost [127.0.0.1]) by dragon.nuxi.com (8.12.10/8.12.9) with ESMTP id i0JGVQip004977; Mon, 19 Jan 2004 08:31:26 -0800 (PST) (envelope-from obrien@dragon.nuxi.com) Received: (from obrien@localhost) by dragon.nuxi.com (8.12.10/8.12.10/Submit) id i0JGVQSh004976; Mon, 19 Jan 2004 08:31:26 -0800 (PST) (envelope-from obrien) Date: Mon, 19 Jan 2004 08:31:26 -0800 From: "David O'Brien" To: Mathieu Arnold Message-ID: <20040119163126.GA4722@dragon.nuxi.com> References: <200401191015.i0JAF9iR071128@repoman.freebsd.org> Mime-Version: 1.0 Content-Type: text/plain; charset=us-ascii Content-Disposition: inline In-Reply-To: <200401191015.i0JAF9iR071128@repoman.freebsd.org> User-Agent: Mutt/1.4.1i X-Operating-System: FreeBSD 5.2-CURRENT Organization: The NUXI BSD Group X-Pgp-Rsa-Fingerprint: B7 4D 3E E9 11 39 5F A3 90 76 5D 69 58 D9 98 7A X-Pgp-Rsa-Keyid: 1024/34F9F9D5 cc: perl@FreeBSD.org Subject: Re: cvs commit: ports/mail/p5-Mail-SpamAssassin Makefile X-BeenThere: freebsd-perl@freebsd.org X-Mailman-Version: 2.1.1 Precedence: list Reply-To: obrien@FreeBSD.org List-Id: maintainer of a number of perl-related ports List-Unsubscribe: , List-Archive: List-Post: List-Help: List-Subscribe: , X-List-Received-Date: Mon, 19 Jan 2004 16:31:28 -0000 On Mon, Jan 19, 2004 at 02:15:09AM -0800, Mathieu Arnold wrote: > mat 2004/01/19 02:15:09 PST > > FreeBSD ports repository > > Modified files: > mail/p5-Mail-SpamAssassin Makefile > Log: > - change spaces to tab to make portlint a little happier > - allow users override RC_DIR and RC_SUFX > - fix install spamd.sh in RC_DIR, not PREFIX/etc/rc.d WHY?? rc scripts live in /etc/rc.d peroid. Please make this port always ahear to our standards. There is no need to make ever little thing overrideable as that will only encourage people to try to do weird things we don't support. Also it is well known that one can turn off a packages's rc script by renaming the rc script such that it doesn't have a ".sh" extention. It is our defacto standard that packages rc scripts have ".sh" as their extention, again please don't encourage a wide sweaping change. Note that pkg-plist and spamd.sh were broken WRT RC_DIR -- neither did anything with it. I'd like to commit this patch to make this port to follow the FreeBSD Ports estabished ways: Index: Makefile =================================================================== RCS file: /home/pcvs/ports/mail/p5-Mail-SpamAssassin/Makefile,v retrieving revision 1.53 diff -u -r1.53 Makefile --- Makefile 19 Jan 2004 15:44:37 -0000 1.53 +++ Makefile 19 Jan 2004 16:30:22 -0000 @@ -42,14 +42,8 @@ SED_SCRIPT= -e 's|%%PREFIX%%|${PREFIX}|g' USE_RC_SUBR= yes -RC_DIR?= ${PREFIX}/etc/rc.d -RC_SUFX?= .sh -SED_SCRIPT+= -e 's|%%RC_SUBR%%|${RC_SUBR}|g' \ - -e 's|%%RC_DIR%%|${RC_DIR}|g' \ - -e 's|%%RC_SUFX%%|${RC_SUFX}|g' -PLIST_SUB+= RC_DIR=${RC_DIR} \ - RC_SUFX=${RC_SUFX} - +SED_SCRIPT+= -e 's|%%PREFIX%%|${PREFIX}|g' \ + -e 's|%%RC_SUBR%%|${RC_SUBR}|g' post-patch: @${FIND} ${WRKSRC} -name \*.orig -delete @@ -63,7 +57,7 @@ post-install: @${STRIP_CMD} ${PREFIX}/bin/spamc - ${INSTALL_SCRIPT} ${WRKDIR}/spamd.sh ${RC_DIR}/spamd${RC_SUFX} + ${INSTALL_SCRIPT} ${WRKDIR}/spamd.sh ${PREFIX}/etc/rc.d/spamd.sh .if !defined(NOPORTDOCS) ${MKDIR} ${DOCSDIR} Index: pkg-plist =================================================================== RCS file: /home/pcvs/ports/mail/p5-Mail-SpamAssassin/pkg-plist,v retrieving revision 1.20 diff -u -r1.20 pkg-plist --- pkg-plist 13 Jan 2004 08:29:28 -0000 1.20 +++ pkg-plist 19 Jan 2004 16:30:22 -0000 @@ -3,7 +3,7 @@ bin/spamc bin/spamd etc/mail/spamassassin/local.cf.sample -etc/rc.d/spamd%%RC_SUFX%% +etc/rc.d/spamd.sh %%SITE_PERL%%/Mail/SpamAssassin.pm %%SITE_PERL%%/Mail/SpamAssassin/ArchiveIterator.pm %%SITE_PERL%%/Mail/SpamAssassin/AuditMessage.pm