clean maintdb.pp from all puppetlint errors

1 files changed, 51 insertions, 54 deletions

diff --git a/modules/buildsystem/manifests/maintdb.pp b/modules/buildsystem/manifests/maintdb.pp
index 58e67a9f..156821e1 100644
--- a/modules/buildsystem/manifests/maintdb.pp
+++ b/modules/buildsystem/manifests/maintdb.pp
@@ -1,56 +1,53 @@
-class buildsystem {
-    class maintdb {
-    	include sudo
-        $login = "maintdb"
-        $homedir = "/var/lib/maintdb"
-        $dbdir = "$homedir/db"
-        $binpath = "/usr/local/sbin/maintdb"
-        $dump = "/var/www/bs/data/maintdb.txt"
-        $unmaintained = "/var/www/bs/data/unmaintained.txt"
-
-        user {"$login":
-            comment => "Maintainers database",
-            home => "$homedir",
-        }
-
-        file { ["$homedir","$dbdir"]:
-            ensure => directory,
-            owner => "$login",
-            group => "$login",
-            mode => 711,
-            require => User["$login"],
-        }
-
-        file { "$binpath":
-            mode => 755,
-            content => template("buildsystem/maintdb/maintdb.bin")
-        }
-
-        local_script { "wrapper.maintdb":
-            content => template("buildsystem/maintdb/wrapper.maintdb")
-        }
-
-        sudo::sudoers_config { "maintdb":
-            content => template("buildsystem/maintdb/sudoers.maintdb")
-        }
-
-        file { ["$dump","$dump.new",
-                "$unmaintained","$unmaintained.new"]:
-            owner => $login,
-            require => File["/var/www/bs/data"],
-        }
-
-        cron { "update maintdb export":
-            user => $login,
-            command => "$binpath root get > $dump.new; cp -f $dump.new $dump; grep ' nobody\$' $dump | sed 's/ nobody\$//' > $unmaintained.new; cp -f $unmaintained.new $unmaintained",
-            minute => "*/30",
-            require => User[$login],
-        }
-
-        apache::vhost_base { "maintdb.$domain":
-            location => $dbdir,
-            content => template("buildsystem/maintdb/vhost_maintdb.conf"),
-        }
+class buildsystem::maintdb {
+    include sudo
+    $login = 'maintdb'
+    $homedir = '/var/lib/maintdb'
+    $dbdir = "$homedir/db"
+    $binpath = '/usr/local/sbin/maintdb'
+    $dump = '/var/www/bs/data/maintdb.txt'
+    $unmaintained = '/var/www/bs/data/unmaintained.txt'
+
+    user { $login:
+        comment => 'Maintainers database',
+        home    => $homedir,
+    }
+
+    file { [$homedir,$dbdir]:
+        ensure  => directory,
+        owner   => $login,
+        group   => $login,
+        mode    => '0711',
+        require => User[$login],
+    }
+
+    file { $binpath:
+        mode    => '0755',
+        content => template('buildsystem/maintdb/maintdb.bin')
+    }
+
+    local_script { 'wrapper.maintdb':
+        content => template('buildsystem/maintdb/wrapper.maintdb')
     }
-}
 
+    sudo::sudoers_config { 'maintdb':
+        content => template('buildsystem/maintdb/sudoers.maintdb')
+    }
+
+    file { [$dump,"$dump.new",
+            $unmaintained,"$unmaintained.new"]:
+        owner   => $login,
+        require => File['/var/www/bs/data'],
+    }
+
+    cron { 'update maintdb export':
+        user    => $login,
+        command => "$binpath root get > $dump.new; cp -f $dump.new $dump; grep ' nobody\$' $dump | sed 's/ nobody\$//' > $unmaintained.new; cp -f $unmaintained.new $unmaintained",
+        minute  => '*/30',
+        require => User[$login],
+    }
+
+    apache::vhost_base { "maintdb.$::domain":
+        location => $dbdir,
+        content  => template('buildsystem/maintdb/vhost_maintdb.conf'),
+    }
+}