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
qtkeychain: Fixes Qt4 build. #46085
qtkeychain: Fixes Qt4 build. #46085
Conversation
Success on x86_64-linux (full log) Attempted: qtkeychain Partial log (click to expand)
|
Success on x86_64-darwin (full log) Attempted: qtkeychain Partial log (click to expand)
|
Success on aarch64-linux (full log) Attempted: qtkeychain Partial log (click to expand)
|
Thanks for spotting and fixing this.
That would require us to adapt the dead and broken |
(cherry picked from commit c59d9d3)
backport: dea9a04 |
Motivation for this change
Not 100% enthusiastic at "fixing" Qt4 build as this is. Why? Because the only use (AFAICT) of Qt4 qtkeychain was tomahawk, which has been marked broken and is abandoned upstream. Furthermore, tomahawk supposedly can be built using Qt5 instead (according to recent unreleased commits). Thus this is untested. I won't be mad, possibly glad if instead we remove the Qt4 attribute of qtkeychain.
Though, in the vein of ZHF #45960, I have this fixed cooked and ready. (I cooked it before I could realize this wouldn't be tested.)
It'll need to be cherry-picked to 18.09.
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)@GrahamcOfBorg build qtkeychain libsForQt5.qtkeychain