[Cython] [cython-users] Re: Cython 0.17 beta 1 released

[Stefan Behnel]

Robert Bradshaw, 31.07.2012 19:24:
> On Tue, Jul 31, 2012 at 7:24 AM, Stefan Behnel wrote:
>> mark florisson, 28.07.2012 14:04:
>>> On 27 July 2012 23:30, Bradley Froehle wrote:
>>>> Thanks to the work of Yaroslav Halchenko, there is an experimental Debian
>>>> package for Cython 0.17.beta1 -- http://packages.qa.debian.org/c/cython.html
>>>>
>>>> However, the builds are showing a lot of test failures on non-amd64 sytems.
>>>> See https://buildd.debian.org/status/package.php?p=cython&suite=experimental
>>>> Many of the errors seem related to assumptions made about the sizes of
>>>> various integers (int vs. long vs long long).  Ideally we'll be able to
>>>> clean up these failures before the final release.
>>>
>>> Thanks, I think it's mostly the tests that are wrong. I'll try to get
>>> it fixed on linux 32 bit.
>>
>> I'm not sure the it's only the tests. The "char" vs. "unsigned char" errors
>> hint at platform specific differences, which might trigger bugs (aka.
>> incorrect assumptions) in the generated memory view code. Specifically,
>> plain "char" is unsigned at least on ARM and PowerPC, both of which fail
>> with this error.
> 
> Yes, I think somewhere we're assuming char is signed, but wasn't able
> to see where in my quick investigations.

Yes, it wasn't immediately obvious to me either. Here is a patch that
*might* fix the issue - obviously untested for the platforms in question.

Yaroslav, could you give it a try on the Debian build servers?

Stefan

-------------- next part --------------
A non-text attachment was scrubbed...
Name: buffer_integer_signedness.patch
Type: text/x-patch
Size: 1570 bytes
Desc: not available
URL: <http://mail.python.org/pipermail/cython-devel/attachments/20120731/d892dbb6/attachment.bin>