FS#43821 - [python2-pyqt4] non-standard path to sip files

Attached to Project: Arch Linux
Opened by Paul Hollensen (pallegro) - Saturday, 14 February 2015, 20:42 GMT
Last edited by Felix Yan (felixonmars) - Friday, 27 February 2015, 04:56 GMT
Task Type Bug Report
Category Packages: Extra
Status Closed
Assigned To Felix Yan (felixonmars)
Architecture All
Severity Low
Priority Normal
Reported Version
Due in Version Undecided
Due Date Undecided
Percent Complete 100%
Votes 0
Private No

Details

Description:
pyqt4 4.11.3-2 release changed the location of pyqt's .sip files in python2-pyqt4 from /usr/share/sip to /usr/share/sip/Py2-PyQt4.
This broke my build of python-qt-binding for ROS, since unfortunately pyqt4 no longer provides a way to determine the .sip files' location ('pyqt_sip_dir' in the pre 4.10 build system). AFAIK 'Py2-PyQt4' is not used by any other distribution, whereas /usr/share/sip/PyQt4 appears to be standard and hard-coded all over the place.

Since the sip files are identical for the python 2 and 3 packages and we already have a pyqt4-common package I propose moving the sip files there. This would make pyqt4-common match up with Fedora's PyQt4-devel. This would help with getting pyqt4 4.11 compatibility pushed to upstream packages. PKGBUILD diff attached. I could not find a way to get pyqt's root makefile to only install the sip files, so this manually copies them.
This task depends upon

Closed by  Felix Yan (felixonmars)
Friday, 27 February 2015, 04:56 GMT
Reason for closing:  Implemented
Additional comments about closing:  pyqt5 5.4.1-1
pyqt4 4.11.3-3
Comment by Felix Yan (felixonmars) - Friday, 27 February 2015, 03:35 GMT
There is a minor difference AFAIK:

$ diff -ru /usr/share/sip/PyQt4 /usr/share/sip/Py2-PyQt4
Only in /usr/share/sip/PyQt4: Qsci

I'll look into how Fedora handles this.
Comment by Paul Hollensen (pallegro) - Friday, 27 February 2015, 04:01 GMT
This is from python-qscintilla-common, and contains all of its sip files.
So this change to pyqt would just bring it in line with how qscintilla is packaged.
Comment by Felix Yan (felixonmars) - Friday, 27 February 2015, 04:13 GMT
Ah I see. Not sure how I missed that :P

Updating pyqt4 and pyqt5 for this change.

Loading...