Review Request 119454: Make FindPyKDE4 work with PyQt's new build system.

Raphael Kubo da Costa rakuco at FreeBSD.org
Tue Jul 29 18:10:02 BST 2014


-----------------------------------------------------------
This is an automatically generated e-mail. To reply, visit:
https://git.reviewboard.kde.org/r/119454/
-----------------------------------------------------------

(Updated July 29, 2014, 5:10 p.m.)


Status
------

This change has been marked as submitted.


Review request for kdelibs, Scott Kitterman, Luca Beltrame, and Andrea Scarpino.


Bugs: 327633
    http://bugs.kde.org/show_bug.cgi?id=327633


Repository: kdelibs


Description
-------

Since PyQt 4.10, PyQt.pyqtconfig is deprecated and not available unless
PyQt is built using the old configure script.

PyKDE4 itself has recently been adjusted to mimic PyQt itself and only
install its pykdeconfig module if pyqtconfig is present. Additionally,
information such as SIP flags and the directory where PyKDE's SIP files
are installed are now also provided in the
PyKDE4.kdecore.PYKDE_CONFIGURATION dict.

This commit completely rewrites FindPyKDE4.py to make it look like
FindPyQt.cmake after commit a7e4743: most of the information used by
FindPyKDE4.cmake is fetched from PyKDE4.kdecore, and we first try to
obtain the SIP flags and directory from pykdeconfig and, if it fails, we
use PYKDE_CONFIGURATION.

Furthermore, FindPyKDE4.py now only prints the variables that are
actually consumed by FindPyKDE4.cmake -- it is not possible to obtain
all the data provided by pykdeconfig in PYKDE_CONFIGURATION. We've also
stopped reading and setting PYKDE4_VERSION_TAG, since PyKDE4 stopped
setting a KDE tag in 2008 (and its value was 3_92_0 at the time).


Diffs
-----

  cmake/modules/FindPyKDE4.cmake 98f7c37477393731152d2d8fd51fc50cb9bbd9ef 
  cmake/modules/FindPyKDE4.py e436c2a907efe5e8f764e14ed921d3a87d413f34 

Diff: https://git.reviewboard.kde.org/r/119454/diff/


Testing
-------

Modules such as Kate's Pate plugin detects and builds fine without pyqtconfig and pykdeconfig on the system.


Thanks,

Raphael Kubo da Costa

-------------- next part --------------
An HTML attachment was scrubbed...
URL: <http://mail.kde.org/pipermail/kde-core-devel/attachments/20140729/3c304fdf/attachment.htm>


More information about the kde-core-devel mailing list