Merge lp:~jelmer/brz/time-t-max into lp:brz

[Jelmer Vernooij]

Guard against time_t overflowing in test_timestamp.

[Martin Packman (gz)]

Hm, this test isn't great but sure, the change to limit its stupidity seems fine.

review: Approve

[The Breezy Bot (the-breezy-bot)]

There was nothing to merge
https://ci.breezy-vcs.org/job/land-brz/496/

=== modified file 'breezy/tests/test_timestamp.py'
--- breezy/tests/test_timestamp.py	2017-06-04 18:09:30 +0000
+++ breezy/tests/test_timestamp.py	2018-09-21 19:45:50 +0000
@@ -125,6 +125,14 @@
         o = -12*3600
         for count in range(500):
             t += random.random()*24*3600*30
+            try:
+                time.gmtime(t)
+            except OverflowError:
+                # We've reached the maximum for time_t on this platform
+                break
+            if time.localtime(t).tm_year > 9998:
+                # strptime doesn't always understand years with more than 4 digits.
+                break
             o = ((o/3600 + 13) % 25 - 12)*3600 # Add 1 wrap around from [-12, 12]
             date = timestamp.format_highres_date(t, o)
             t2, o2 = timestamp.unpack_highres_date(date)