conda-forge / pyqt-feedstock

A conda-smithy repository for pyqt.
BSD 3-Clause "New" or "Revised" License
5 stars 36 forks source link

Missing libpyqt5.so #44

Closed hhslepicka closed 5 years ago

hhslepicka commented 5 years ago

Issue: On previous releases of the PyQt package for this channel the PyQt plugin was available but at the latest it is gone.

What changed? Can we have a release with the plugin back? If not, what is the recommended approach?

Thank you

hhslepicka commented 5 years ago

Comparing the logs produced by CircleCI for Python under Linux I saw this:

The Qt Designer plugin was disabled because a dynamic Python library couldn't
be found.
The qmlscene plugin was disabled because a dynamic Python library couldn't be
found.

I will keep my investigation going and add more evidence that can help to find a solution in here...

Looks like at the new gcc7 label, which is also used for the PyQt missing the designer plugin, python is statically linked (ref. https://github.com/conda-forge/python-feedstock/issues/222#issuecomment-436432209)... in the version that has the plugin Python was dynamically linked and that is the reason why the plugin was built and installed.

Question remaining... How to address this? Should we patch the configure.py to bypass this verification? What would be the downside of it?

more to come...

hhslepicka commented 5 years ago

Here is a workaround for other people facing the same problem:

conda install -c conda-forge/label/cf201901 pyqt

This will install a version that still has the plugin.

CJ-Wright commented 5 years ago

The newer version has been built with gcc7. Relevent PRs: https://github.com/conda-forge/pyqt-feedstock/pull/41 https://github.com/conda-forge/pyqt-feedstock/pull/42

CJ-Wright commented 5 years ago

Also: attn: @conda-forge/pyqt

hhslepicka commented 5 years ago

@CJ-Wright exactly. I'm wondering if a patch to configure.py would be a good way out of this situation.

CJ-Wright commented 5 years ago

I have no idea. At this point I'd put up a PR and see if it works. People may be able to comment better on a PR than an issue.

ocefpaf commented 5 years ago

@hhslepicka please test this against the new packages and, if the problem persists, re-open the issue.

hhslepicka commented 5 years ago

@ocefpaf the library is there now but with linking issues. The new issue is open.