<table><tr><td style="">arojas created this revision.<br />arojas added reviewers: Frameworks, bruns.<br />Herald added projects: Frameworks, Build System.<br />Herald added subscribers: kde-buildsystem, kde-frameworks-devel.<br />arojas requested review of this revision.
</td><a style="text-decoration: none; padding: 4px 8px; margin: 0 8px 8px; float: right; color: #464C5C; font-weight: bold; border-radius: 3px; background-color: #F7F7F9; background-image: linear-gradient(to bottom,#fff,#f1f0f1); display: inline-block; border: 1px solid rgba(71,87,120,.2);" href="https://phabricator.kde.org/D15091">View Revision</a></tr></table><br /><div><strong>REVISION SUMMARY</strong><div><p>PyQt>=5.11 is compiled with the 'PyQt5.sip' sipname by default. This causes a sipname mismatch with KF5 bindings and prevents them from loading:</p>

<p>----> 1 from PyKF5.KCoreAddons import KJob<br />
RuntimeError: the PyQt5.QtCore module failed to register with the sip module</p>

<p>Unfortunately the solution is not as easy as checking the PyQt version, since some distros (such as Debian) patch PyQt to use the default 'sip' sipname. Instead we read the sipname PyQt was compiled with from PYQT_CONFIGURATION and use it to compile the KF5 bindings.</p></div></div><br /><div><strong>TEST PLAN</strong><div><p>Compiled kcoreaddons against PyQt 5.10.1 (with default 'sip' sipname), PyKF5.KCoreAddons is correctly imported<br />
Compiled kcoreaddons against PyQt 5.11.2 (with 'PyQt5.sip' sipname), PyKF5.KCoreAddons is correctly imported<br />
Without this patch, they fail to load with PyQt 5.11.2</p></div></div><br /><div><strong>REPOSITORY</strong><div><div>R240 Extra CMake Modules</div></div></div><br /><div><strong>REVISION DETAIL</strong><div><a href="https://phabricator.kde.org/D15091">https://phabricator.kde.org/D15091</a></div></div><br /><div><strong>AFFECTED FILES</strong><div><div>find-modules/run-sip.py</div></div></div><br /><div><strong>To: </strong>arojas, Frameworks, bruns<br /><strong>Cc: </strong>kde-frameworks-devel, kde-buildsystem, michaelh, ngraham, bruns<br /></div>