Justin,

Thanks for the feedback!



On Thu, Jun 10, 2021 at 8:23 PM Justin Mclean <jmcl...@apache.org> wrote:

> HI,
>
> I'm not on your PMC but I took a look at your source release and I think
> there a few things that need to be added to the LICENSE file as per the
> guidelines here [1]
>
> These BSD licensed files:
>   ./zookeeper-client/zookeeper-client-c/src/hashtable/hashtable.c
>   ./zookeeper-client/zookeeper-client-c/src/hashtable/hashtable.h
>   ./zookeeper-client/zookeeper-client-c/src/hashtable/hashtable_itr.c
>   ./zookeeper-client/zookeeper-client-c/src/hashtable/hashtable_itr.h
>   ./zookeeper-client/zookeeper-client-c/src/hashtable/hashtable_private.h
>
> This MIT licensed file:
>
> ./zookeeper-contrib/zookeeper-contrib-loggraph/src/main/resources/webapp/org/apache/zookeeper/graph/resources/date.format.js
>
> This MIT licensed files:
>
> ./zookeeper-contrib/zookeeper-contrib-loggraph/src/main/resources/webapp/org/apache/zookeeper/graph/resources/g.bar.js
>
> ./zookeeper-contrib/zookeeper-contrib-loggraph/src/main/resources/webapp/org/apache/zookeeper/graph/resources/g.dot.js
>
> ./zookeeper-contrib/zookeeper-contrib-loggraph/src/main/resources/webapp/org/apache/zookeeper/graph/resources/g.line.js
>
> ./zookeeper-contrib/zookeeper-contrib-loggraph/src/main/resources/webapp/org/apache/zookeeper/graph/resources/g.pie.js
>
> ./zookeeper-contrib/zookeeper-contrib-loggraph/src/main/resources/webapp/org/apache/zookeeper/graph/resources/g.raphael.js
>
> This MIT licensed file
>
> ./zookeeper-contrib/zookeeper-contrib-loggraph/src/main/resources/webapp/org/apache/zookeeper/graph/resources/raphael.js
>
> This BSD licensed file:
>
> ./zookeeper-contrib/zookeeper-contrib-loggraph/src/main/resources/webapp/org/apache/zookeeper/graph/resources/yui-min.js
>
> This Apache 2.0 licensed file:
>
> ./zookeeper-contrib/zookeeper-contrib-zooinspector/src/main/java/com/nitido/utils/toaster/Toaster.java
>
> This MIT licensed file:
> ./zookeeper-docs/src/main/resources/markdown/skin/prototype.js
>
> Please note that I may not have found all the 3rd party files in the
> source release.
>
> Kind Regards,
> Justin
>
>
> 1. https://infra.apache.org/licensing-howto.html#permissive-deps
>

Reply via email to