[Nut-upsdev] nut_clock_* UT review

[VaclavKrpec at Eaton.com]

please note that revisions 3771 and 3772 conclude nut_clock_*
development (including UTs):

my conclusion was that nut_clock devs were completed with 3772, and only the QRT side was remaining.
but that last requires:
1) that I provide you with a procedure to setup QRT (and this requires me to push a fix upstream to Ubuntu)
2) that we have a jail mechanism to execute QRT, including system clock changes to not impact negatively the buildbot system.

Right; I forgot to mention these tests; they are not covered by the commits,
of course.

Please review the code.
Note the RTC UT on Solaris; it uses perl -e 'print time;' as the external
RTC authority, because date +%s is not implemented on Solaris :-(

yep, that was my only question from a preliminary review ;)
but I recall, in the meantime, the solaris situation: either gnu date (coreutils) or "perl print" to get "%s"...

I've installed gnu coreutils, including date. It's available in /usr/local/bin.
I've also modified the buildslave init script to first use this directory.
thus, you should now be able to use the same "time authority" everywhere...

OK, so I’ll modify it so that it only uses Perl if date bin. desn’t
support the %s format string.

vasek


I've just forced a build on 3772 to check the new PATH...
We might want to discuss whether such dependency on Perl is wise
on Solaris; perhaps do a (configure) detection & skip the test
if Perl isn't installed...

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 ;)

Any questions, comments, recommendations etc welcome, as usual.

I'll start the review and post back.
Thanks for your hard work Vasek!

cheers,
Arnaud
--
Linux / Unix / Opensource Engineering Expert - Eaton - http://opensource.eaton.com
Network UPS Tools (NUT) Project Leader - http://www.networkupstools.org
Debian Developer - http://www.debian.org
Free Software Developer - http://arnaud.quette.fr



________________________________
Eaton Elektrotechnika s.r.o. ~ S�dlo spolecnosti, jak je zaps�no v rejstr�ku: Kom�rovsk� 2406, Praha 9 - Horn� Pocernice, 193 00, Cesk� Republika ~ Jm�no, m�sto, kde byla spolecnost zaregistrov�na: Praha ~ Identifikacn� c�slo (ICO): 498 11 894

________________________________


-------------- next part --------------
An HTML attachment was scrubbed...
URL: <http://lists.alioth.debian.org/pipermail/nut-upsdev/attachments/20121109/107d00c2/attachment.html>