Percona/mysql/postgres shouldn't be included by default

jesseadams / sphinx-cookbook

Chef cookbook for Sphinx Search (searchd)

14 stars 27 forks source link

Percona/mysql/postgres shouldn't be included by default #35

Closed bricker closed 2 years ago

bricker commented 9 years ago

The percona-chef cookbook rudely monkey-patches the MysqlChefGem provider and makes it impossible to install a different mysql client with this cookbook.

Plus, having these as dependencies adds a bunch of unnecessary attributes and cookbooks to my nodes.

Instead, the README should specify that you must declare "percona", "postgres", or "mysql" as a dependency in your cookbook in order to compile sphinx with them.

alainchiasson commented 9 years ago

My workaround has been to add depends 'percona', '~> 0.15.0' in my metadata.rb . That version of the cookbook does not contain the offending code.

There are a few forks which patch this by removing the percona dependancy - are there any that could be merged back ?