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
python.pkgs.pyqt5: 5.11.3 -> 5.12 #56354
Conversation
Not yet ready for merge. I still have to investigate why running qutebrowser yields
|
According to https://www.riverbankcomputing.com/news/pyqt-512, "the QtWebEngine, QtWebEngineCore and QtWebEngineWidgets modules have been moved to a separate PyQtWebEngine package." I'm currently packaging that one. |
Now I'm getting
That's still true when merging #56349. |
mkdir -p "$out/share/sip/PyQt5" | ||
|
||
# FIXME: Without --no-dist-info, I get | ||
# unable to create /nix/store/yv4pzx3lxk3lscq0pw3hqzs7k4x76xsm-python3-3.7.2/lib/python3.7/site-packages/PyQtWebEngine-5.12.dist-info |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
I really don't see any configure option to fix this :(
Could you rebase? |
Done. Note there's a new version out. However, I currently don't have time to look into this any further or work on nixpkgs at all. |
Build fails for me:
|
format = "other"; | ||
|
||
src = fetchurl { | ||
url = "https://www.riverbankcomputing.com/static/Downloads/PyQtWebEngine/PyQtWebEngine_gpl-${version}.tar.gz"; |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
nix-prefetch-url "https://www.riverbankcomputing.com/static/Downloads/PyQtWebEngine/PyQtWebEngine_gpl-5.12.tar.gz"
[0.0 MiB DL]
error: unable to download 'https://www.riverbankcomputing.com/static/Downloads/PyQtWebEngine/PyQtWebEngine_gpl-5.12.tar.gz': HTTP error 404
url = "https://www.riverbankcomputing.com/static/Downloads/PyQtWebEngine/PyQtWebEngine_gpl-${version}.tar.gz"; | |
url = "https://www.riverbankcomputing.com/static/Downloads/PyQtWebEngine/${version}/PyQtWebEngine_gpl-${version}.tar.gz"; |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
That, and version 5.12 doesn't exist anymore — you need 5.12.1
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
It does:
nix-prefetch-url https://www.riverbankcomputing.com/static/Downloads/PyQtWebEngine/5.12/PyQtWebEngine_gpl-5.12.tar.gz
[0.0 MiB DL]
path is '/nix/store/8wdwbx0vir0jkyfv4cnry1x1p1pd4625-PyQtWebEngine_gpl-5.12.tar.gz'
0j9zzgwrvh61mrzjfcdlhr08vg931ycb53ri51vynsj0grp07smn
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Hah, the lack of version in path, compounded by the availability of the new version, tripped me up.
pname = "PyQt"; | ||
version = "5.11.3"; | ||
pname = "pyqt"; | ||
version = "5.12"; |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
version = "5.12"; | |
version = "5.12.1"; |
This can probably be fixed by adding more options into |
So I tried to contiue this by updating pyqt to 5.12.2 and pyqtwebengine to 5.12.1 since old versions are not available anymore. I can not make pyqtwebengine compile: |
I'm taking this further in #65836. |
Motivation for this change
#56349 (comment)
Things done
sandbox
innix.conf
on non-NixOS)nix-shell -p nox --run "nox-review wip"
./result/bin/
)nix path-info -S
before and after)