[Backport release-24.05] nixos/tsm-client: Fix multi-value dsm.sys options #328031
Conversation
![github-actions[bot]](https://avatars.githubusercontent.com/in/15368?s=60&v=4){kind=small}
A configuration such as:
programs.tsmClient.servers.backup.domain = [ "/dir1" "dir2" ];
...would previously result in an error ("cannot coerce a list to a
string"), since `makeDsmSysLines` would return a nested list.
(cherry picked from commit f52ee2a)
ofborg
bot
added
10.rebuild-darwin: 0
Review per https://github.com/NixOS/nixpkgs/blob/d95b1060ca298c62f56de90d0bf755c4c6b48e0a/nixos/README.md#module-updatesReviewed points
Possible improvements(none) CommentsTested again, this time with NixOS 24.05 (stable). I can confirm: After applying the pull request at hand, values with a list, I approve this pull request and recommend it to be merged. (Note: I'm the maintainer of the |
|
This pull request has been mentioned on NixOS Discourse. There might be relevant details there: https://discourse.nixos.org/t/prs-already-reviewed/2617/1836 |
|
This pull request has been mentioned on NixOS Discourse. There might be relevant details there: https://discourse.nixos.org/t/prs-already-reviewed/2617/1866 |
|
This pull request has been mentioned on NixOS Discourse. There might be relevant details there: https://discourse.nixos.org/t/prs-already-reviewed/2617/1907 |
|
This pull request has been mentioned on NixOS Discourse. There might be relevant details there: https://discourse.nixos.org/t/prs-already-reviewed/2617/1926 |
Bot-based backport to
release-24.05, triggered by a label in #327748.