From 16bac72ec736c403cc59f1388bd067cf2f214bb1 Mon Sep 17 00:00:00 2001
From: Silvio Rhatto <rhatto@riseup.net>
Date: Sun, 6 Dec 2009 14:17:44 -0200
Subject: Expliciting dependency order

---
 manifests/gitosis.pp | 26 ++++++++++++++------------
 1 file changed, 14 insertions(+), 12 deletions(-)

(limited to 'manifests/gitosis.pp')

diff --git a/manifests/gitosis.pp b/manifests/gitosis.pp
index d09e11d..2cf19ab 100644
--- a/manifests/gitosis.pp
+++ b/manifests/gitosis.pp
@@ -7,17 +7,24 @@ class gitosis inherits git {
     ensure => directory,
     mode   => 0755,
     owner  => gitosis,
-    group  => gitosis;
+    group  => gitosis,
   }
 
   # symbolic link
   file { "/var/cache/git":
     ensure => "/var/git/repositories",
-    force => true,
+    force  => true,
   }
 
   # the needed packages
-  package { "gitosis": ensure => installed; }
+  package { "gitosis": ensure => installed }
+
+  # ensures that the group exists
+  group { "gitosis":
+    ensure    => present,
+    allowdupe => false;
+    require   => Package["gitosis"],
+  }
 
   # alters the user's home dir
   user { "gitosis":
@@ -28,18 +35,13 @@ class gitosis inherits git {
     shell     => "/bin/sh",
     gid       => "gitosis",
     groups    => [ "puppet" ],
-    require   => Group["gitosis"];
-  }
-
-  # ensures that the group exists
-  group { "gitosis":
-    ensure => present,
-    allowdupe => false;
+    require   => Group["gitosis"],
   }
 
   # tries to get rid of ugly directory structure
   file { "/srv/gitosis":
-    ensure => absent,
-    force  => true;
+    ensure  => absent,
+    force   => true,
+    require => User["gitosis"],
   }
 }
-- 
cgit v1.2.3