pykde fails to compile on ARM

Bug #708714 reported by Jonathan Riddell
10
This bug affects 1 person
Affects Status Importance Assigned to Milestone
KDE Bindings
Invalid
Medium
kdebindings (Ubuntu)
Fix Released
Undecided
Unassigned
Natty
Fix Released
Undecided
Unassigned
python-qt4 (Ubuntu)
Fix Released
Undecided
Unassigned
Natty
Fix Released
Undecided
Unassigned
qgis (Ubuntu)
Invalid
Undecided
Unassigned
Natty
Invalid
Undecided
Unassigned

Bug Description

PyKDE fails to compile on ARM with version 4.6.0

/usr/bin/sip -t ALL -t WS_X11 -t Qt_4_7_1 -x VendorID -x PyQt_NoPrintRangeBug -x PyQt_qreal_double -g -j 8 -c /build/buildd/kdebindings-4.6.0/obj-arm-linux-gnueabi/python/pykde4-2.7/sip/akonadi -I /usr/share/sip/PyQt4 -I /build/buildd/kdebindings-4.6.0/python/pykde4/sip /build/buildd/kdebindings-4.6.0/python/pykde4/sip/akonadi/akonadimod.sip
sip: KPlotAxis::majorTickMarks() unsupported function return type - provide %MethodCode and a C++ signature

 -x PyQt_qreal_double should stop this problem from occuring but it doesn't seem to work, it's still correctly defined in qglobal.sip though

Jonathan Riddell (jr)
Changed in kdebindings (Ubuntu):
milestone: none → natty-alpha-2
tags: added: kubuntu
Changed in kdebindings:
status: Unknown → Confirmed
tags: added: armel
Changed in kdebindings:
importance: Unknown → Medium
Martin Pitt (pitti)
Changed in kdebindings (Ubuntu Natty):
milestone: natty-alpha-2 → natty-alpha-3
Revision history for this message
Michael Casadevall (mcasadevall) wrote :

Issue doesn't appear to be qreal related at first glance as the breaking function properly defines a double as does the kplotaxis.h header. Continuing to investigate.

Steve Langasek (vorlon)
tags: added: arm-porting-queue
Revision history for this message
Jani Monoses (jani) wrote :

In the python/pykde4 dir running

sip -t ALL -t WS_X11 -t Qt_4_7_1 -x VendorID -x PyQt_NoPrintRangeBug -x PyQt_qreal_double -g -c someoutputdir -I /usr/share/sip/PyQt4 -I sip sip/kdeui/kdeuimod.sip

fails as in the FTBFS log.

After dropping the -x PyQt_qreal_double it works.

Similarly it works if the above is not dropped but the signature of the offending functions is changed to return QList<float>
or QList<qreal> in
kplotaxis.sip whre the originals read

    QList<double> majorTickMarks () const;
    QList<double> minorTickMarks () const;

By works I mean the command does not error, whether the generated bindings are correct I don't know.

One of the first mentions of MethodCode missing errors after googling shows it was related to templated return type, maybe that case needs more testing?

Revision history for this message
Felix Geyer (debfx) wrote :

qgis fails to build with the same error:
> sip: QgsComposerScaleBar::segmentPositions() unsupported function argument type - provide %MethodCode and a valid C++ signature

Revision history for this message
Jani Monoses (jani) wrote :

That has an argument of QList<QPair<double, double> >
So a combination of QList templates and doubles is common for these failures.

Revision history for this message
Jani Monoses (jani) wrote :

There was a regression in python-qt4-dev that caused the problem. An Ubuntu qreal_float patch was dropped which lead to some types of Method signatures not to be recognized. QList<qreal> was defined so on most platforms QList<double> worked, but not on ARM.

Changed in kdebindings (Ubuntu Natty):
status: New → Invalid
Jonathan Riddell (jr)
Changed in qgis (Ubuntu Natty):
status: New → Invalid
Changed in python-qt4 (Ubuntu Natty):
milestone: none → natty-alpha-3
Revision history for this message
Jani Monoses (jani) wrote :

Past the previous failure caused by a regressed python-qt4 but now failing in another place much later.
http://launchpadlibrarian.net/65100586/buildlog_ubuntu-natty-armel.kdebindings_4:4.6.0-0ubuntu1_FAILEDTOBUILD.txt.gz

Changed in python-qt4 (Ubuntu Natty):
status: New → Fix Released
Changed in kdebindings (Ubuntu Natty):
status: Invalid → Confirmed
Revision history for this message
Jonathan Riddell (jr) wrote :

This is a general problem not specific to ARM so I'll close this bug. Thanks for the help Jani.

Changed in kdebindings (Ubuntu Natty):
status: Confirmed → Fix Released
Changed in kdebindings:
status: Confirmed → Invalid
To post a comment you must log in.
This report contains Public information  
Everyone can see this information.

Other bug subscribers

Remote bug watches

Bug watches keep track of this bug in other bug trackers.