Github user hanm commented on the issue:
https://github.com/apache/zookeeper/pull/165
@eolivelli - can you please also closing this pull request?
On a side note, this patch works, I see my PR to branch 3.5 is now kicking
build bots.
---
If your project is set up for it, you can
Github user hanm commented on the issue:
https://github.com/apache/zookeeper/pull/165
Thanks @eolivelli, this is merged in 3.4. and 3.5.
---
If your project is set up for it, you can reply to this email and have your
reply appear on GitHub as well. If your project does not have this
Github user eolivelli commented on the issue:
https://github.com/apache/zookeeper/pull/165
@hanm the patch is almost the same as for trunk, just a work of "copy and
paste"
Eventually you can compare the diff with #97
---
If your project is set up for it, you can reply to
Github user hanm commented on the issue:
https://github.com/apache/zookeeper/pull/165
I think it is fine to leave the names as they are for now and later update
scripts for all branches all together.
@eolivelli - are there any specific changes you made on top of
Github user eolivelli commented on the issue:
https://github.com/apache/zookeeper/pull/165
@eribeiro that you for reviewing this.
IMHO it it just a variable name. The script is basically the same as in the
trunk version. I think it would be better to leave all versions as much