[ 
https://issues.apache.org/jira/browse/JOSHUA-248?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=15322242#comment-15322242
 ] 

Tommaso Teofili commented on JOSHUA-248:
----------------------------------------

I'm digging a bit deeper on a per file basis on the specific licensing 
requirements.

For 
[BasicPhrase|https://github.com/apache/incubator-joshua/blob/master/src/main/java/org/apache/joshua/corpus/BasicPhrase.java],
 
[Alignment|https://github.com/apache/incubator-joshua/blob/master/src/main/java/org/apache/joshua/subsample/Alignment.java],
 
[BiCorpus|https://github.com/apache/incubator-joshua/blob/master/src/main/java/org/apache/joshua/subsample/BiCorpus.java],
 
[PhrasePair|https://github.com/apache/incubator-joshua/blob/master/src/main/java/org/apache/joshua/subsample/PhrasePair.java],
 
[PhraseReader|https://github.com/apache/incubator-joshua/blob/master/src/main/java/org/apache/joshua/subsample/PhraseReader.java],
 
[Subsampler|https://github.com/apache/incubator-joshua/blob/master/src/main/java/org/apache/joshua/subsample/Subsampler.java],
 
[SubsamplerCLI|https://github.com/apache/incubator-joshua/blob/master/src/main/java/org/apache/joshua/subsample/SubsamplerCLI.java]
 I think we can replace the current header with the AL2 header as it seems the 
current header was required when Joshua was (L)GPL licensed.

[LexicalFeatures|https://github.com/apache/incubator-joshua/blob/master/src/main/java/org/apache/joshua/decoder/ff/LexicalFeatures.java]
 is completely missing the license header, I'll add the AL2 one.

I'll go ahead and fix them.

> Add Apache License headers to Joshua code
> -----------------------------------------
>
>                 Key: JOSHUA-248
>                 URL: https://issues.apache.org/jira/browse/JOSHUA-248
>             Project: Joshua
>          Issue Type: Task
>            Reporter: Tommaso Teofili
>            Assignee: Tommaso Teofili
>             Fix For: 6.1
>
>         Attachments: JOSHUA-248.0.patch
>
>
> Joshua source code should include standard headers with AL2.0.



--
This message was sent by Atlassian JIRA
(v6.3.4#6332)

Reply via email to