From 4b0c492cce4a6ed7af8e79a84f11af6df77f2672 Mon Sep 17 00:00:00 2001
From: Anand Kumria <akumria@acm.org>
Date: Mon, 18 Jun 2012 01:54:25 +0100
Subject: [PATCH 1/2] String interpolation rather than concatisation is the
 norm with puppet.

Opps. No idea why this slipped through. Apologies.
---
 manifests/vhost.pp | 2 +-
 1 file changed, 1 insertion(+), 1 deletion(-)

diff --git a/manifests/vhost.pp b/manifests/vhost.pp
index 093033161e..9d2fb2ec26 100644
--- a/manifests/vhost.pp
+++ b/manifests/vhost.pp
@@ -46,7 +46,7 @@
     $options            = $apache::params::options,
     $apache_name        = $apache::params::apache_name,
     $vhost_name         = $apache::params::vhost_name,
-    $logroot            = "/var/log/" + $apache::params::apache_name
+    $logroot            = "/var/log/$apache::params::apache_name"
   ) {
 
   include apache

From 4cadac81822a74f61cd32ce6cafd6aaf8077fe4e Mon Sep 17 00:00:00 2001
From: Anand Kumria <akumria@acm.org>
Date: Mon, 18 Jun 2012 02:14:05 +0100
Subject: [PATCH 2/2] Another syntax error with a missing ",".

Sigh.
---
 manifests/vhost.pp | 2 +-
 1 file changed, 1 insertion(+), 1 deletion(-)

diff --git a/manifests/vhost.pp b/manifests/vhost.pp
index 9d2fb2ec26..33fb3db46c 100644
--- a/manifests/vhost.pp
+++ b/manifests/vhost.pp
@@ -91,7 +91,7 @@
           Package['httpd'],
           File["${apache::params::vdir}/${priority}-${name}-$docroot"],
           File["${apache::params::vdir}/${priority}-${name}-$logroot"],
-      ]
+      ],
       notify  => Service['httpd'],
   }