Bug 307676

Summary: PyKDE doesn't build with PyQt 4.9.5
Product: [Unmaintained] bindings Reporter: Andrea Scarpino <scarpino>
Component: generalAssignee: bindings-bugs-null
Status: RESOLVED FIXED    
Severity: major CC: Arfrever.FTA, johu, rdieter, stupor_scurvy343
Priority: NOR    
Version First Reported In: 4.9   
Target Milestone: ---   
Platform: Compiled Sources   
OS: Linux   
Latest Commit: 03c76bc44e8b789b776b27297afa5751cfc52ee5 Version Fixed/Implemented 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.