Martin Wilke wrote: > miwi 2008-12-25 20:24:43 UTC > > FreeBSD ports repository > > Modified files: > net-mgmt/nrpe2 Makefile > net-mgmt/nrpe2/files nrpe2.in > Log: > - Modify rc.d script to allow administrator to change PID file location > without editing rc.d script. New knob nrpe2_pidfile is provided for > this and it defaults to the old value of the PID file. > - Bump PORTREVISION > > PR: 129386 http://www.FreeBSD.org/cgi/query-pr.cgi?pr=129386 > Submitted by: Eygene Ryabinkin <rea-f...@codelabs.ru> > Approved by: maintainer > > Revision Changes Path > 1.26 +1 -0 ports/net-mgmt/nrpe2/Makefile > 1.3 +5 -2 ports/net-mgmt/nrpe2/files/nrpe2.in > > http://www.FreeBSD.org/cgi/cvsweb.cgi/ports/net-mgmt/nrpe2/Makefile.diff?&r1=1.25&r2=1.26&f=h > http://www.FreeBSD.org/cgi/cvsweb.cgi/ports/net-mgmt/nrpe2/files/nrpe2.in.diff?&r1=1.2&r2=1.3&f=h
Overall the rc.d file looks pretty good, just a couple notes. You need to move the load_rc_config line above the default variable assignments, otherwise it's not very effective. I'm also unclear as to why you have two assignments for command_args. Finally you might want to consult http://www.freebsd.org/doc/en_US.ISO8859-1/books/porters-handbook/rc-scripts.html hth, Doug -- This .signature sanitized for your protection _______________________________________________ cvs-all@freebsd.org mailing list http://lists.freebsd.org/mailman/listinfo/cvs-all To unsubscribe, send any mail to "cvs-all-unsubscr...@freebsd.org"