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
Drop kdm and kde4 modules #22677
Drop kdm and kde4 modules #22677
Conversation
Excluding modules/programs/environment.nix for PATHand QT_PLUGIN_PATH to allow the programs to continue running.
That should be fine. |
As a data point on quassel, I'm using the qt5 version and it seems fine. In all fairness my total experience with IRC clients in general is a whopping 12 hours so I may not have exercised all the features... |
👍 |
@@ -25,12 +25,12 @@ in | |||
|
|||
package = mkOption { | |||
type = types.package; | |||
default = pkgs.kde4.quasselDaemon; | |||
defaultText = "pkgs.kde4.quasselDaemon"; | |||
default = pkgs.quasselDaemon_qt5; |
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 might be missing something here but it doesn't looks like quasselDaemon_qt5
exists in all-packages.nix
. I think this should quasselDaemon
(which is still qt5).
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.
See #22853.
Motivation for this change
re: #15866
cc: @ttuegel @vcunat @globin @domenkozar @dezgeg @fpletz @edolstra
One thing I'm not sure about is reverting @Phreedom's pinning of Quassel to KDE4.
Things done
(nix.useSandbox on NixOS,
or option
build-use-sandbox
innix.conf
on non-NixOS)
nix-shell -p nox --run "nox-review wip"
./result/bin/
)