Hey Daniel and Charles,<br><br><div class="gmail_quote">2009/5/5 Daniel O'Connor <span dir="ltr"><<a href="mailto:doconnor@gsoft.com.au">doconnor@gsoft.com.au</a>></span><br><blockquote class="gmail_quote" style="border-left: 1px solid rgb(204, 204, 204); margin: 0pt 0pt 0pt 0.8ex; padding-left: 1ex;">
<div class="im">On Tue, 5 May 2009, Charles Lepple wrote:<br>
> oops, this wasn't supposed to make it into the trunk yet. I can<br>
> revert it until further testing, but has anyone else tried running<br>
> with this patch?<br>
><br>
> <a href="http://lists.alioth.debian.org/pipermail/nut-upsdev/2009-April/003821" target="_blank">http://lists.alioth.debian.org/pipermail/nut-upsdev/2009-April/003821</a><br>
>.html<br>
<br>
</div>me ;)<br>
<font color="#888888"></font></blockquote><div><br>I still have a draft answer to your patch Daniel, but only really got back on NUT since monday...<br>and I'm since in a deep update round.<br><br>I was about to make the following proposition:<br>
what would you think about adding your patch (as an override option) but defaulting to "upslog-"<ups>"-"<hostname> (for example: "upslog-myups-localhost"). This would always satisfy the unique name, with a sane default naming, and give a pattern for the logrotate definition (btw, would be interesting to add a sample config somewhere in the tree).<br>
</div></div><br>thoughts and comments?<br clear="all"><br>cheers,<br>Arnaud<br>-- <br>Linux / Unix Expert R&D - Eaton - <a href="http://www.eaton.com/mgeops">http://www.eaton.com/mgeops</a><br>Network UPS Tools (NUT) Project Leader - <a href="http://www.networkupstools.org/">http://www.networkupstools.org/</a><br>
Debian Developer - <a href="http://www.debian.org">http://www.debian.org</a><br>Free Software Developer - <a href="http://arnaud.quette.free.fr/">http://arnaud.quette.free.fr/</a><br>