search

evertrue / zookeeper-cookbook

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

fix for link error in ark #184

Closed Stromweld closed 8 years ago

Stromweld commented 8 years ago

This is a fix that I have tested and works to fix #183. I simply moved up the ark resource to be performed before the directories are created as ark will will link the the unpacked zookeeper archive with version number folder to the install_dir/zookeeper folder. The link step breaks though if the directory is created first before the link due to recursive directory declarations.

jeffbyrnes commented 8 years ago

@Stromweld thanks for providing the fix for your own issue! I’ll see about testing this myself (really do need to get Travis working for forks) and if it works, I’ll merge it in & cut a patch release.

Stromweld commented 8 years ago

thanks for taking a look

jeffbyrnes commented 8 years ago

Tests well, I’m going to merge this in & cut a release.