Merge lp:~jelmer/brz/ancient-time into lp:brz/3.0

Proposed by Jelmer Vernooij
Status: Merged
Approved by: Jelmer Vernooij
Approved revision: no longer in the source branch.
Merge reported by: The Breezy Bot
Merged at revision: not available
Proposed branch: lp:~jelmer/brz/ancient-time
Merge into: lp:brz/3.0
Diff against target: 28 lines (+4/-3)
2 files modified
breezy/bzr/_dirstate_helpers_pyx.pyx (+1/-3)
doc/en/release-notes/brz-3.0.txt (+3/-0)
To merge this branch: bzr merge lp:~jelmer/brz/ancient-time
Reviewer Review Type Date Requested Status
Jelmer Vernooij Approve
Review via email: mp+377980@code.launchpad.net

Commit message

Fix conversion of time_t to int on certain platforms.

Description of the change

Fix conversion of time_t to int on certain platforms.

To post a comment you must log in.
Revision history for this message
Jelmer Vernooij (jelmer) :
review: Approve

Preview Diff

[H/L] Next/Prev Comment, [J/K] Next/Prev File, [N/P] Next/Prev Hunk
=== modified file 'breezy/bzr/_dirstate_helpers_pyx.pyx'
--- breezy/bzr/_dirstate_helpers_pyx.pyx 2019-11-18 06:20:31 +0000
+++ breezy/bzr/_dirstate_helpers_pyx.pyx 2020-01-23 03:26:48 +0000
@@ -797,10 +797,8 @@
797797
798798
799cdef unsigned long _time_to_unsigned(object t): # cannot_raise799cdef unsigned long _time_to_unsigned(object t): # cannot_raise
800 cdef double dt
801 if PyFloat_Check(t):800 if PyFloat_Check(t):
802 dt = PyFloat_AsDouble(t)801 t = t.__int__()
803 return <unsigned long>dt
804 return PyInt_AsUnsignedLongMask(t)802 return PyInt_AsUnsignedLongMask(t)
805803
806804
807805
=== modified file 'doc/en/release-notes/brz-3.0.txt'
--- doc/en/release-notes/brz-3.0.txt 2019-12-23 03:05:20 +0000
+++ doc/en/release-notes/brz-3.0.txt 2020-01-23 03:26:48 +0000
@@ -40,6 +40,9 @@
40 * Don't rely on private transport attribute in fast-import.40 * Don't rely on private transport attribute in fast-import.
41 (Jelmer Vernooij, #1854607)41 (Jelmer Vernooij, #1854607)
4242
43 * Fix conversion of time_t to int on certain platforms (arm64, armhf,
44 ppc64el). (Jelmer Vernooij)
45
43Documentation46Documentation
44*************47*************
4548

Subscribers

People subscribed via source and target branches