Merge pull request #61 from sebroeder/only-tap-completions-when-needed

Only tap homebrew/completions when $completion parameter is set to true
boxen · Aug 14, 2015 · e454f30 · e454f30
2 parents c04a895 + fcdc197
commit e454f30
Showing 1 changed file with 5 additions and 3 deletions.
diff --git a/manifests/init.pp b/manifests/init.pp
@@ -10,7 +10,7 @@
 ) {
   validate_bool($completion)
 
-  $ensure_pkg = $completion ? {
+  $ensure_completion = $completion ? {
     true    => 'present',
     default => 'absent',
   }
@@ -26,10 +26,12 @@
     require => Package["Vagrant_${version}"],
   }
 
-  homebrew::tap { 'homebrew/completions': }
+  homebrew::tap { 'homebrew/completions':
+    ensure => $ensure_completion,
+  }
 
   package { 'vagrant-completion':
-    ensure   => $ensure_pkg,
+    ensure   => $ensure_completion,
     provider => 'homebrew',
     require  => Homebrew::Tap['homebrew/completions'],
   }