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
qt 5.12.0 -> 5.12.3 WIP #60119
qt 5.12.0 -> 5.12.3 WIP #60119
Conversation
See also #57042 |
Does the configure script produce a log file, so we could see what compilation tests it's doing? |
Yes there was. The first
|
qtbase compiles in the latest version of this patch. |
The current patch compiles, but does not work with X11: a KDE system with it does not start the graphical login This is the configuration report of Qt 5.12.0 next to that of Qt 5.12.3:
The GLX Plugin and EGL-X11 Plugin are missing, as is the XLib detection. |
The current version of the patch detects X properly. All the features detected in Qt 5.12.0 are also detected in Qt 5.12.3. Testing will be done once the patched version and KDE software has compiled. |
I'm running my desktop now with Qt 5.12.3 (this patch) and the latest KDE software on NixOS 19.03. Everything works except the global KWin shortcuts (Alt-Tab, Ctrl-F1 etc). Other global shortcut like Alt-Space / Alt-F2, Meta-P still work. I'm not sure this is related to this patch. |
After rebooting the machine the global kwin shortcuts work again. In fact, the bug that I hoped would be fixed by upgrading from Qt 5.12.0 to Qt 5.12.3 is gone. https://bugs.kde.org/show_bug.cgi?id=406829 |
On Mon, 06 May 2019 16:13:59 -0700, Jos van den Oever ***@***.***> wrote:
After rebooting the machine the global kwin shortcuts work again. In fact, the bug that I hoped would be fixed by upgrading from Qt 5.12.0 to Qt 5.12.3 is gone. https://bugs.kde.org/show_bug.cgi?id=406829
I don't run KDE, but use lots of qt-based/kde software -- and so far
this is working great here as well! <3
…
--
You are receiving this because you are subscribed to this thread.
Reply to this email directly or view it on GitHub:
#60119 (comment) part: text/html
|
pkgs/top-level/all-packages.nix
Outdated
@@ -12717,7 +12717,7 @@ in | |||
inherit stdenv fetchurl fetchFromGitHub makeSetupHook; | |||
bison = bison2; # error: too few arguments to function 'int yylex(... | |||
inherit cups; | |||
harfbuzz = harfbuzzFull; | |||
inherit harfbuzz; |
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.
This should be under qt512
in order for qtbase
to pass configurePhase
.
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.
Fixed!
Builds fine in sandbox on |
In the meantime qt 5.12.4 has been released. |
5.12.4 seems close to 5.12.3, but I'm having trouble "sanely" forward-porting the |
I'm not sure we would be able to move to it piecemeal, but notably qt 5.12.4 supports openssl 1.1 which might motivate sorting these things out eventually :). |
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.
This looks good to me! We can worry about Qt 5.12.4 later. I'm not going to "approve" because this should go into staging
; @vandenoever would you mind opening a new pull request? Sorry 🙁
Closing in favor of #64598 |
Motivation for this change
Qt 5.12 has received 3 bugfix releases since 5.12.0. However updating it has proven non-trivial. This PR is a WIP.
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)