Merge pull request #280 from covermymeds/virtualenv_version_fix

Force virtualenv_version to be a string.
voxpupuli · Jan 5, 2016 · e9417e2 · e9417e2
2 parents 05db472 + 36df8d8
commit e9417e2
Showing 1 changed file with 2 additions and 2 deletions.
diff --git a/manifests/virtualenv.pp b/manifests/virtualenv.pp
@@ -116,9 +116,9 @@
     # --system-site-packages flag, default off for prior versions
     # Prior to version 1.7 the default was equal to --system-site-packages
     # and the flag --no-site-packages had to be passed to do the opposite
-    if (( versioncmp($::virtualenv_version,'1.7') > 0 ) and ( $systempkgs == true )) {
+    if (( versioncmp("${::virtualenv_version}",'1.7') > 0 ) and ( $systempkgs == true )) { # lint:ignore:only_variable_string
       $system_pkgs_flag = '--system-site-packages'
-    } elsif (( versioncmp($::virtualenv_version,'1.7') < 0 ) and ( $systempkgs == false )) {
+    } elsif (( versioncmp("${::virtualenv_version}",'1.7') < 0 ) and ( $systempkgs == false )) { # lint:ignore:only_variable_string
       $system_pkgs_flag = '--no-site-packages'
     } else {
       $system_pkgs_flag = $systempkgs ? {