[PyQt] PyKDE does not compile on ARM

[Phil Thompson]

On Fri, 25 Feb 2011 13:38:37 +0100, "Hans-Peter Jansen" <hpj at urpla.net>
wrote:
> On Friday 25 February 2011, 12:20:05 Phil Thompson wrote:
>> On Fri, 25 Feb 2011 10:51:17 +0000, Jonathan Riddell
>> <jriddell at ubuntu.com>
>>
>> wrote:
>> > On 29 January 2011 17:57, Jonathan Riddell <jriddell at ubuntu.com> 
> wrote:
>> >> -x PyQt_qreal_double doesn't seem to be working with current sip
>> >> and pyqt.  Anyone know why?
>> >>
>> >> http://bugs.kde.org/show_bug.cgi?id=264506
>> >
>> > This was due to us missing out this patch
>>
>> http://bazaar.launchpad.net/~ubuntu-branches/ubuntu/natty/python-qt4/
>>natty/view/head:/debian/patches/qreal_float_support.patch
>>
>> > which improves support for architectures where qreal != double.
>> >
>> > Jonathan
>>
>> The first patch only makes sense if a qreal has the same size as a
>> double, but has a different binary format. Is this the case?
>>
>> The second patch might be better as a part of PyKDE, as it is not
>> needed by PyQt.
> 
> but is a real PITA for packagers, if a PyQt patch has to be maintained 
> separately in order to get PyKDE working on certain platforms..

I wasn't suggesting it as a PyKDE patch - just include it as part of
PyKDE. Particularly as it's necessity suggests a bug in KDE. (Why aren't
they using qreal throughout?)

> IMHO, given the upswing of arm based mobile devices, maintaining it in 
> PyQt would yield profit for the whole project (as it would result in 
> much easier package distribution on those platforms..).

Agreed - I'm suggesting reducing the total patch count by one.

If the first patch is really needed then I'll include it in PyQt.

Phil