Fails to build from source on s390x

[GoogleCodeExporter]

Hello,
this report comes from a fellow Debian Developer:

python-regex fails to build from source on s390x due to failures in the
testsuite. The problem is that the flags properties internally uses an
uint32, but is declared as T_PYSSIZET, that is an int64 value. The patch
below fixes the problem.

--- python-regex-0.1.20140120.orig/Python2/_regex.c
+++ python-regex-0.1.20140120/Python2/_regex.c
@@ -17089,7 +17089,7 @@ static PyGetSetDef pattern_getset[] = {
 static PyMemberDef pattern_members[] = {
     {"pattern", T_OBJECT, offsetof(PatternObject, pattern), READONLY,
       "The pattern string from which the regex object was compiled."},
-    {"flags", T_PYSSIZET, offsetof(PatternObject, flags), READONLY,
+    {"flags", T_UINT, offsetof(PatternObject, flags), READONLY,
       "The regex matching flags."},
     {"groups", T_PYSSIZET, offsetof(PatternObject, public_group_count),
       READONLY, "The number of capturing groups in the pattern."},
--- python-regex-0.1.20140120.orig/Python3/_regex.c
+++ python-regex-0.1.20140120/Python3/_regex.c
@@ -17082,7 +17082,7 @@ static PyGetSetDef pattern_getset[] = {
 static PyMemberDef pattern_members[] = {
     {"pattern", T_OBJECT, offsetof(PatternObject, pattern), READONLY,
       "The pattern string from which the regex object was compiled."},
-    {"flags", T_PYSSIZET, offsetof(PatternObject, flags), READONLY,
+    {"flags", T_UINT, offsetof(PatternObject, flags), READONLY,
       "The regex matching flags."},
     {"groups", T_PYSSIZET, offsetof(PatternObject, public_group_count),
       READONLY, "The number of capturing groups in the pattern."},

Original issue reported on code.google.com by sandro.tosi on 16 Feb 2014 at 5:33

[GoogleCodeExporter]

reference to the issue (and for proper credits): http://bugs.debian.org/739122

Original comment by sandro.tosi on 16 Feb 2014 at 5:40

• Added labels: ****
• Removed labels: ****

[GoogleCodeExporter]

Fixed in regex 2014.02.16.

Original comment by re...@mrabarnett.plus.com on 16 Feb 2014 at 8:08

• Changed state: Fixed
• Added labels: ****
• Removed labels: ****

[GoogleCodeExporter]

thanks!

Original comment by sandro.tosi on 16 Feb 2014 at 8:12

• Added labels: ****
• Removed labels: ****