Purge and Recurse should be set together (#399)

okopop · okopop · web-flow · commit 2d19df9ac39c · 2024-12-24T16:51:37.000+01:00
If you for whatever reason decide to change the value of purge, recurse should change with it. This is nothing new and the solution as it's implemented here is the same I have seen in other popular modules (saz-sudo, ghoneycutt-ssh) regarding the relation between purge and recurse. If you don't you can get unexpected behaviour. Related to #390 Co-authored-by: okopop <okopop@localhost.com>
diff --git a/manifests/server/config.pp b/manifests/server/config.pp
@@ -55,7 +55,7 @@
       group   => 0,
       mode    => $ssh::server::include_dir_mode,
       purge   => $ssh::server::include_dir_purge,
-      recurse => true,
+      recurse => $ssh::server::include_dir_purge,
     }
 
     $ssh::server::config_files.each |$file, $params| {

Original file line number	Diff line number	Diff line change
`@@ -55,7 +55,7 @@`
`55`	`55`	`group => 0,`
`56`	`56`	`mode => $ssh::server::include_dir_mode,`
`57`	`57`	`purge => $ssh::server::include_dir_purge,`
`58`		`- recurse => true,`
	`58`	`+ recurse => $ssh::server::include_dir_purge,`
`59`	`59`	`}`
`60`	`60`
`61`	`61`	`$ssh::server::config_files.each \|$file, $params\| {`