Removed owner attribute from the pip class as it was not needed and touched up a few mistakes.

1 files changed, 7 insertions, 8 deletions

diff --git a/manifests/pip.pp b/manifests/pip.pp
index 0693057..3940374 100644
--- a/manifests/pip.pp
+++ b/manifests/pip.pp
@@ -28,13 +28,12 @@
 # Sergey Stankevich
 #
 define python::pip (
-  $virtualenv = 'system',
-  $ensure     = present,
-  $url        = false,
-  $owner      = 'root',
-  $group      = 'root',
-  $proxy      = false,
-  environment = []
+  $ensure      = present,
+  $virtualenv  = 'system',
+  $url         = false,
+  $owner       = 'root',
+  $proxy       = false,
+  $environment = []
 ) {
 
   # Parameter validation
@@ -42,7 +41,7 @@ define python::pip (
     fail('python::pip: virtualenv parameter must not be empty')
   }
 
-  if $virtualenv == 'system' and ($owner != 'root' or $group != 'root') {
+  if $virtualenv == 'system' and $owner != 'root' {
     fail('python::pip: root user must be used when virtualenv is system')
   }