New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
pythonPackages.pyqt5: upgrade and solve qml issue #65836
Conversation
Qutebrowser now runs without the error. However, there is a regression in PyQt5.sip for Python 2 it seems. |
I can imagine we can have collisions with sip as well because for each variant we create |
This upgrades pyqt5 and fixes the build. Note that the QtWebEngine module is split into `pyqtwebengine` by upstream and us. If your application gives an error that a Qt module is missing, then that is most likely because it was not included when building pyqt5. The solution is to simply add it here to the `(native)BuildInputs`. Don't forget to add an import check. We should include all the modules (except qtwebkit) that are needed.
5.13.0 is available, however, it assumes qt 5.13 is used and fails to build due to a missing QWebEngineNotification class.
qgis and calibre don't build yet, but I leave that for someone else (or at least another day). |
@GrahamcOfBorg build picard qutebrowser anki python3Packages.spyder |
Great news that this is finally figured out 🎉 |
Yes, I don't like it either. Maybe |
Motivation for this change
This PR upgrades
.pyqt5
and.pyqtwebengine
and fixes their build. During build-time many Qt modules were not detected and thus not included in the build. Apparently they also need to be added tonativeBuildInputs
. I suppose this is because of qmake?Anyway, both packages work fine when using with
python3.withPackages
.Things done
sandbox
innix.conf
on non-NixOS)nix-shell -p nix-review --run "nix-review wip"
./result/bin/
)nix path-info -S
before and after)Notify maintainers
cc @