10.0.x forwarder of LICENSE and script adjustments + license adjustments for LEGAL-691

[porcelli]

Related: apache/incubator-kie-tools#2799

[jomarko]

Thank you for the PR, don't we need to add alos these drools LICENSE entries?

• drools-drl/drools-drl-parser/src/main/antlr4/org/drools/drl/parser/antlr4/JavaLexer.g4
• drools-drl/drools-drl-parser/src/main/antlr4/org/drools/drl/parser/antlr4/JavaParser.g4
• drools-model/drools-mvel-parser/src/main/javacc/mvel.jj
• drools-util/src/main/java/org/drools/util/StringUtils.java
• *pmml*

[jomarko]

Probably we should merge first this PR:
apache/incubator-kie-drools#6195

then we can probably drop highlighted lines from #1271

[tkobayas]

Thank you for the PR, don't we need to add alos these drools LICENSE entries?

* drools-drl/drools-drl-parser/src/main/antlr4/org/drools/drl/parser/antlr4/JavaLexer.g4

* drools-drl/drools-drl-parser/src/main/antlr4/org/drools/drl/parser/antlr4/JavaParser.g4

* drools-model/drools-mvel-parser/src/main/javacc/mvel.jj

* drools-util/src/main/java/org/drools/util/StringUtils.java

* _pmml_

Thank you for the points, @jomarko . I think it's better to combine License files in main branch than backporting 10.0.x

[jomarko]

Thank you for the PR, don't we need to add alos these drools LICENSE entries?

* drools-drl/drools-drl-parser/src/main/antlr4/org/drools/drl/parser/antlr4/JavaLexer.g4

* drools-drl/drools-drl-parser/src/main/antlr4/org/drools/drl/parser/antlr4/JavaParser.g4

* drools-model/drools-mvel-parser/src/main/javacc/mvel.jj

* drools-util/src/main/java/org/drools/util/StringUtils.java

* _pmml_

Thank you for the points, @jomarko . I think it's better to combine License files in main branch than backporting 10.0.x

Not sure I follow @tkobayas . You say should we or we should not add those entries ?

[tkobayas]

@jomarko , I mean we should add the entries.

Those entries are missed in this PR because Alex tries to backport the 10.0.x LICENSE file. But they exist in the main LICENSE. We maintain LICENSE files in main of kie repos, so combining them would naturally result in a valid single LICENSE for root. Does it make sense? @jomarko @porcelli

[porcelli]

correct @tkobayas! @jomarko my recommendation is to bring these files as is.. and we improve those after merge.

note that in this backport (actually forwardport) it also brings the script that adjusts the source zip

[tkobayas]

and we improve those after merge.

Yeah, we can improve it with a new PR after merge. It's good to merge this.

[jomarko]

Could you please share a ticket, epic, whatever where we track this plan for a future fixes?

[porcelli]

@jomarko apache/incubator-kie-issues#1739

[jomarko]

Thank you for sharing link for the ticket @porcelli