Bug 437672 – [kde overlay] kde-misc/pate-9999 doesnt compile with =dev-python/PyQt4-4.9.5 =kde-base/pykde4-4.9.2-r1

• Status: RESOLVED FIXED
• Product: Gentoo Linux

[Description Alex Turbov 2012-10-09 05:12:22 UTC]

I'm trying to emerge kde-misc/pate-9999 from kde-testing overlay and it fails to install w/ =dev-python/PyQt4-4.9.5 =kde-base/pykde4-4.9.2-r1 w/ the following message:

sip: /usr/share/sip/PyKDE4/kdecore/typedefs.sip:958: Mapped type has already been defined in another module


Btw, the patch for pykde4-4.9.2-r1 (pykde4-4.9.2-pyqt45.patch) is incorrect: It defines `Py_v3' for Python v2 also!!!
(check the comment marked w/ 'ATTENTION'):

 # Use an extra option when compiling on Python 3.
 IF (PYTHON_3)
-    SET(SIP_EXTRA_OPTIONS -g -x Py_v3)
+    IF(PYQT4_VERSION STRGREATER "040904")
+	    #Disable for newer PyQt
+	    SET(SIP_EXTRA_OPTIONS -g -x PyKDE_QVector -x Py_v3)
+    ELSE(PYQT4_VERSION STRGREATER "040904")
+	    SET(SIP_EXTRA_OPTIONS -g -x Py_v3)
+    ENDIF(PYQT4_VERSION STRGREATER "040904")
 ELSE (PYTHON_3)
-    SET(SIP_EXTRA_OPTIONS -g)
+    IF(PYQT4_VERSION STRGREATER "040904")
+	    #Disable for newer PyQt
+	    SET(SIP_EXTRA_OPTIONS -g -x PyKDE_QVector)
+    ELSE(PYQT4_VERSION STRGREATER "040904")

# ATTENTION HERE: Py_v3 defined AGAIN!

+	    SET(SIP_EXTRA_OPTIONS -g  -x Py_v3)
+    ENDIF(PYQT4_VERSION STRGREATER "040904")
 ENDIF (PYTHON_3)

Anyway, it doesn't work...

Upstream bug is here: https://bugs.kde.org/show_bug.cgi?id=307676
... and they just choose to remove a duplicate definition and do not backport it :(

[Comment 1 Johannes Huber (RETIRED) 2012-10-14 09:04:45 UTC]

Last commit of pate was 4 years ago...

[Comment 2 Alex Turbov 2012-10-14 10:23:51 UTC]

(In reply to comment #1)
> Last commit of pate was 4 years ago...

so what? this is not a reason for compile failure...
it compile and work as expected if conflicted types will be removed from sip/kdecore/typedefs.sip (as suggested by upstream).

[Comment 3 Alex Turbov 2012-10-14 10:24:30 UTC]

Created attachment 326520 [details, diff]
remove conflicting types

[Comment 4 Michael Palimaka (kensington) 2012-10-23 15:22:15 UTC]

pate is part of kate as of 4.10

[Comment 5 Alex Turbov 2012-10-23 15:31:06 UTC]

(In reply to comment #4)
> pate is part of kate as of 4.10

I knew it... but while KDE 4.10 (and kate) is in future do you think current version doesn't deserve to be fixed?

[Comment 6 Michael Palimaka (kensington) 2012-10-23 17:10:43 UTC]

Thanks, this is now fixed in git.

http://git.overlays.gentoo.org/gitweb/?p=proj/kde.git;a=commitdiff;h=5ba40aee27ab4b0449e81b2ee6e39d29291dd308