search

evertrue / zookeeper-cookbook

Chef cookbook for installing and managing Zookeeper.
https://supermarket.chef.io/cookbooks/zookeeper
Apache License 2.0
81 stars 117 forks source link

Always include build-essential #118

Closed kppullin closed 9 years ago

kppullin commented 9 years ago

Build tools including GCC, patch, etc must be installed in order to build the zookeeper and json gems. build-essential handles this in the install recipe, however the build-essential recipe is only included when the use_java_cookbook attribute is set to true.

This patch unconditionally includes the build-essential recipe.

This was previously addressed in the exhibitor recipe for issue #73/#74, however the exhibitor recipe no longer exists.

Tested with CentOS 6.6 and Ubuntu 14.04.

jeffbyrnes commented 9 years ago

:+1: on this, merging in, as it is definitely a solid fix.