<br><div class="gmail_quote">2012/11/9  <span dir="ltr"><<a href="mailto:VaclavKrpec@eaton.com" target="_blank">VaclavKrpec@eaton.com</a>></span><br><blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex">
Hello gentlemen,<br></blockquote><div> <br>Hi Vasek,<br><br>I'm moving the thread to -upsdev, since it has not to be private...<br><br></div><blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex">

please note that revisions 3771 and 3772 conclude nut_clock_*<br>
development (including UTs):<br></blockquote><div><br>interestingly enough, we were talking about that 5 mn ago with Emilien ;)<br>my conclusion was that nut_clock devs were completed with 3772, and only the QRT side was remaining.<br>
but that last requires:<br>1) that I provide you with a procedure to setup QRT (and this requires me to push a fix upstream to Ubuntu)<br>2) that we have a jail mechanism to execute QRT, including system clock changes to not impact negatively the buildbot system.<br>
<br>thus, postponed for now.<br> </div><blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex">
<a href="http://anonscm.debian.org/viewvc/nut?view=revision&revision=3771" target="_blank">http://anonscm.debian.org/viewvc/nut?view=revision&revision=3771</a><br>
<a href="http://anonscm.debian.org/viewvc/nut?view=revision&revision=3772" target="_blank">http://anonscm.debian.org/viewvc/nut?view=revision&revision=3772</a><br>
<br>
Please review the code.<br>
Note the RTC UT on Solaris; it uses perl -e 'print time;' as the external<br>
RTC authority, because date +%s is not implemented on Solaris :-(<br></blockquote><div><br>yep, that was my only question from a preliminary review ;)<br>but I recall, in the meantime, the solaris situation: either gnu date (coreutils) or "perl print" to get "%s"...<br>
<br>I've installed gnu coreutils, including date. It's available in /usr/local/bin.<br>I've also modified the buildslave init script to first use this directory.<br>thus, you should now be able to use the same "time authority" everywhere...<br>
<br>I've just forced a build on 3772 to check the new PATH...<br><br></div><blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex">
We might want to discuss whether such dependency on Perl is wise<br>
on Solaris; perhaps do a (configure) detection & skip the test<br>
if Perl isn't installed...<br></blockquote><div><br>well, that would not be a real problem since we already have a few Perl / Python requirements for our buildslaves and developers systems. And Perl, as a dependency, is not that exotic ;)<br>
 </div><blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex">
Any questions, comments, recommendations etc welcome, as usual.<br></blockquote></div><br>I'll start the review and post back.<br>Thanks for your hard work Vasek!<br><br>cheers,<br>Arnaud<br>-- <br>Linux / Unix / Opensource Engineering Expert - Eaton - <a href="http://opensource.eaton.com" target="_blank">http://opensource.eaton.com</a><br>
Network UPS Tools (NUT) Project Leader - <a href="http://www.networkupstools.org" target="_blank">http://www.networkupstools.org</a><br>Debian Developer - <a href="http://www.debian.org" target="_blank">http://www.debian.org</a><br>
Free Software Developer - <a href="http://arnaud.quette.fr" target="_blank">http://arnaud.quette.fr</a><br><br>