On Tue 01 Jul 2008 at 04:34PM, Shawn Walker wrote:
> >> I plan on reviewing Dan's changes after work later tonight.
> >
> > utcnow() does not appear to actually add a UTC timezone to the datetime.
> > I found the python docs for this stuff to be somewhat incomprehensible,
> > so I decided that I would just assume all datetimes were in UTC, and
> > take care to generate them in UTC.
>
> I'm not certain about that. I believe I found local timestamps when I
> was working with the catalog for the RSS/Atom feeds.
>
> I'll look closer later tonight.
The point I was trying to make (but obliquely) is that there seem to be
two kinds of datetime: those which carry TZ info in them, and those that
do not. Calling datetime.datetime.now() or datetime.datetime.utcnow()
both yield datetimes which do *not* carry a tzinfo.
If we ensure that all of our datetimes are filled with UTC timestamps,
then they don't need to have time zone info in them-- in other words,
as long as you're consistent, and don't need to convert back and forth,
it all pretty much works. As Johansen pointed out, there are other
spots in the code which need to be updated to be based on UTC time,
and the catalog is one of them...
-dp
--
Daniel Price - Solaris Kernel Engineering - [EMAIL PROTECTED] - blogs.sun.com/dp
_______________________________________________
pkg-discuss mailing list
[email protected]
http://mail.opensolaris.org/mailman/listinfo/pkg-discuss
