search

criteo-cookbooks / nssm

Chef cookbook to install the Non-Sucking Service Manager
http://nssm.cc
MIT License
13 stars 15 forks source link

Include default recipe in provide only if required #4

Closed Annih closed 8 years ago

Annih commented 8 years ago

recipe_eval method converges at each run, updating the nssm resource. This is not a good pattern & even worse when using notify/subscribe.

This PR fix this behavior by checking if the default recipe has been already included.

cc. @aboten

Annih commented 8 years ago

Thanks for your quick reactivity =]

dhoer commented 8 years ago

No, Thank you for the making nssm better!