Skip to content

Conversation

@lschuermann
Copy link
Contributor

By allowing users to optionally force using systemd unit-based activation, they can inject dependencies on other services or mountpoints (for instance, when the age key is not stored on the root file system).

By allowing users to optionally force using systemd unit-based
activation, they can inject dependencies on other services or
mountpoints (for instance, when the age key is not stored on the root
file system).
@lschuermann
Copy link
Contributor Author

@Mic92 Only after opening this I realized this is effectively a duplicate of #39. Can either this or the other PR be merged? I have my key files stored on a persistent partition while the rest of the system is wiped each reboot, and with the activation-script based method sops-nix apparently always runs before that file system is mounted...

@Mic92 Mic92 merged commit b80c966 into Mic92:master Nov 13, 2025
5 checks passed
@Mic92
Copy link
Owner

Mic92 commented Nov 13, 2025

Reverted because it broke evaluation: #857

@lschuermann
Copy link
Contributor Author

lschuermann commented Nov 13, 2025

Reverted because it broke evaluation: #857

Huh? I'm confused by this error. I've run nix flake check and have been using this revision on my own systems without issues. Also, the default for the introduced option is a verbatim copy of the let ... in expression before my change. So I'm puzzled as to how this could be breaking for anyone. From the error it looks like config.system is not defined?

If either you @Mic92 or @l0b0 can tell me how to produce this issue, I can try to come up with a different solution to this issue...

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.

2 participants