Hi,

> The LICENSE file has a table that maps a file to its license "key" and the
> license is then stored in the (top-level) "licenses" directory (in the
> header of the table, there is a note about that: "text is in file in
> licenses directory"). We used to put all the licenses into the LICENSE
> file, but we were told that is confusing (as the LICENSE file gets fairly
> huge, esp. for the convenience binary), so the licenses were put into a
> separate directory.

Yep that the recommended way i.e. add a pointer to where the license is located 
rather than the full license text when there are many licenses. I think you are 
missing a couple, give me a couple of days and I’ll come up with a list for you.

> Or we can change the reference in the table in the LICENSE file to
> read like "licenses/CDDL-1.0" instead of "CDDL-1.0”. 

That I think would make it clearer.

> Do you mean licenses under nbbuild/licenses and under the modules, like:
> java/maven.embedder/external? The LICENSE file should not refer to them
> (that only ever refers to the top-level licenses directory), so these
> should be OK?

It OK it just makes it very hard to check the release for what is bundled and 
not and what needs to go in LICENSE.

>> 1.
>> ./ide/css.editor/test/unit/src/org/netbeans/modules/css/editor/module/main/properties/PropertiesATest.java
>> 
> 
> What is the problem with this file? Looking at the file, I am not sure
> what's the problem.

May of been a false positive as it contains the text:
"/*!* Many thanks to Sorin Stefan and Nicole Sullivan!*(Refers to section A. 
Libraries) * Copyright(c) 2007,Yahoo!Inc. All rights reserved. * Code licensed 
under the BSD License:* http://developer.yahoo.net/yui/license.txt * 
version:2.2.1 */“

But it seems odd to have that in there.

Thanks,
Justin
---------------------------------------------------------------------
To unsubscribe, e-mail: general-unsubscr...@incubator.apache.org
For additional commands, e-mail: general-h...@incubator.apache.org

Reply via email to