Hi,

>> LICENSE issues:
>> - Looks like there may be  an issue with the app-packages directory.
> That's odd: I don't see it

find . -name "LIC*" -print
./apache-slider-0.80.0-incubating/app-packages/accumulo/LICENSE.txt
./apache-slider-0.80.0-incubating/app-packages/accumulo/target/classes/META-INF/LICENSE
./apache-slider-0.80.0-incubating/app-packages/accumulo/target/maven-shared-archive-resources/META-INF/LICENSE
./apache-slider-0.80.0-incubating/app-packages/accumulo/target/test-classes/META-INF/LICENSE
./apache-slider-0.80.0-incubating/app-packages/command-logger/application-pkg/target/maven-shared-archive-resources/META-INF/LICENSE
./apache-slider-0.80.0-incubating/app-packages/command-logger/slider-pkg/target/maven-shared-archive-resources/META-INF/LICENSE
./apache-slider-0.80.0-incubating/app-packages/hbase/target/maven-shared-archive-resources/META-INF/LICENSE
./apache-slider-0.80.0-incubating/app-packages/storm/target/maven-shared-archive-resources/META-INF/LICENSE
./apache-slider-0.80.0-incubating/app-packages/target/maven-shared-archive-resources/META-INF/LICENSE
(...)

> that just tells the merge checker which strings to merge together when 
> generating aggregate reports of what licenses you depend on. We don't have 
> any such dependencies; it's probably just there because we cut and paste it 
> in while setting up the license check. I can cut the line from the source if 
> you want; it doesn't make any difference to the build.

Probably best to remove it for future releases.

Thanks,
Justin

Reply via email to