[Nut-upsdev] nut_clock_* UT review

[VaclavKrpec at Eaton.com]

Hello Arnaud,

OK rev. 3775 adds support for date %s format checking
+ its usage is no longer disabled for Solaris, explicitly.

vasek





________________________________
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

________________________________


From: Arnaud Quette [mailto:aquette.dev at gmail.com]
Sent: Friday, November 09, 2012 4:10 PM
To: Krpec, Vaclav
Cc: nut-upsdev at lists.alioth.debian.org
Subject: Re: nut_clock_* UT review


2012/11/9 <VaclavKrpec at eaton.com<mailto:VaclavKrpec at eaton.com>>
OK, so I’ll modify it so that it only uses Perl if date bin. desn’t
support the %s format string.

that would indeed be the most suitable approach.
do you need some help for the configure test?

Couldn’t I just do something like this?

date +%s | grep -q '^[0-9]*$' && date_s_works=”yes”

a patch is attached.
you can then use "#ifdef HAVE_DATE_S..."

also note that I'll have to fix a few more things on the Solaris, since the PATH change generated some issues:
http://buildbot.networkupstools.org/public/nut/builders/Solaris-sparc/builds/258/steps/configure/logs/stdio

enjoy your week-end,
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
-------------- next part --------------
An HTML attachment was scrubbed...
URL: <http://lists.alioth.debian.org/pipermail/nut-upsdev/attachments/20121109/e8b64896/attachment.html>