Merged in oherrala/puppet/oherrala-array-not-string (pull request #44)

firewall, gnu, sendmail: Fix "Name must be a String not Array" errors produced by latest Puppet.

firewall/manifests/init.pp

@@ -122,19 +122,34 @@ class firewall::common::iptables {
         }
     }
 
+    case $::operatingsystem {
+      'debian', 'ubuntu': {
         package { "iptables":
           ensure  => installed,
-        name   => $::operatingsystem ? {
+          require => Package["iptables-persistent"],
-            "centos" => [ "iptables", "iptables-ipv6" ],
+        }
-            "redhat" => [ "iptables", "iptables-ipv6" ],
+        package { "iptables-persistent":
-            "debian" => [ "iptables", "iptables-persistent" ],
+          ensure => installed,
-            "fedora" => $::operatingsystemrelease ? {
+        }
-                /^1[0-5]/ => [ "iptables", "iptables-ipv6" ],
+      }
-                /^1[6-7]/ => "iptables",
+      'centos','redhat': {
-                default   => [ "iptables", "iptables-services" ],
+        package { "iptables":
-            },
+          ensure  => installed,
-            "ubuntu" => [ "iptables", "iptables-persistent" ],
+          require => Package["iptables-ipv6"],
-        },
+        }
+        package { "iptables-ipv6":
+          ensure => installed,
+        }
+      }
+      'fedora': {
+        package { "iptables":
+          ensure  => installed,
+          require => Package["iptables-services"],
+        }
+        package { "iptables-services":
+          ensure => installed,
+        }
+      }
     }
 
     file { $iptables:

gnu/manifests/init.pp

@@ -10,15 +10,24 @@ class gnu::gcc {
     }
 
     if $::kernel == Linux {
+      case $::operatingsystem {
+        'debian', 'ubuntu': {
           package { "kernel-headers":
-            name   => $::operatingsystem ? {
+            name   => "linux-libc-dev",
-                "debian" => "linux-libc-dev",
-                "ubuntu" => "linux-libc-dev",
-                default  => [ "kernel-headers", "kernel-devel", ],
-            },
             ensure => installed,
           }
         }
+        default: {
+          package { "kernel-headers":
+            ensure  => installed,
+            require => Package["kernel-devel"],
+          }
+          package { "kernel-devel":
+            ensure => installed,
+          }
+        }
+      }
+    }
 
 }
 

sendmail/manifests/init.pp

@@ -9,8 +9,11 @@ class sendmail::common {
     if $::operatingsystem != "OpenBSD" {
         package { "sendmail":
             ensure => installed,
-            name   => [ "sendmail", "sendmail-cf", ],
             before => File["/etc/mail/certs"],
+            require => Package["sendmail-cf"],
+        }
+        package { "sendmail-cf":
+          ensure => installed,
         }
         service { "sendmail":
             ensure  => running,