Hi Justin, Just a quick question for adding the NOTICE file, if we just use netty binary file, do we still need to add the NOTICE about the copyright of netty?
According this[1], I don't think we need to add NOTICE information for the binary usage of Netty. [1]https://softwareengineering.stackexchange.com/questions/234511/what-is-the-best-practice-for-arranging-third-party-library-licenses-paperwork Regards, Willem On 2018/09/08 01:34:36, Justin Mclean <[email protected]> wrote: > Hi, > > >>> This product contains code derived from Google Guava project, which is > >>> available under a "Apache License 2.0" license. > >>> (https://github.com/google/guava <https://github.com/google/guava>). > >>> > >>> This product contains code derived from the Netty Project, which is > >>> available under a "Apache License 2.0" license. > >>> (https://github.com/netty/netty <https://github.com/netty/netty>) > > First off license information should not be repeated in NOTICE [1], it > belongs in LICENSE and there no need to add URLs in NOTICE to repo unless it > is specified in the license conditions (so called required 3rd-party > notices). [2] > > The Guava project doesn’t have a NOTICE file so nothing needs to be added for > it. > > The Netty project has a long (and sadly IMO malformed) NOTICE file. > > I would suggest you just add: > > "This product contains code form the Netty Project: > > The Netty Project > ================= > Please visit the Netty web site for more information: > * http://netty.io/ > > Copyright 2014 The Netty Project” > > Normally I would not add the URL but as they have it in their NOTICE file we > should include it. [3] > > Also double check if any of the code copies from Netty is from anything > mentioned further down in Netty’s NOTICE file. If it is we would need to add > more to the NOTICE file. > > Thanks, > Justin > > 1. http://www.apache.org/dev/licensing-howto.html#mod-notice > 2. https://www.apache.org/legal/resolved.html#required-third-party-notices > 3. http://www.apache.org/dev/licensing-howto.html#alv2-dep > >
