Skip to content
This repository has been archived by the owner on Jul 1, 2024. It is now read-only.

Update download url w/r/t version #229

Open
wants to merge 1 commit into
base: master
Choose a base branch
from

Conversation

mgrenonville
Copy link

In version 3.5+, tarball is named like apache-zookeeper, while in 3.4
and before, it was just zookeeper.

In version 3.5+, tarball is named like `apache-zookeeper`, while in 3.4
and before, it was just `zookeeper`.
jeffbyrnes added a commit to jeffbyrnes/zookeeper-cookbook that referenced this pull request Jun 25, 2022
Per evertrue#229, the tarball changed naming scheme
as of ZooKeeper v3.5.0.

Not sure how this cookbook has continued to work but this should
definitely fix things.

H/t to @mgrenoville & @MEDIARITHMICS for pointing this out & their code.
@jeffbyrnes
Copy link
Contributor

@mgrenonville I lost write access to this repo at some point, but I still have the ability to publish the cookbook to the Supermarket, so I’ll take care of this via jeffbyrnes#2 & publish a new version of the cookbook.

I’ll include quite a few other fixes, since I’ve got a little spare time right now, so it’ll probably be a breaking version. If you need this as a non-breaking release, let me know & I’ll backport it to the v14.x version series.

@jeffbyrnes
Copy link
Contributor

Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants