py-pyqt4: switch from mac to x11 dist for dbus support

[okmacports@…]

I have ported the HPLIP software successfully and have been working on the GUI variant, which requires a Python module: ​http://bugs.debian.org/cgi-bin/bugreport.cgi?bug=520789

Specifically, the GUI app fails on this statement:

  from dbus.mainloop.qt import DBusQtMainLoop

this is the "dbus support module" in PyQt4, which is offered only for the X11 (not the mac) variant.

I went to the site that hosts the source code, which has been updated to version 4.8.2, which in turn requires the latest version of SIP. Both built with no problem with only minor changes in the Portfiles. (SIP only needed to change the version number and checksums.)

I have attached a diff for the working new Portfile for py26-pyqt4 enabling dbus support.

[okmacports@…]

Portfile diff for py26-pyqt4

[okmacports@…]

Portfile diff for py26-sip

[jmroot (Joshua Root)]

These were both updated 5 days ago.

[okmacports@…]

Attaching a new Portfile.diff (against the current @4.8.2) to address the dbus issue.

[okmacports@…]

diff against 4.8.2 Portfile

[ryandesign (Ryan Carsten Schmidt)]

You most likely shouldn't just change the distname and the checksums if you're not also going to update the version or revision as well. Doesn't changing the distname change what files get installed?

Does this issue affect py-pyqt4, py25-pyqt4, py27-pyqt4 and py31-pyqt4 as well?

[okmacports@…]

Replying to ryandesign@…:

You most likely shouldn't just change the distname and the checksums if you're not also going to update the version or revision as well. Doesn't changing the distname change what files get installed?

Does this issue affect py-pyqt4, py25-pyqt4, py27-pyqt4 and py31-pyqt4 as well?

As I read the documentation on the "revision" keyword it should be used only if everyone who has an existing install should upgrade. I'll leave that decision to people with more authority than me. Can't change the version because that depends on the source version.

It surely does affect all those other ports, for people who would want dbus support. For me, that affects only HPLIP, which uses py26.

I don't know if this will build without having the dbus port installed. It seems to me that most people would not want to install dbus, so perhaps the best approach would be to make this a variant (like +gui, perhaps call it +dbus). In any case, the sole difference between the existing py26-pyqt4 and the proposed new version is that it is built from the x11 tree rather than the mac tree at the source. (The mac tree does not have the dbus module; the x11 tree does.) All the patches remain the same.

Thanks for taking a look.

[nerdling (Jeremy Lavergne)]

It sounds like we should create a py26-pyqt4-x11 and rename the existing one to py26-pyqt4-mac.

[okmacports@…]

Replying to snc@…:

It sounds like we should create a py26-pyqt4-x11 and rename the existing one to py26-pyqt4-mac.

That's fine to me, but then what about all the other versions? (py25, py31, etc.) In any event, since the only difference is dbus support, perhaps py26-pyqt4 for the one and py26-pyqt4-dbus for the other (which is why I suggested the +dbus variant -- just one package, with the existing name, where you could turn dbus on or off depending on the variant).

[michaelld (Michael Dickens)]

If this ticket is still valid, can you do the following:

sudo port clean py26-pyqt4
sudo port selfupdate

and then try to install or upgrade py26-pyqt4 again. There's a new qt4-mac and py26-pyqt4 that might fix this issue.

[jmroot (Joshua Root)]

py26-pyqt4 is no more; newer versions remain. Looks like this is probably either already fixed (in a different way) or heading for a wontfix resolution though.

[petrrr]

Cc Me!

[chrstphrchvz (Christopher Chavez)]

I think this ticket can be closed. qt4-x11 was recently removed ([4a4c1313f8/macports-ports]), which I would think would have been required for this change.

[petrrr]

Obsolete