search

claranet / puppet-consul_template

A Puppet module to manage the config and jobs of Consul Template from Hashicorp
Apache License 2.0
30 stars 89 forks source link

Require package on init script #67

Closed scarby closed 7 years ago

scarby commented 8 years ago

because our package actually includes an init script - we should probably requre the package first.

gdhbashton commented 8 years ago

Yah I can see how the wrong ordering could be annoying - are you using a homebrew package for consul-template ?

How do your require => .... changes affect the default situation where the install_method is the default of url ? In that case $requires will be undef - Does require => undef not throw an exception?

craigwatson commented 7 years ago

Hi, as this PR hasn't seen any activity in a number of months, I'm closing it. Please do feel free to fix the bug mentioned above and re-open.