From owner-freebsd-ports@FreeBSD.ORG Tue Jan 17 11:41:38 2006 Return-Path: X-Original-To: ports@FreeBSD.org 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 3F7C116A41F; Tue, 17 Jan 2006 11:41:38 +0000 (GMT) (envelope-from boris@brooknet.com.au) Received: from mailout1.pacific.net.au (mailout1.pacific.net.au [61.8.0.84]) by mx1.FreeBSD.org (Postfix) with ESMTP id CF48143D45; Tue, 17 Jan 2006 11:41:35 +0000 (GMT) (envelope-from boris@brooknet.com.au) Received: from mailproxy2.pacific.net.au (mailproxy2.pacific.net.au [61.8.0.87]) by mailout1.pacific.net.au (8.13.4/8.13.4/Debian-3) with ESMTP id k0HBfXrb022900; Tue, 17 Jan 2006 22:41:33 +1100 Received: from [61.8.44.149] (ppp2C95.dyn.pacific.net.au [61.8.44.149]) by mailproxy2.pacific.net.au (8.13.4/8.13.4/Debian-3) with ESMTP id k0HBfUjq009227; Tue, 17 Jan 2006 22:41:31 +1100 In-Reply-To: <12B98969-6186-4CA7-A3C6-11F5F36E2E1C@brooknet.com.au> References: <200601150911.k0F9B6eG062331@repoman.freebsd.org> <43CC3140.9040604@FreeBSD.org> <8F69821A-31A2-4E2D-A9E9-5CE1BEB2EE1F@brooknet.com.au> <1137494743.38904.41.camel@pav.hide.vol.cz> <12B98969-6186-4CA7-A3C6-11F5F36E2E1C@brooknet.com.au> Mime-Version: 1.0 (Apple Message framework v746.2) Content-Type: text/plain; charset=WINDOWS-1252; delsp=yes; format=flowed Message-Id: <98CB6F55-69F4-4944-A85C-E79DDAEB001A@brooknet.com.au> Content-Transfer-Encoding: quoted-printable From: Sam Lawrance Date: Tue, 17 Jan 2006 22:41:29 +1100 To: Sam Lawrance X-Mailer: Apple Mail (2.746.2) Cc: ports@FreeBSD.org, pav@FreeBSD.org, Doug Barton Subject: Re: cvs commit: ports/mail/dk-milter Makefile pkg-plist ports/mail/dk-milter/files milter-dk.sh.in 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: Tue, 17 Jan 2006 11:41:38 -0000 On 17/01/2006, at 9:56 PM, Sam Lawrance wrote: > > On 17/01/2006, at 9:45 PM, Pav Lucistnik wrote: > >> Sam Lawrance p=ED=9Ae v =FAt 17. 01. 2006 v 21:39 +1100: >>> Over to ports@ ... >>> >>> On 17/01/2006, at 10:50 AM, Doug Barton wrote: >>> >>>> Pav Lucistnik wrote: >>>>> pav 2006-01-15 09:11:04 UTC >>>>> >>>>> FreeBSD ports repository >>>>> >>>>> Modified files: >>>>> mail/dk-milter Makefile pkg-plist >>>>> mail/dk-milter/files milter-dk.sh.in >>>>> Log: >>>>> - Convert RC script to rc_subr >>>>> >>>>> PR: ports/91595 http://www.FreeBSD.org/cgi/query- >>>>> pr.cgi?pr=3D91595 >>>>> Submitted by: Hirohisa Yamaguchi >>>>> >>>>> Revision Changes Path >>>>> 1.6 +3 -2 ports/mail/dk-milter/Makefile >>>>> 1.2 +43 -48 ports/mail/dk-milter/files/milter-dk.sh.in >>>>> 1.2 +0 -1 ports/mail/dk-milter/pkg-plist >>>>> >>>>> http://www.FreeBSD.org/cgi/cvsweb.cgi/ports/mail/dk-milter/ >>>>> Makefile.diff?&r1=3D1.5&r2=3D1.6&f=3Dh >>>>> http://www.FreeBSD.org/cgi/cvsweb.cgi/ports/mail/dk-milter/files/ >>>>> milter-dk.sh.in.diff?&r1=3D1.1&r2=3D1.2&f=3Dh >>>>> http://www.FreeBSD.org/cgi/cvsweb.cgi/ports/mail/dk-milter/pkg- >>>>> plist.diff?&r1=3D1.1&r2=3D1.2&f=3Dh >>>> >>>> It's not a big enough issue to warrant a change for this port, =20 >>>> but in >>>> general it's a good idea if the name of the rc.d file is the same >>>> as what >>>> the script PROVIDE's. This removes one potential source of >>>> confusion for users. >>> >>> Is it worth a patch to portlint? There are probably a stack of =20 >>> other >>> rc-related things that could be checked for, too. For example, =20 >>> if an >>> rc script is in the packing list, warn to use USE_RC_SUBR. Others? >> >> If an rc.d script is in the packing list! Old styled scripts are not >> affected. How will you check that from portlint? > > Some grepwork? I've seen people put new-style rc.d scripts in the =20 > packing list and install them in a post-install target. For example, a quick and incomplete grep shows that these ports have =20 an etc/*.sh script entry in the packing list, which contains the =20 string PROVIDE. Some roll their own substitution and install target instead of using =20 USE_RC_SUBR. Some do USE_RC_SUBR=3Dyes (another check, I know Pav was tracking this =20= problem at some stage). One or two just still have the rc.d script in the packing list, even =20 though USE_RC_SUBR takes care of it. /s/ports/net/3proxy/files/3proxy.sh.in:# PROVIDE: threeproxy /s/ports/japanese/Canna/files/canna.sh.in:# PROVIDE: canna /s/ports/mail/couriergraph/files/couriergraph.sh.in:# PROVIDE: =20 couriergraph /s/ports/net/freenet6/files/freenet6.sh.in:# PROVIDE: freenet6 /s/ports/www/junkbuster/files/junkbuster.sh.in:# PROVIDE: junkbuster /s/ports/www/nginx/files/nginx.sh.in:# PROVIDE: nginx /s/ports/net/gnu-radius/files/radiusd.sh.in:# PROVIDE radiusd /s/ports/net/vncreflector/files/vncreflector.sh.in:# PROVIDE: =20 vncreflector /s/ports/net/vls/files/vlsd.sh.in:# PROVIDE: vlsd