Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

[NETBEANS-1074] Module Review websvc.core #789

Merged

Conversation

vikasprabhakar
Copy link
Contributor

Checked the rat-report.txt file, are these files ok with Apache License ?
glassfish-license.txt
Apache Bug: https://issues.apache.org/jira/browse/NETBEANS-1189

Add the license header for binaries-list
Add the license header for java files
Add the license header for Bundle.properties
Add the licenseinfo.xml

@matthiasblaesing
Copy link
Contributor

The license of websvc.core-external-resources.zip is not correct and it would be better to include the files directly in the source.

We elect the CDDL as the license under which we use the files (add a licenseinfo.xml for this). We can include the source according to (second last paragraph):
https://www.apache.org/legal/resolved.html#category-b

This way the hole external binary can be dropped and we don't have to deal with the broken license.

@vikasprabhakar
Copy link
Contributor Author

Following steps taken to remove external folder

  1. Remove external folder
  2. Update build.xml file
  3. Restore xsd file to "/src/org/netbeans/modules/websvc/core/resources" folder
  4. Update license details in xsd file.

@matthiasblaesing
Copy link
Contributor

Great so far - I think the final nitpick: CDDL code needs reasoning to be included in ASF projects. Please see here:

https://github.com/apache/incubator-netbeans/blob/dffa844eece9ffc49dd4073db043d0807cb6e50e/web.core/licenseinfo.xml#L90-L102

I faced the same situation as you did. CATEGORY-B here references to an explanation that give the reasoning required in Apache Legal FAQ (second last paragraph)

@matthiasblaesing
Copy link
Contributor

Thank you - merging.

@matthiasblaesing matthiasblaesing merged commit a73c477 into apache:master Aug 28, 2018
@vikasprabhakar vikasprabhakar deleted the code_donation_websvc_core branch August 29, 2018 01:41
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

3 participants