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
services.plasma5: Update start menu with an activationScript #42910
Conversation
To update the plasma start menu `kbuildsyscoca5` needs to be executed. There are several people complaining about missing applications in their plasma start menu. This patch adds a activationScript for plasma, that runs `kbuildsyscoca5` for each user that has `isNormalUser` == `true`.
How about patching |
I would like to have an approach where we don't need to restart the display manager. |
# Update the start menu for each user that has `isNormalUser` set. | ||
system.activationScripts.plasmaSetup = stringAfter [ "users" "groups" ] | ||
(concatStringsSep "\n" | ||
(mapAttrsToList (name: value: "${pkgs.su}/bin/su ${name} -c kbuildsycoca5") |
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.
Shouldn't this be a direct path via ${kdeWhatever}/bin/kbuildsycoca5
? Does this affect users that aren't running kde (which is possible through a custom ~/.xsession file)?
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.
Does this affect users that aren't running kde (which is possible through a custom ~/.xsession file)?
This only applies if config.services.xserver.desktopManager.plasma5.enable
is true
so if somebody sets that and then overrides the environment using ~/.xsession
it would run even when KDE isn't active.
@peterhoeg @bkchr What's holding this up? |
Good question, I think we wait for @ttuegel . But I think that he does not have that much time lately? (based on missing feedback on several of my prs) |
I noticed this also. They ended up getting merged by someone else eventually... Nudge Nudge @peterhoeg 😄 |
We already do that, but like @bkchr said, we want this to happen without restarting the display manager if the user runs |
Great! Planned to backport to release-18.03? |
i am running kde5/plasma, and i am getting this error now on rebuild |
@nand0p on which version of nixpkgs are you running? Did you had kde/plasma installed before or is this on a fresh install? |
* silence normal output from the command * avoid running it when the data it wants to update is encrypted * specify explicit bash in kbuildsycoca5 su command when the activation script runs at boot time, the users' default shells (typically /run/current-system/sw/bin/bash) will not be available. instead, specify an explicit store path for bash. follow-on to NixOS#42910 and NixOS#44544
To update the plasma start menu
kbuildsyscoca5
needs to be executed.There are several people complaining about missing applications in their
plasma start menu.
This patch adds a activationScript for plasma, that runs
kbuildsyscoca5
for each user that hasisNormalUser
==true
.Motivation for this change
#41416
Fixes: #40915
#42548
Things done
@peterhoeg @ttuegel
sandbox
innix.conf
on non-NixOS)nix-shell -p nox --run "nox-review wip"
./result/bin/
)nix path-info -S
before and after)