Date: Fri, 06 Jul 2012 23:48:23 -0700 From: Doug Barton <dougb@FreeBSD.org> To: "George V. Neville-Neil" <gnn@FreeBSD.org> Cc: cvs-ports@FreeBSD.org, Jason Helfman <jgh@FreeBSD.org>, cvs-all@FreeBSD.org, ports-committers@FreeBSD.org Subject: Re: cvs commit: ports/devel/zookeeper Makefile distinfo pkg-descr pkg-plist ports/devel/zookeeper/files patch-src-c-src-zookeeper.c zookeeper Message-ID: <4FF7DBB7.1000801@FreeBSD.org> In-Reply-To: <20120707043907.GA71944@dormouse.experts-exchange.com> References: <201207062027.q66KRZD1074466@repoman.freebsd.org> <20120707043907.GA71944@dormouse.experts-exchange.com>
next in thread | previous in thread | raw e-mail | index | archive | help
On 07/06/2012 21:39, Jason Helfman wrote: > On Fri, Jul 06, 2012 at 08:27:35PM +0000, George V. Neville-Neil thus > spake: >> gnn 2012-07-06 20:27:35 UTC >> >> FreeBSD ports repository (src committer) >> >> Added files: >> devel/zookeeper Makefile distinfo pkg-descr pkg-plist >> devel/zookeeper/files patch-src-c-src-zookeeper.c zookeeper >> Log: >> Initial commit of Apache Zookeeper a distributed coordination system. >> >> Approved by: skreuzer (mentor) >> >> Revision Changes Path >> 1.1 +79 -0 ports/devel/zookeeper/Makefile (new) >> 1.1 +2 -0 ports/devel/zookeeper/distinfo (new) >> 1.1 +21 -0 >> ports/devel/zookeeper/files/patch-src-c-src-zookeeper.c (new) >> 1.1 +33 -0 ports/devel/zookeeper/files/zookeeper (new) >> 1.1 +17 -0 ports/devel/zookeeper/pkg-descr (new) >> 1.1 +378 -0 ports/devel/zookeeper/pkg-plist (new) > > This seems to be broken, but I submitted a pr to fix it. > > http://www.freebsd.org/cgi/query-pr.cgi?pr=169693 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-scripts.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. Also, don't install the script in do-install, see the web page above (and/or the PR) for USE_RC_SUBR. And FYI, there is no need to have the function in that script. You could use (for example) start_cmd="$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 ... Doug -- This .signature sanitized for your protection
Want to link to this message? Use this URL: <https://mail-archive.FreeBSD.org/cgi/mid.cgi?4FF7DBB7.1000801>