Please do not reply directly to this email. All additional comments should be made in the comments box of this bug.
https://bugzilla.redhat.com/show_bug.cgi?id=761623
Joseph D. Wagner joe@josephdwagner.info changed:
What |Removed |Added ---------------------------------------------------------------------------- Attachment #542672|0 |1 is obsolete| |
--- Comment #3 from Joseph D. Wagner joe@josephdwagner.info 2011-12-11 20:01:00 EST --- Created attachment 545466 --> https://bugzilla.redhat.com/attachment.cgi?id=545466 More robust implementation.
I reworked my original proposed patches. I believe these to be of a much higher caliber and ready for peer review. I made sure to address the issues I brought up earlier.
Issue #1: This revised patch checks for both /usr/sbin/jsvc AND the option USE_JSVC="true". It does not assume that jsvc should be used simply because it is installed.
Issue #2: This revised patch correctly handles the pidfile and logging.