Bug 307676

Summary:	PyKDE doesn't build with PyQt 4.9.5
Product:	[Unmaintained] bindings	Reporter:	Andrea Scarpino <scarpino>
Component:	general	Assignee:	kde-bindings
Status:	RESOLVED FIXED
Severity:	major	CC:	Arfrever.FTA, johu, rdieter, stupor_scurvy343
Priority:	NOR
Version:	4.9
Target Milestone:	---
Platform:	Compiled Sources
OS:	Linux
Latest Commit:	03c76bc44e8b789b776b27297afa5751cfc52ee5	Version Fixed In:	4.9.3
Sentry Crash Report:
Attachments:	Fix for PyKDE

Description Andrea Scarpino 2012-10-01 16:29:37 UTC

[ 31%] Generating sip/kio/sipkiopart0.cpp, sip/kio/sipkiopart1.cpp, sip/kio/sipkiopart2.cpp, sip/kio/sipkiopart3.cpp, sip/kio/sipkiopart4.cpp, sip/kio/sipkiopart5.cpp, sip/kio/sipkiopart6.cpp, sip/kio/sipkiopart7.cpp

sip: /build/src/pykde4-4.9.1/sip/kdecore/typedefs.sip:955: Mapped type has already been defined in another module
make[2]: *** [sip/kio/sipkiopart0.cpp] Error 1
make[1]: *** [CMakeFiles/python_module_PyKDE4_kio.dir/all] Error 2
make: *** [all] Error 2

Reproducible: Always

Steps to Reproduce:
1. Update pyqt to 4.9.5
2. Try to build pykde4 from git
Actual Results:  
The build fails

Expected Results:  
It compiles

KDE 4.9.1
PyQt 4.9.5

Comment 1 Arfrever Frehtes Taifersar Arahesis 2012-10-01 23:35:20 UTC

Created attachment 74273 [details]
Fix for PyKDE

This patch for PyKDE deletes a mapped type, which is now available in PyQt >=4.9.5.

Comment 2 Rex Dieter 2012-10-03 15:06:59 UTC

See also, thread started here:
http://www.riverbankcomputing.com/pipermail/pyqt/2012-October/031981.html

already commited
017822bd0dfc83fe9a7a483ecc33f4aab839a3c6
on pykde4 master branch.

It was not backported to 4.9 since it would increase the minimal build requirements for sip/pyqt in a minor point release.

(feel free to reopen if you or anyone can come up with a way to make the code removable conditional on using the newer sip/pyqt or not)

Comment 3 Rex Dieter 2012-10-03 15:08:29 UTC

Oh, latest posts in that ml thread I mentioned seems to have some ideas on conditionlizing it after all, reopening.