Open dhendriks opened 1 year ago
@waynebeaton Do you have any idea when this can be looked at?
The Dash License Tool just uses the standard Maven Reactor.
It looks like these warnings are all coming from Tycho.
It's not an option that is turned on explicitly by the Dash License Tool's mojo, so I don't offhand know how to turn it off.
I will investigate, but don't have a time frame for resolution. What repository are you running this against?
Have you tried playing with the logging levels?
I will investigate, but don't have a time frame for resolution. What repository are you running this against?
I'm running it for Eclipse ESCET. See https://gitlab.eclipse.org/eclipse/escet/escet/-/merge_requests/454. Or https://ci.eclipse.org/escet/job/Eclipse%20ESCET%20dependencies%20license%20check/job/develop/.
Have you tried playing with the logging levels?
I executed the Eclipse ESCET build from the branch of the above merge request, with -e -X
, and captured the output, for:
pom.xml
.In both cases, I truncated the resulting command line output after the relevant part of the log has past. I then produced a diff/patch:
with-vs-without-dash-license-plugin.patch
Note how in the 2nd run the Dash license tool output is gone, but so are the many warnings. And the only change is removing the configuration for the Dash license tool plugin from pom.xml
. So, clearly the Dash license tool plugin is somehow causing the extra warnings.
@waynebeaton I still think the Dash license check tool plugin is causing the extra warnings, as per my previous comment.
I've done a little reading and a couple of local experiments and it seems that changing from requiresDependencyResolution
to requiresDependencyCollection
in the Mojo annotation removes the warnings. I want to test it on a few more repositories before I push a change.
Hi @waynebeaton. That sounds promising! Did you have time to do the additional testing yet?
I just pushed the update. It will be included in the next build (sometime later today).
It works. Thanks!
This change broke some builds (see #232). I've reverted the change and am reopening this.
@waynebeaton Will a new solution become available at some point?
When running a Maven/Tycho build with the Dash license tool enabled, I get the following output:
The warning lines all concern projects from the reactor. There are 1481 warning lines in total, which is quite a lot, and pollutes the build log. By disabling the Dash license check tool, the warnings disappear.
I don't know how the Dash license tool works internally, but it seems it should handle projects being built in the reactor in a different way from other dependencies?
I tried to remedy this by adding the following to my
pom.xml
as part of the Dash license check tool configuration:I would have expected these dependencies to be excluded. They are indeed excluded from checking/reporting, but it seems they are still first queried, and only excluded later on?