Compare $::is_pe as bool, not string

stdlib now provides an $::is_pe fact, which defaults to "false". When compared as a string, it is evaluated to true.
jguiditta · Jun 19, 2015 · 17b836b · 17b836b
1 parent 130a334
commit 17b836b
Showing 1 changed file with 1 addition and 1 deletion.
diff --git a/manifests/init.pp b/manifests/init.pp
@@ -205,7 +205,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 defined('$is_pe') and $::is_pe {
+    if defined('$is_pe') and str2bool("${::is_pe}") { # lint:ignore:only_variable_string
       if $::kernel == 'windows' {
         $command_path = "${::env_windows_installdir}/bin:${::path}"
       } else {