mojohaus / license-maven-plugin

Maven plugin to download and collect license files from project dependencies.
https://www.mojohaus.org/license-maven-plugin/
GNU Lesser General Public License v3.0
106 stars 126 forks source link

MAVEN 3.9.2: Validation failure #572

Open AndreasIgelCC opened 3 months ago

AndreasIgelCC commented 3 months ago

We started to analyse all maven-plugins if they are prepared for maven4. You can gain this with mvn license:aggregate-third-party-report -Dmaven.plugin.validation=verbose and by using a maven-version in 3.9.2 or above. For license-maven-plugin the validation returns the following warnings:

[WARNING] org.codehaus.mojo:license-maven-plugin:2.4.0 [WARNING] Declared at location(s): [WARNING] unknown [WARNING] Mojo EXTERNAL issue(s): [WARNING] * Mojo license:aggregate-third-party-report (org.codehaus.mojo.license.AggregatorThirdPartyReportMojo) [WARNING] - Parameter 'localRepository' uses deprecated parameter expression '${localRepository}': ArtifactRepository type is deprecated and its use in Mojos should be avoided.

AndreasIgelCC commented 3 months ago

Would it be possible, that you @slawekjaranowski have a look on that? Seems like, you do not need the parameter 'localRepository' (see https://github.com/mojohaus/license-maven-plugin/commit/fe03c6da6db4da4cb8e066e21479ca598b0c97d6, but fulltext-search still show it), so it just should be removed on the other places too?

slawekjaranowski commented 3 months ago

It is from reporting / doxia api we need switch to 2.x

https://cwiki.apache.org/confluence/display/MAVEN/Towards+Doxia+2.0.0+Stack#TowardsDoxia2.0.0Stack-Progress

AndreasIgelCC commented 3 months ago

Is that switch planned and do you have time to do that? Or do you need help with it?

slawekjaranowski commented 3 months ago

Doxia 2.x is under preparing https://lists.apache.org/thread/l778r6cwj604hxhjlqs8kt58nngcd602