devel/zookeeper rc.d script
George Neville-Neil
gnn at freebsd.org
Mon Aug 20 15:43:30 UTC 2012
On Aug 6, 2012, at 03:04 , Doug Barton <dougb at FreeBSD.org> wrote:
> On 07/09/2012 08:57, George Neville-Neil wrote:
>>
>> On Jul 9, 2012, at 00:15 , Doug Barton wrote:
>>
>>> On 07/08/2012 20:01, George Neville-Neil wrote:
>>>>
>>>> On Jul 8, 2012, at 22:39 , Doug Barton wrote:
>>>>
>>>>> On 07/08/2012 19:33, George Neville-Neil wrote:
>>>>>> A followup. zookeeper is now ported to Freebsd (/usr/ports/devel/zookeeper)
>>>>>
>>>>> George, did you see the PR and the followup from me regarding the port?
>>>>>
>>>>
>>>> I got a mail from jgh@ but only today figured out what the PR was.
>>>
>>> Are you not getting your gnn at FreeBSD.org mail?
>>>
>>>> I'll look at the patches from him tomorrow.
>>>
>>> I copied the text from my message below for your convenience.
>>>
>>>> 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 ...
>
> George,
>
> 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 at FreeBSD.org.
>
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
More information about the freebsd-ports
mailing list