var isn't needed in server config, but it's nice to require the config dir, and...

diff --git a/manifests/server/config.pp b/manifests/server/config.pp
index e9b1ed96b27c043dea2b0608269f707f589b964f..9699857bb4eec278921b0c2200bde1e330eb1147 100644 (file)
--- a/manifests/server/config.pp
+++ b/manifests/server/config.pp
@@ -1,11 +1,19 @@
 class samba::server::config {
-  
+
+  file { "/etc/samba":
+    ensure => directory,
+    owner => "root",
+    group => "root",
+    mode => 0755,
+  }
+
   file { "/etc/samba/smb.conf":
-    ensure => $ensure,
-    owner => root,
-    group => root,
-    require => Class["samba::server::install"],
+    ensure => present,
+    owner => "root",
+    group => "root",
+    mode => 0644,
+    require => [File["/etc/samba"], Class["samba::server::install"]],
     notify => Class["samba::server::service"]
   }
-  
-}
\ No newline at end of file
+
+}