Join GitHub today
GitHub is home to over 40 million developers working together to host and review code, manage projects, and build software together.
Sign upbpo-39502: Fix 64-bit Python PyTime_localtime() on AIX #18285
Conversation
… overflow
@@ -0,0 +1,2 @@ | |||
Fix PyTime_localtime() on AIX so that only 32-bit is range tested for overflow. |
This comment has been minimized.
This comment has been minimized.
vstinner
Jan 30, 2020
Member
I prefer to mention time.localtime() which is seen by users, than mention a private C function. 32-bit overflow is also obscure. I propose:
Fix :func:`time.localtime` on 64-bit AIX to support years before 1902 and after 2038.
Patch by M Felt.
This comment has been minimized.
This comment has been minimized.
This comment has been minimized.
This comment has been minimized.
This PR is only for 64-bit AIX. The skip is still useful for 32-bit AIX which is still limited to year 2038: see https://bugs.python.org/issue39502 |
This comment has been minimized.
This comment has been minimized.
Not sure what to do. Apparently when I did my refresh - it was before #18282 was in master. I see:
And when I run the test I see:
So, I can try something tricky for me (rebase to include #18282), or, leave it asis, or maybe #18282 is removed. Let me know how you would like (me) to proceed. p.s. Seems JFS2 could be added -- although, I am testing on NFSv3, and it may be something caused by the NFS server (which is Linux based). I'll try copying the whole thing to a local drive and see what happens. |
aixtools commentedJan 30, 2020
•
edited by bedevere-bot
so that only 32-bit is range tested for overflow
https://bugs.python.org/issue39502