bindir makes heaps more sense

Author: dgoodlad

manifests/db.pp

@@ -10,12 +10,12 @@
 
   exec { "postgresql-db-${name}":
     command => join([
-      "${postgresql::binpath}/createdb",
+      "${postgresql::bindir}/createdb",
       "-p${postgresql::port}",
       '-E UTF-8',
       "-O ${postgresql::user}",
       $name
     ], ' '),
-    unless  => "${postgresql::binpath}/psql -aA -p${postgresql::port} -t -l | cut -d \\| -f 1 | grep -w '${name}'"
+    unless  => "${postgresql::bindir}/psql -aA -p${postgresql::port} -t -l | cut -d \\| -f 1 | grep -w '${name}'"
   }
 }

manifests/init.pp

@@ -6,7 +6,7 @@
 class postgresql(
   $ensure     = $postgresql::params::ensure,
 
-  $binpath    = $postgresql::params::binpath,
+  $bindir     = $postgresql::params::bindir,
   $executable = $postgresql::params::executable,
 
   $host       = $postgresql::params::host,

manifests/params.pp

@@ -5,7 +5,7 @@
     Darwin: {
       include boxen::config
 
-      $binpath    = "${boxen::config::homebrewdir}/bin"
+      $bindir     = "${boxen::config::homebrewdir}/bin"
       $executable = "${binpath}/postgres"
       $datadir    = "${boxen::config::datadir}/postgresql"
       $logdir     = "${boxen::config::logdir}/postgresql"