(original) (raw)

changeset: 87226:fd9ce1d4b820 user: Victor Stinner victor.stinner@gmail.com date: Sun Nov 17 23:39:21 2013 +0100 files: Lib/test/test_strftime.py Misc/NEWS Modules/timemodule.c description: Issue #19634: time.strftime("%y") now raises a ValueError on AIX when given a year before 1900. diff -r 00766fa3366b -r fd9ce1d4b820 Lib/test/test_strftime.py --- a/Lib/test/test_strftime.py Sun Nov 17 22:21:02 2013 +0000 +++ b/Lib/test/test_strftime.py Sun Nov 17 23:39:21 2013 +0100 @@ -182,15 +182,13 @@ a date before 1900 is passed with a format string containing "%y" """ - @unittest.skipUnless(sys.platform == "win32", "Only applies to Windows") - def test_y_before_1900_win(self): - with self.assertRaises(ValueError): - time.strftime("%y", (1899, 1, 1, 0, 0, 0, 0, 0, 0)) - - @unittest.skipIf(sys.platform == "win32", "Doesn't apply on Windows") - def test_y_before_1900_nonwin(self): - self.assertEqual( - time.strftime("%y", (1899, 1, 1, 0, 0, 0, 0, 0, 0)), "99") + def test_y_before_1900(self): + t = (1899, 1, 1, 0, 0, 0, 0, 0, 0) + if sys.platform == "win32" or sys.platform.startswith("aix"): + with self.assertRaises(ValueError): + time.strftime("%y", t) + else: + self.assertEqual(time.strftime("%y", t), "99") def test_y_1900(self): self.assertEqual( diff -r 00766fa3366b -r fd9ce1d4b820 Misc/NEWS --- a/Misc/NEWS Sun Nov 17 22:21:02 2013 +0000 +++ b/Misc/NEWS Sun Nov 17 23:39:21 2013 +0100 @@ -50,6 +50,9 @@ Library ------- +- Issue #19634: time.strftime("%y") now raises a ValueError on AIX when given a + year before 1900. + - Fix test.support.bind_port() to not cause an error when Python was compiled on a system with SO_REUSEPORT defined in the headers but run on a system with an OS kernel that does not support that reasonably new socket option. diff -r 00766fa3366b -r fd9ce1d4b820 Modules/timemodule.c --- a/Modules/timemodule.c Sun Nov 17 22:21:02 2013 +0000 +++ b/Modules/timemodule.c Sun Nov 17 23:39:21 2013 +0100 @@ -650,6 +650,20 @@ return NULL; } } +#elif defined(_AIX) + for(outbuf = wcschr(fmt, '%'); + outbuf != NULL; + outbuf = wcschr(outbuf+2, '%')) + { + /* Issue #19634: On AIX, wcsftime("y", (1899, 1, 1, 0, 0, 0, 0, 0, 0)) + returns "0/" instead of "99" */ + if (outbuf[1] == L'y' && buf.tm_year < 0) { + PyErr_SetString(PyExc_ValueError, + "format %y requires year >= 1900 on AIX"); + Py_DECREF(format); + return NULL; + } + } #endif fmtlen = time_strlen(fmt); /victor.stinner@gmail.com