Skip to content

nix: Support specifying systemd target#1385

Merged
LuckShiba merged 1 commit intoAvengeMedia:masterfrom
bailey-busc:bailey/push-kumytqzsnzqy
Jan 16, 2026
Merged

nix: Support specifying systemd target#1385
LuckShiba merged 1 commit intoAvengeMedia:masterfrom
bailey-busc:bailey/push-kumytqzsnzqy

Conversation

@bailey-busc
Copy link
Copy Markdown
Contributor

This is a pretty common pattern in home manager: https://home-manager-options.extranix.com/?query=systemd.target

It's not necessarily true that one will always want to bind to the default target, like in the case that one is running DMS in Niri but sometimes running other compositor sessions where one doesn't want to run DMS. This supports that case in a cleaner way than mkForceing the unit configuration. I opted to just duplicate the option between the home manager module and the NixOS module due to NixOS not having anything analogous to config.wayland.systemd.target.

@bailey-busc bailey-busc force-pushed the bailey/push-kumytqzsnzqy branch from d37b5db to 8610e62 Compare January 16, 2026 04:14
@bailey-busc bailey-busc force-pushed the bailey/push-kumytqzsnzqy branch from 8610e62 to 25003f3 Compare January 16, 2026 04:22
Copy link
Copy Markdown
Collaborator

@LuckShiba LuckShiba left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

We already have this option in the nixpkgs module, so it's fair to have here too. Thanks :)

@LuckShiba LuckShiba merged commit aab4b67 into AvengeMedia:master Jan 16, 2026
2 checks passed
shalevc1098 added a commit to shalevc1098/DankMaterialShell that referenced this pull request Jan 16, 2026
@shalevc1098
Copy link
Copy Markdown
Contributor

hey, this pr seems to have introduced a syntax error in distro/nix/home.nix - nix complains about a missing semicolon at the end of the file. opened #1386 about it

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants