Re: [libvirt] [PATCH 1/3] util: new function virTimeLocalOffsetFromUTC

On Wed, May 21, 2014 at 04:16:29PM +0300, Laine Stump wrote: > Since there isn't a single libc API to get this value, this patch > supplies one which gets the value by grabbing current UTC, then > converting that into a struct tm with localtime_r(), then back to a > time_t using mktime; it again does the same operation, but using > gmtime_r() instead (for UTC). It then subtracts utc time from the > localtime, and finally adjusts if dst is set in the localtime timeinfo > (because for some reason mktime doesn't take that into account).

I think we ought to be able to unit test this code to make sure it is doing what we want. ie setenv("TZ", "EDT") in the start of the test suite to force a predictable timezone, then check the delta is correct.