search

juanje / cookbook-redmine

Chef's Cookbook for installing Redmine
25 stars 21 forks source link

Missing requirements #1

Closed tuxinaut closed 12 years ago

tuxinaut commented 12 years ago

I have tested the cookbook and noticed that exists missing requirements.

juanje commented 12 years ago

I was thinking about those depends and I din't add because they are already depends of some of the others cookbooks (build-essential of passenger_apache2 and openssl of mysql).

Chef should warn if those cookbooks are missed, but probably is good idea to put them at the README so the users install them before try and fail.

Thanks for your time and contribution :-)

tuxinaut commented 12 years ago

Thanks for merging. Yesterday i was to quick and don't look at the rest depends. After the error message i was a little bit frustrated :) The Pull request should avoid other user to make the same mistake.