From owner-freebsd-ports@FreeBSD.ORG Mon Aug 20 15:43:30 2012 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 340F1106566B; Mon, 20 Aug 2012 15:43:30 +0000 (UTC) (envelope-from gnn@freebsd.org) Received: from vps.hungerhost.com (vps.hungerhost.com [216.38.53.176]) by mx1.freebsd.org (Postfix) with ESMTP id E152B8FC0A; Mon, 20 Aug 2012 15:43:29 +0000 (UTC) Received: from [209.249.190.124] (port=56468 helo=gnnmac.hudson-trading.com) by vps.hungerhost.com with esmtpsa (TLSv1:AES128-SHA:128) (Exim 4.77) (envelope-from ) id 1T3U8T-0005j0-Os; Mon, 20 Aug 2012 11:43:25 -0400 Content-Type: text/plain; charset=iso-8859-1 Mime-Version: 1.0 (Mac OS X Mail 6.0 \(1485\)) From: George Neville-Neil In-Reply-To: <501F6C8E.3030008@FreeBSD.org> Date: Mon, 20 Aug 2012 11:43:16 -0400 Content-Transfer-Encoding: quoted-printable Message-Id: References: <86hatodavd.wl%gnn@neville-neil.com> <1341340703.6639@da3m0n8t3r.com> <20120704194917.GA15426@misty.eyesbeyond.com> <2244ACD7-892A-4CA7-90B1-FFF41AF6B317@FreeBSD.org> <49BAAAB5-BF68-43B3-8D7C-B4E49DE29ED3@neville-neil.com> <4FFA4454.5070803@FreeBSD.org> <4FFA5AF0.607@FreeBSD.org> <8DB0856C-A5D6-4C03-9246-743E51BC8B86@freebsd.org> <501F6C8E.3030008@FreeBSD.org> To: Doug Barton X-Mailer: Apple Mail (2.1485) X-AntiAbuse: This header was added to track abuse, please include it with any abuse report X-AntiAbuse: Primary Hostname - vps.hungerhost.com X-AntiAbuse: Original Domain - freebsd.org X-AntiAbuse: Originator/Caller UID/GID - [47 12] / [47 12] X-AntiAbuse: Sender Address Domain - freebsd.org Cc: freebsd-ports@FreeBSD.org Subject: Re: devel/zookeeper rc.d script 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: Mon, 20 Aug 2012 15:43:30 -0000 On Aug 6, 2012, at 03:04 , Doug Barton wrote: > On 07/09/2012 08:57, George Neville-Neil wrote: >>=20 >> On Jul 9, 2012, at 00:15 , Doug Barton wrote: >>=20 >>> On 07/08/2012 20:01, George Neville-Neil wrote: >>>>=20 >>>> On Jul 8, 2012, at 22:39 , Doug Barton wrote: >>>>=20 >>>>> On 07/08/2012 19:33, George Neville-Neil wrote: >>>>>> A followup. zookeeper is now ported to Freebsd = (/usr/ports/devel/zookeeper) >>>>>=20 >>>>> George, did you see the PR and the followup from me regarding the = port? >>>>>=20 >>>>=20 >>>> I got a mail from jgh@ but only today figured out what the PR was. >>>=20 >>> Are you not getting your gnn@FreeBSD.org mail? >>>=20 >>>> I'll look at the patches from him tomorrow. >>>=20 >>> I copied the text from my message below for your convenience. >>>=20 >>>> http://www.freebsd.org/cgi/query-pr.cgi?pr=3D169693 >>>=20 >>> Furthermore the rc.d script is a mess, and should not have been >>> committed like it was (numerous missing bits, bad format, set_rcvar, >>> hard-coded /usr/local, no REQUIRE, no KEYWORD: shutdown, etc.). = Please >>> read >>> = http://www.freebsd.org/doc/en_US.ISO8859-1/books/porters-handbook/rc-scrip= ts.html >>> and then ask in freebsd-rc@ if you have any additional questions. = Sorry >>> to be so blunt, but I'm really, really tired of repeating the same = stuff >>> over and over again, and this script is really a mess. >>>=20 >>> Also, don't install the script in do-install, see the web page above >>> (and/or the PR) for USE_RC_SUBR. >>>=20 >>> And FYI, there is no need to have the function in that script. You = could >>> use (for example) start_cmd=3D"$command start" just as well. Not to >>> mention that the function you have should be using $1 as the = argument to >>> $command, not $rc_arg. Reasons why left as an exercise for the = reader ... >=20 > George, >=20 > Some of these issues were covered by the PR, but many of them were = not. > If you have any questions about the changes that are needed you should > feel free to ask on freebsd-rc@FreeBSD.org. >=20 I think many or most of these are addressed by updates being made by Steven Kreuzer to both this port and the zookeeper-devel port. Best, George