
<html><body>

<p><tt>andrew@beekhof.net wrote on 06/30/2009 14:14:12 PM:<br>

<br>

> I'd prefer sbin_PROGRAMS instead of halib_PROGRAMS for<br>

> notifyServicelogEvent (the others are only installed there for legacy<br>

> reasons).<br>

> Also, since you're linking against crmcommon (which should be<br>

> $(top_builddir)/lib/common/libcrmcommon.la btw), you may want to use<br>

> the crm_err(), crm_debug() macros instead of using syslog directly.<br>

> <br>

> If you're happy with those two changes I'll make them and commit<br>

</tt><br>

<tt>As I was making the changes locally, I did notice that notifyServicelogEvent needed</tt><br>

<tt>to now support --help and --version in order for help2man to work.</tt><br>

<br>

<tt>So I am attaching a revised patch for you here...</tt><br>

<br>

<i>(See attached file: pacemaker.mark.1.patch)</i><tt><br>

</tt><br>

Mark</body></html>