we don't require any of these execs, so we don't need them
authormh <mh@immerda.ch>
Sat, 25 Oct 2008 21:25:06 +0000 (21:25 +0000)
committerMicah Anderson <micah@riseup.net>
Thu, 2 Sep 2010 22:10:24 +0000 (18:10 -0400)
manifests/classes/lsb_release.pp

index 8455a56..73a9114 100644 (file)
@@ -19,7 +19,6 @@ class assert_lsbdistcodename {
        case $lsbdistcodename {
                '': {
                        err("Please install lsb_release or set facter_lsbdistcodename in the environment of $fqdn")
-                       exec { "false # assert_lsbdistcodename": alias => assert_lsbdistcodename }
                }
                'n/a': {
                        case $operatingsystem {
@@ -32,15 +31,9 @@ class assert_lsbdistcodename {
                        }
                        exec { "false # assert_lsbdistcodename": alias => assert_lsbdistcodename }
                }
-               default: {
-                       exec { "true # assert_lsbdistcodename": alias => assert_lsbdistcodename }
-                       exec { "true # require_lsbdistcodename": alias => require_lsbdistcodename }
-               }
        }
 
 }
 
 # To fail the complete compilation, include this class
-class require_lsbdistcodename inherits assert_lsbdistcodename {
-       exec { "false # require_lsbdistcodename": require => Exec[require_lsbdistcodename], }
-}
+class require_lsbdistcodename inherits assert_lsbdistcodename {}