From 59d7a3a75c70a446b26a6d5501eb9042611e2749 Mon Sep 17 00:00:00 2001 From: Colleen Murphy Date: Mon, 23 Feb 2015 09:51:44 -0800 Subject: [PATCH] Lookup is_pe fact with getvar If the system is not running PE, the $::is_pe fact will be undefined, causing errors when run under strict variables mode. This patch uses the getvar() function to look up the fact so that it won't fail if not defined. Alternative to https://github.com/puppetlabs/puppetlabs-concat/pull/270 This should fix the broken CI for the postgresql module. --- manifests/init.pp | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-) diff --git a/manifests/init.pp b/manifests/init.pp index 78c408149..1ec86bd38 100644 --- a/manifests/init.pp +++ b/manifests/init.pp @@ -199,7 +199,7 @@ $command = strip(regsubst("${script_command} -o \"${fragdir}/${concat_name}\" -d \"${fragdir}\" ${warnflag} ${forceflag} ${orderflag} ${newlineflag}", '\s+', ' ', 'G')) # make sure ruby is in the path for PE - if $::is_pe { + if getvar('::is_pe') { if $::kernel == 'windows' { $command_path = "${::env_windows_installdir}/bin:${::path}" } else {