[PATCH] test_timestamp.py changes for win32

Aaron Bentley aaron.bentley at utoronto.ca
Wed Apr 18 23:49:18 BST 2007


-----BEGIN PGP SIGNED MESSAGE-----
Hash: SHA1

Alexander Belchenko wrote:
>>> This test is not checking negative dates.  1969-12-31 19:00:00 -0500 is
>>> 1970-01-01 0:00:00 +0000, and we ought to be able to represent that on
>>> all platforms.  So win32 seems to be broken (admittedly on an edge case.)
>>>
>>>> We shouldn't avoid the test on platforms where it succeeds. and
>>>> "knownFailure" gives us a way to say "yeah, this doesn't work on this
>>>> platform".
>>> I think KnownFailure is meant to reflect a bug in our code.  A
>>> KnownFailure should always fail.  It is an error for such a test to
>>> succeed when we expect it to fail.
>>>
>>> Since it relates to platforms, UnavailableFeature seems more appropriate.
> 
> Here the patch with GmtimeNegativeSecondsFeature.
> Can someone check this patch on Linux?
> (python bzr --no-plugins selftest test_timestamp -v)

Having looked at the implementation, I don't think that win32 is broken
at all.  I think the implementation is broken, but only win32 reveals
the bug.  Any timestamp on or after the epoch should not cause us to
call gmtime with a negative value.

Aaron
-----BEGIN PGP SIGNATURE-----
Version: GnuPG v1.4.3 (GNU/Linux)
Comment: Using GnuPG with Mozilla - http://enigmail.mozdev.org

iD8DBQFGJqBu0F+nu1YWqI0RArzmAJ9N1JYnVNc1tvGeDFucO/fcraWaoACcCzXR
PVIYp1BGw6sxQ0qlGS2e/i0=
=vqAt
-----END PGP SIGNATURE-----



More information about the bazaar mailing list