msonnabaum / chef-drush

Drush chef cookbook
27 stars 33 forks source link

Remove subversion from suggested cookbook list. #25

Closed mparker17 closed 11 years ago

mparker17 commented 11 years ago

Drush provides integration with Subversion; but also with Bazaar, curl or wget, rsync, gzip, open or xdg-open, etc, and presumably other programs in the future.

IMO, we should either include suggestions for the aforementioned programs or avoid referencing programs that work with Drush altogether.

If we avoid referencing programs that work with Drush, then we can avoid chasing Drush HEAD as often.

patcon commented 11 years ago

suggests isn't actually used yet in chef, but semantically it means exactly how we use it:

suggests: Adds a dependency on another cookbook that is suggested, but not required. This field is weaker than recommends; a cookbook will still work even when suggested dependencies are not available.

http://docs.opscode.com/essentials_cookbook_metadata.html

But I'm easy, and consistency makes sense :)