Review Request 119302: Make FindPyQt4 work with PyQt's new build system.
Raphael Kubo da Costa
rakuco at FreeBSD.org
Wed Jul 16 14:21:44 BST 2014
-----------------------------------------------------------
This is an automatically generated e-mail. To reply, visit:
https://git.reviewboard.kde.org/r/119302/
-----------------------------------------------------------
(Updated July 16, 2014, 4:21 p.m.)
Review request for kdelibs, Alex Merry, Luca Beltrame, and Simon Edwards.
Changes
-------
Use a default value for `pyqt_sip_dir`.
Bugs: 337462
http://bugs.kde.org/show_bug.cgi?id=337462
Repository: kdelibs
Description (updated)
-------
Since PyQt 4.10, PyQt.pyqtconfig is deprecated and not available unless
PyQt is built using the old configure script.
There is no direct replacement for it, as PyQt's new build system does
not provide as much information as before by design. Luckily, most of
the variables we are interested in can be obtained from PyQt's QtCore
module itself even if its old build system is used.
The only exception is pyqt_sip_dir, which cannot be determined at all if
pyqtconfig is not available. In this case, the most we can do is guess
the default path like QScintilla2 does, and fail if it does not exist.
The user then needs to specify it manually via CMake with something like
-DPYQT4_SIP_DIR=/usr/share/sip/PyQt4. To this effect, all variables set
by FindPyQt4.cmake have been made cache variables, which means their
values can be overriden by the user, thus ignoring the contents read via
FindPyQt.py.
Diffs (updated)
-----
cmake/modules/FindPyQt.py 5d2f9514d87553d5a16a95943618572316c92861
cmake/modules/FindPyQt4.cmake b176b4f8cfee471a1b7aecdd2723d165b0496a85
Diff: https://git.reviewboard.kde.org/r/119302/diff/
Testing
-------
I was able to make Kate find PyQt by passing `-DPYQT4_SIP_DIR=<...>` with my PyQt installation without `pyqtconfig.py`, and calling `FindPyQt.py` by hand on a Debian system with `pyqtconfig.py` worked as before.
Thanks,
Raphael Kubo da Costa
-------------- next part --------------
An HTML attachment was scrubbed...
URL: <http://mail.kde.org/pipermail/kde-core-devel/attachments/20140716/ff6e3cbb/attachment.htm>
More information about the kde-core-devel
mailing list