Merge branch 'puppet4' into 'master'

Puppet4

Fixes stuff for puppet 4

See merge request !54

6 files changed, 17 insertions, 17 deletions

diff --git a/manifests/apt_conf.pp b/manifests/apt_conf.pp
index 949f615..fa8cfa3 100644
--- a/manifests/apt_conf.pp
+++ b/manifests/apt_conf.pp
@@ -1,15 +1,15 @@
 define apt::apt_conf(
   $ensure = 'present',
-  $source = '',
+  $source = undef,
   $content = undef,
   $refresh_apt = true )
 {
 
-  if $source == '' and $content == undef {
+  if $source == undef and $content == undef {
     fail("One of \$source or \$content must be specified for apt_conf ${name}")
   }
 
-  if $source != '' and $content != undef {
+  if $source != undef and $content != undef {
     fail("Only one of \$source or \$content must specified for apt_conf ${name}")
   }
 
diff --git a/manifests/init.pp b/manifests/init.pp
index 4bf126c..3d93d22 100644
--- a/manifests/init.pp
+++ b/manifests/init.pp
@@ -40,14 +40,14 @@ class apt(
       mode    => '0644';
   }
 
-  apt_conf { '02show_upgraded':
+  ::apt::apt_conf { '02show_upgraded':
     source => [ "puppet:///modules/site_apt/${::fqdn}/02show_upgraded",
                 'puppet:///modules/site_apt/02show_upgraded',
                 'puppet:///modules/apt/02show_upgraded' ]
   }
 
   if ( $::virtual == 'vserver' ) {
-    apt_conf { '03clean_vserver':
+    ::apt::apt_conf { '03clean_vserver':
       source => [ "puppet:///modules/site_apt/${::fqdn}/03clean_vserver",
                   'puppet:///modules/site_apt/03clean_vserver',
                   'puppet:///modules/apt/03clean_vserver' ],
@@ -55,7 +55,7 @@ class apt(
     }
   }
   else {
-    apt_conf { '03clean':
+    ::apt::apt_conf { '03clean':
       source => [ "puppet:///modules/site_apt/${::fqdn}/03clean",
                   'puppet:///modules/site_apt/03clean',
                   'puppet:///modules/apt/03clean' ]
diff --git a/manifests/preferences_snippet.pp b/manifests/preferences_snippet.pp
index b7dba0d..2b2fb74 100644
--- a/manifests/preferences_snippet.pp
+++ b/manifests/preferences_snippet.pp
@@ -2,9 +2,9 @@ define apt::preferences_snippet (
   $priority = undef,
   $package = false,
   $ensure = 'present',
-  $source = '',
-  $release = '',
-  $pin = ''
+  $source = undef,
+  $release = undef,
+  $pin = undef
 ) {
 
   $real_package = $package ? {
@@ -36,9 +36,9 @@ define apt::preferences_snippet (
   }
 
   case $source {
-    '': {
+    undef: {
       case $release {
-        '': {
+        undef: {
           File["/etc/apt/preferences.d/${name}"]{
             content => template('apt/preferences_snippet.erb')
           }
diff --git a/manifests/proxy_client.pp b/manifests/proxy_client.pp
index 9ba79f2..e7972f7 100644
--- a/manifests/proxy_client.pp
+++ b/manifests/proxy_client.pp
@@ -3,7 +3,7 @@ class apt::proxy_client(
   $port = '3142',
 ){
 
-  apt_conf { '20proxy':
+  ::apt::apt_conf { '20proxy':
     content => template('apt/20proxy.erb'),
   }
 }
diff --git a/manifests/sources_list.pp b/manifests/sources_list.pp
index 7565bce..6ff4fc3 100644
--- a/manifests/sources_list.pp
+++ b/manifests/sources_list.pp
@@ -1,15 +1,15 @@
 # Configure an apt source
 define apt::sources_list (
-  $ensure = 'present',
-  $source = '',
+  $ensure  = 'present',
+  $source  = undef,
   $content = undef
 ) {
 
   if $ensure == 'present' {
-    if $source == '' and $content == undef {
+    if $source == undef and $content == undef {
       fail("One of \$source or \$content must be specified for apt_sources_snippet ${name}")
     }
-    if $source != '' and $content != undef {
+    if $source != undef and $content != undef {
       fail("Only one of \$source or \$content must specified for apt_sources_snippet ${name}")
     }
   }
diff --git a/manifests/unattended_upgrades.pp b/manifests/unattended_upgrades.pp
index 52d7542..d604125 100644
--- a/manifests/unattended_upgrades.pp
+++ b/manifests/unattended_upgrades.pp
@@ -26,7 +26,7 @@ class apt::unattended_upgrades (
     default => $config_content
   }
 
-  apt_conf { '50unattended-upgrades':
+  ::apt::apt_conf { '50unattended-upgrades':
     content     => $file_content,
     require     => Package['unattended-upgrades'],
     refresh_apt => false