Skip to content

Commit

Permalink
Packages can't have undef names in future parser
Browse files Browse the repository at this point in the history
FreeBSD sets undef as its dev package list. Spec tests with the future
parser claim this is invalid. This probably should not have worked with
the current parser anyway. This patch adds a conditional so that dev
packages are explicitly installed or not installed if they are needed.
  • Loading branch information
Colleen Murphy committed Mar 24, 2015
1 parent 6314dec commit 09afbca
Showing 1 changed file with 5 additions and 3 deletions.
8 changes: 5 additions & 3 deletions manifests/dev.pp
Original file line number Diff line number Diff line change
Expand Up @@ -4,8 +4,10 @@
}
include ::apache::params
$packages = $::apache::params::dev_packages
package { $packages:
ensure => present,
require => Package['httpd'],
if $packages { # FreeBSD doesn't have dev packages to install
package { $packages:
ensure => present,
require => Package['httpd'],
}
}
}

0 comments on commit 09afbca

Please sign in to comment.