From 09afbcacc82b8fe535695980464baedfd66efccc Mon Sep 17 00:00:00 2001
From: Colleen Murphy <colleen@puppetlabs.com>
Date: Wed, 11 Feb 2015 14:55:52 -0800
Subject: [PATCH] Packages can't have undef names in future parser

FreeBSD sets undef as its dev package list. Spec tests with the future
parser claim this is invalid. This probably should not have worked with
the current parser anyway. This patch adds a conditional so that dev
packages are explicitly installed or not installed if they are needed.
---
 manifests/dev.pp | 8 +++++---
 1 file changed, 5 insertions(+), 3 deletions(-)

diff --git a/manifests/dev.pp b/manifests/dev.pp
index 4eaeb5578..6bd865606 100644
--- a/manifests/dev.pp
+++ b/manifests/dev.pp
@@ -4,8 +4,10 @@
   }
   include ::apache::params
   $packages = $::apache::params::dev_packages
-  package { $packages:
-    ensure  => present,
-    require => Package['httpd'],
+  if $packages { # FreeBSD doesn't have dev packages to install
+    package { $packages:
+      ensure  => present,
+      require => Package['httpd'],
+    }
   }
 }