nix: add nixos module for noctalia#3024
Conversation
|
Thanks for the PR :) I’ll take a proper look at it once I get home. Did you test whether the app launcher works when running Noctalia via the systemd service? |
The app launcher launches and you can search just fine, but I noticed that certain apps won't launch through the app launcher. I assumed it was just a v5 instability issue, but I don't know the cause. |
I just tested it myself, and the launcher fails when I try to launch anything. Did you perhaps set anything beyond enabling the module and the systemd service that could make a difference? |
With verbose debug logging enabled in noctalia I can see these errors: The systemd service defined in nixos for noctalia doesn't have access to the users PATH. Therefore you get EDIT: I've update the module, exposing all system packages to the systemd service. The launcher now works as expected. |
5e137c8 to
fe7db5b
Compare
add a plain nixos module for noctalia which doesn't rely on any home managing frameworks
That approach doesn't cover user packages. |
I'm not sure what you mean by user packages in this instance. If you want globally installed packages on NixOS using |
I mean packages installed via users.users.<name>.packages |
Good catch, I've now updated the PR again so that it exposes both user and globally installed packages to the systemd service. I've also tested that this works locally on my machine. |
Might be worth testing if enableDefaultPath false alone does the trick without setting PATH explicitly. |
yes, that works as well. I've removed the explicit PATH now |
Summary and Motivation
add a plain nixos module for noctalia
which doesn't rely on any home managing
frameworks
Type of Change
Testing
I tested this manually by enabling the module in my machines nixos config and verifying that the systemd service and package worked as expected which it did.
Manual Coverage
Checklist
CONTRIBUTING.md.just formatwith clang-format v22+ installed, or this PR has no code changes.assets/translations/en.json, or this PR adds no new user-facing strings.