From 1ee285d39980512b506084368f4f2208aa4d7c1f Mon Sep 17 00:00:00 2001
From: intrigeri <intrigeri@boum.org>
Date: Sun, 11 Nov 2012 21:43:03 +0100
Subject: Use single-quotes for strings that do not contain variables.

Puppet style guide, section "8. Quoting", reads: "All strings that do not
contain variables should be enclosed in single quotes."
---
 manifests/daemon.pp        |  6 +++---
 manifests/init.pp          |  2 +-
 manifests/munin.pp         | 16 ++++++++--------
 manifests/polipo/base.pp   | 12 ++++++------
 manifests/polipo/debian.pp |  6 +++---
 5 files changed, 21 insertions(+), 21 deletions(-)

diff --git a/manifests/daemon.pp b/manifests/daemon.pp
index 28d257d..683d2c6 100644
--- a/manifests/daemon.pp
+++ b/manifests/daemon.pp
@@ -136,11 +136,11 @@ class tor::daemon (
                   $ensure                  = present ) {
 
     if $cookie_authentication == '0' and $hashed_control_password == '' and $ensure != 'absent' {
-      fail("You need to define the tor control password")
+      fail('You need to define the tor control password')
     }
 
     if $cookie_authentication == 0 and ($cookie_auth_file != '' or $cookie_auth_file_group_readable != '') {
-      notice("You set a tor cookie authentication option, but do not have cookie_authentication on")
+      notice('You set a tor cookie authentication option, but do not have cookie_authentication on')
     }
     
     concatenated_file_part { '04.control':
@@ -178,7 +178,7 @@ class tor::daemon (
     }
     
     file { '/etc/tor/tor.html':
-      source  => "puppet:///modules/tor/tor.html",
+      source  => 'puppet:///modules/tor/tor.html',
       require => File['/etc/tor'],
       ensure  => $ensure,
       owner => 'debian-tor', group => 'debian-tor', mode => 0644, 
diff --git a/manifests/init.pp b/manifests/init.pp
index 613eb29..b8e1917 100644
--- a/manifests/init.pp
+++ b/manifests/init.pp
@@ -3,7 +3,7 @@ class tor (
   $use_munin      = false,
 ){
 
-  package { [ "tor", "tor-geoipdb" ]:
+  package { [ 'tor', 'tor-geoipdb' ]:
     ensure => $ensure_version,
   }
 
diff --git a/manifests/munin.pp b/manifests/munin.pp
index 8504f89..24ea53e 100644
--- a/manifests/munin.pp
+++ b/manifests/munin.pp
@@ -1,23 +1,23 @@
 class tor::munin {
 
   file {
-    "/usr/local/share/munin-plugins/tor_connections":
-      source => "puppet:///modules/tor/munin/tor_connections",
+    '/usr/local/share/munin-plugins/tor_connections':
+      source => 'puppet:///modules/tor/munin/tor_connections',
       mode => 0755, owner => root, group => root;
     
-    "/usr/local/share/munin-plugins/tor_routers":
-      source => "puppet:///modules/tor/munin/tor_routers",
+    '/usr/local/share/munin-plugins/tor_routers':
+      source => 'puppet:///modules/tor/munin/tor_routers',
       mode => 0755, owner => root, group => root;
 
-    "/usr/local/share/munin-plugins/tor_traffic":
-      source => "puppet:///modules/tor/munin/tor_traffic",
+    '/usr/local/share/munin-plugins/tor_traffic':
+      source => 'puppet:///modules/tor/munin/tor_traffic',
       mode => 0755, owner => root, group => root;
   }
 
   munin::plugin {
-    [ "tor_connections", "tor_routers", "tor_traffic" ]:
+    [ 'tor_connections', 'tor_routers', 'tor_traffic' ]:
       ensure => present,
       config => "user debian-tor\n env.cookiefile /var/run/tor/control.authcookie",
-      script_path_in => "/usr/local/share/munin-plugins";
+      script_path_in => '/usr/local/share/munin-plugins';
   }
 }
diff --git a/manifests/polipo/base.pp b/manifests/polipo/base.pp
index fca4b21..8c1dd1c 100644
--- a/manifests/polipo/base.pp
+++ b/manifests/polipo/base.pp
@@ -3,19 +3,19 @@ class tor::polipo::base {
     ensure => present,
   }
 
-  file { "/etc/polipo/config":
+  file { '/etc/polipo/config':
     ensure  => present,
     owner   => root,
     group   => root,
     mode    => 0644,
-    source  => "puppet:///modules/tor/polipo/polipo.conf",
-    require => Package["polipo"],
-    notify  => Service["polipo"],
+    source  => 'puppet:///modules/tor/polipo/polipo.conf',
+    require => Package['polipo'],
+    notify  => Service['polipo'],
   }
 
-  service { "polipo":
+  service { 'polipo':
     ensure  => running,
     enable  => true,
-    require => [ Package["polipo"], Service["tor"] ],
+    require => [ Package['polipo'], Service['tor'] ],
   }
 }
diff --git a/manifests/polipo/debian.pp b/manifests/polipo/debian.pp
index 1986119..0a8aa93 100644
--- a/manifests/polipo/debian.pp
+++ b/manifests/polipo/debian.pp
@@ -1,12 +1,12 @@
 class tor::polipo::debian inherits tor::polipo::base {
   # TODO: restore file to original state after the following bug is solved:
   # http://bugs.debian.org/cgi-bin/bugreport.cgi?bug=580434
-  file { "/etc/cron.daily/polipo":
+  file { '/etc/cron.daily/polipo':
     ensure  => present,
     owner   => root,
     group   => root,
     mode    => 0755,
-    require => Package["polipo"],
-    source  => "puppet:///modules/tor/polipo/polipo.cron",
+    require => Package['polipo'],
+    source  => 'puppet:///modules/tor/polipo/polipo.cron',
   }
 }
-- 
cgit v1.2.3