Mark, that you for your reply. I've made the change that you recommended. You can see it here:
https://github.com/apache/incubator-tinkerpop/blob/tp30/gremlin-console/src/main/LICENSE#L249-L250 On Fri, Oct 9, 2015 at 4:59 PM, Mark Thomas <ma...@apache.org> wrote: > On 09/10/2015 19:12, Stephen Mallette wrote: > > Hello all, > > > > I feel like I'm on the verge of getting LICENSE/NOTICE right in all > > respects in The TinkerPop. . Seems like there's one piece that's out of > > place and could use some advice as I wasn't able to map anything I'd read > > in the Apache docs on this topic to my issue. We currently shade several > > libraries: Kryo, minlog, objenesis, and jackson, essentially re-packaging > > those binary dependencies in a jar called gremlin-shaded.jar. The > > gremlin-shaded.jar is then made part of our zip distributions. > > > > As we didn't re-package the source code of these libs, I figured that the > > source LICENSE/NOTICE didn't need to change and that it was just the > binary > > LICENSE/NOTICE that needed to change. I assume this situation is > somewhat > > similar to a project that builds an uber jar for distribution. > > > > So that's where i'm about stuck. I took a wild stab at it and did this: > > > > > https://github.com/apache/incubator-tinkerpop/blob/5a8b61c09868e22a415c0469a2737d68acce9a4e/gremlin-console/src/main/LICENSE#L226-L228 > > > > but I have no idea if that is sufficient (or just plain wrong). If > anyone > > could offer some pointers on what needs to happen here to our binary > > LICENSE/NOTICE, that would be nice. I'd really like to see us get a > clean > > bill of health on the next LICENSE/NOTICE that goes through a release > vote. > > Looks good to me. The only thing I'd consider adding is what package > they were renamed to. > > Mark > > > --------------------------------------------------------------------- > To unsubscribe, e-mail: general-unsubscr...@incubator.apache.org > For additional commands, e-mail: general-h...@incubator.apache.org > >