cvs commit: ports/sysutils/monit/files monit.sh.in
Martin Pala
martinp at tildeslash.com
Wed Mar 17 02:31:07 UTC 2010
On Mar 16, 2010, at 3:40 AM, Doug Barton wrote:
> On 03/15/10 19:09, Wen Heping wrote:
>> wen 2010-03-16 02:09:44 UTC
>>
>> FreeBSD ports repository
>>
>> Modified files:
>> sysutils/monit/files monit.sh.in
>> Log:
>> - Improve rc script
>>
>> PR: ports/144772 http://www.FreeBSD.org/cgi/query-pr.cgi?pr=144772
>> Submitted by: Martin Pala <martinp at tildeslash.com> (maintainer)
>>
>> Revision Changes Path
>> 1.2 +7 -5 ports/sysutils/monit/files/monit.sh.in
>>
>> http://www.FreeBSD.org/cgi/cvsweb.cgi/ports/sysutils/monit/files/monit.sh.in.diff?&r1=1.1&r2=1.2&f=h
>
> Documenting the _flags option in comments is an improvement, yes.
> Everything else is a pessimization; particularly since the default
> assignment for monit_enable is now broken.
>
> I've attached a patch that fixes the following issues:
> 1. General re-sorting to match conventions.
> 2. Change the default for _enable to the conventional method, and fix it
> as a side effect. Also move it down past load_rc_config. Otherwise
> testing for a value first is meaningless.
> 3. Eliminate the need for $default_config
>
> Martin, please test this and respond ASAP. Since the script as committed
> is now broken, it needs to be fixed.
>
Hi,
no, the script is not broken - it works the same as before, i have tested it and found no issues - if you can reproduce any problem, please provide details. The monit_enable works OK with both values. The testing of monit_enable value before load_rc_config works OK - it should always fail and the monit_enable is set to "NO", which can be overridden by load_rc_config (is executed post this setting).
I think your patch is not necessary - it does cosmetic modifications only. For example apache rc script has very similar structure as monit rc script.
Regards,
Martin
More information about the cvs-all
mailing list