----------------------------------------------------------- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/3887/#review5904 -----------------------------------------------------------
http://svn.apache.org/repos/asf/shindig/trunk/NOTICE <https://reviews.apache.org/r/3887/#comment12865> Because it is redundant. The exact same text is already provided just above this section. http://svn.apache.org/repos/asf/shindig/trunk/NOTICE <https://reviews.apache.org/r/3887/#comment12867> Because all these removed notices concern attributions already and sufficiently provided through their included license in the LICENSE file. And their license doesn't require additional attribution beyond that. Note also the removed notice for OpenAjax. The OpenAjax implementation doesn't claim any copyright at all and is released under a verbatim copy of the ASL 2.0 license. So doesn't needs any specific or additional attribution beyond the ASL 2.0 license. See for further background and argument: http://apache.org/legal/resolved.html#required-third-party-notices Base principal is: the NOTICE file should only contain *required* attributions, and nothing beyond those, as our ASL 2.0 license section 4d) requires downstream users to redistribute everything which is provided in our NOTICE file (that is: when applicable) http://svn.apache.org/repos/asf/shindig/trunk/php/NOTICE <https://reviews.apache.org/r/3887/#comment12868> Same answer as for the removed notices from the root NOTICE file: Because all these removed notices concern attributions already and sufficiently provided through their included license in the LICENSE file. And these licenses (all BSD or MIT based) don't require additional attribution beyond that. See for further background and argument: http://apache.org/legal/resolved.html#required-third-party-notices Base principal is: the NOTICE file should only contain *required* attributions, and nothing beyond those, as our ASL 2.0 license section 4d) requires downstream users to redistribute everything which is provided in our NOTICE file (that is: when applicable) - Ate On 2012-02-14 00:03:44, Ate Douma wrote: > > ----------------------------------------------------------- > This is an automatically generated e-mail. To reply, visit: > https://reviews.apache.org/r/3887/ > ----------------------------------------------------------- > > (Updated 2012-02-14 00:03:44) > > > Review request for shindig. > > > Summary > ------- > > This is a patch for resolving the remaining legal NOTICE and LICENSE > requirements as reported here: > http://mail-archives.apache.org/mod_mbox/shindig-dev/201201.mbox/%[email protected]%3E > > The patch also includes fixes needed for Shindig PHP, and updated Copyright > and license claim for OpenSocial 0.8 usages, see also: > https://groups.google.com/d/msg/opensocial-and-gadgets-spec/-/YYIFdxVlwD8J > > > This addresses bug SHINDIG-1689. > https://issues.apache.org/jira/browse/SHINDIG-1689 > > > Diffs > ----- > > http://svn.apache.org/repos/asf/shindig/trunk/LICENSE 1243336 > http://svn.apache.org/repos/asf/shindig/trunk/NOTICE 1243336 > http://svn.apache.org/repos/asf/shindig/trunk/extras/NOTICE 1243336 > > http://svn.apache.org/repos/asf/shindig/trunk/extras/src/main/appended-resources/META-INF/NOTICE > PRE-CREATION > http://svn.apache.org/repos/asf/shindig/trunk/features/LICENSE 1243336 > http://svn.apache.org/repos/asf/shindig/trunk/features/NOTICE 1243336 > http://svn.apache.org/repos/asf/shindig/trunk/features/pom.xml 1243336 > > http://svn.apache.org/repos/asf/shindig/trunk/features/src/main/appended-resources/META-INF/LICENSE > PRE-CREATION > > http://svn.apache.org/repos/asf/shindig/trunk/features/src/main/appended-resources/META-INF/NOTICE > PRE-CREATION > http://svn.apache.org/repos/asf/shindig/trunk/java/LICENSE 1243336 > http://svn.apache.org/repos/asf/shindig/trunk/java/NOTICE 1243336 > > http://svn.apache.org/repos/asf/shindig/trunk/java/server/src/main/appended-resources/META-INF/LICENSE > PRE-CREATION > > http://svn.apache.org/repos/asf/shindig/trunk/java/server/src/main/appended-resources/META-INF/NOTICE > PRE-CREATION > > http://svn.apache.org/repos/asf/shindig/trunk/java/uber/src/main/appended-resources/META-INF/LICENSE > PRE-CREATION > > http://svn.apache.org/repos/asf/shindig/trunk/java/uber/src/main/appended-resources/META-INF/NOTICE > PRE-CREATION > http://svn.apache.org/repos/asf/shindig/trunk/php/LICENSE 1243336 > http://svn.apache.org/repos/asf/shindig/trunk/php/NOTICE 1243336 > > Diff: https://reviews.apache.org/r/3887/diff > > > Testing > ------- > > > Thanks, > > Ate > >
