[issue10655] Wrong powerpc define in Python/ceval.c

Dave Malcolm added the comment: Merged back to 3.1 as r87800 and to 2.7-maint as r87801 -- resolution: -> fixed stage: -> committed/rejected status: open -> closed ___ Python tracker

[issue10655] Wrong powerpc define in Python/ceval.c

Dave Malcolm added the comment: Adrian verified in the downstream bug tracker that this fixed the build for him (and the logs show it successfully ran the test suite), so I committed my patch to py3k as r87796. -- ___ Python tracker

[issue10655] Wrong powerpc define in Python/ceval.c

Antoine Pitrou added the comment: Issue5369 is a duplicate with the same patch. Dave, can you apply? -- assignee: -> dmalcolm ___ Python tracker ___ ___

[issue10655] Wrong powerpc define in Python/ceval.c

Antoine Pitrou added the comment: > FWIW, I just added this patch downstream to Fedora's python 3 builds: > http://pkgs.fedoraproject.org/gitweb/?p=python3.git;a=blob_plain;f=python-3.2b2-fix-ppc-debug-build.patch;hb=5659c63442be2e91eb42e60284d7a9a9ab0b80dd Any reason not to commit it here? --

[issue10655] Wrong powerpc define in Python/ceval.c

Dave Malcolm added the comment: FWIW, I just added this patch downstream to Fedora's python 3 builds: http://pkgs.fedoraproject.org/gitweb/?p=python3.git;a=blob_plain;f=python-3.2b2-fix-ppc-debug-build.patch;hb=5659c63442be2e91eb42e60284d7a9a9ab0b80dd -- ___

[issue10655] Wrong powerpc define in Python/ceval.c

adrian added the comment: Here is a patch that I had to include in my Linux PowerPC build of 2.7 and 3.2 --- a/Python/ceval.c +++ b/Python/ceval.c @@ -31,10 +31,12 @@ typedef unsigned long long uint64; -#if defined(__ppc__) /* <- Don't know if this is the correct symbol; this +#if defined(

[issue10655] Wrong powerpc define in Python/ceval.c

Antoine Pitrou added the comment: Le mercredi 08 décembre 2010 à 23:06 +, Dave Malcolm a écrit : > Dave Malcolm added the comment: > > One of RH's gcc gurus told me in IRC that: > __ppc__ is not a standard powerpc*-linux macro > __PPC__ or __powerpc__ is Keep in mind that we have curr

[issue10655] Wrong powerpc define in Python/ceval.c

Dave Malcolm added the comment: One of RH's gcc gurus told me in IRC that: __ppc__ is not a standard powerpc*-linux macro __PPC__ or __powerpc__ is -- ___ Python tracker ___

[issue10655] Wrong powerpc define in Python/ceval.c

Dave Malcolm added the comment: What version of the compiler was this with? (For reference, I see that you also filed this downstream in Fedora's bug tracker as: https://bugzilla.redhat.com/show_bug.cgi?id=661510 ) -- nosy: +dmalcolm ___ Python tr

[issue10655] Wrong powerpc define in Python/ceval.c

Antoine Pitrou added the comment: How about accepting either of these symbols? Do you want to provide a patch? -- nosy: +pitrou versions: +Python 2.7, Python 3.2 ___ Python tracker

[issue10655] Wrong powerpc define in Python/ceval.c

New submission from adrian : In Python/ceval.c is following line: #if defined(__ppc__) /* <- Don't know if this is the correct symbol; this section should work for GCC on any PowerPC platform, irrespective of OS. PO