maven-dev mailing list archives

Site index · List index
Message view « Date » · « Thread »
Top « Date » · « Thread »
From Maxim Solodovnik <solomax...@gmail.com>
Subject Re: Maven Project Info Reports Plugin 3.0.1 release
Date Thu, 11 Jul 2019 05:15:26 GMT
Thanks a lot Robert

Finally I was able to get back to this task :)

The test was created and fails successfully :) [1]
Then I tried to replace ProjectBuilder with ModelBuilder
Unfortunately it seems impossible :(

ProjectBuilder has method build(artifact)
ModelBuilder doesn't have it :(

Maybe it is possible to pass bundle plugin to projectBuilder somehow?

[1] https://github.com/solomax/maven-project-info-reports-plugin/tree/master/src/it/MPIR-374-dependencymanagement-bundle

On Wed, 19 Jun 2019 at 03:34, Robert Scholte <rfscholte@apache.org> wrote:
>
> [ERROR] Unknown packaging: bundle @ line 6, column 16 at org.apache.maven.project.DefaultProjectBuilder.build
(DefaultProjectBuilder.java:205) at org.apache.maven.project.DefaultProjectBuilder.build (DefaultProjectBuilder.java:340)
at org.apache.maven.report.projectinfo.dependencies.RepositoryUtils.getMavenProjectFromRepository
(RepositoryUtils.java:125) at org.apache.maven.report.projectinfo.dependencies.renderer.DependencyManagementRenderer.getDependencyRow
(DependencyManagementRenderer.java:253)
>
> Based on this stacktrace the root cause is in Maven.
>
> So a little bit more details: the code tries to gather all licenses, see the code of
the getDependencyRow method. Because of the inheritence it can't simply read the pom, it must
respect all its parents too.
> My first impression is that using a ProjectBuilder comes with too much overhead. Instead
I would try to use a ModelBuilder instead. It's result will be an effective model containing
all licenses.
>
> To guide you in the right direction:
> I'd suggest to make an integrationtest under src/it/MPIR-374-dependencymanagement-bundle
with only a pom, containing only dependencyManagement for com.fasterxml.jackson.module:jackson-module-scala_2.10:jar:2.9.5
> Also add the following verify.groovy script:
>   def buildLog = new File( basedir, 'build.log' )
>   assert !( buildLog.text =~ /\[ERROR\] Unknown packaging: bundle/ )
>
> Now run it with mvn verify -Prun-its -Dinvoker.test=MPIR-374*
> It should fail because the build.log contains the error message.
>
> I hope this is enough information to continue and hopefully fix this issue.
>
> thanks
> Robert
>
>
> On 18-6-2019 04:29:06, Maxim Solodovnik <solomax666@gmail.com> wrote:
> +1 :)
> Will be happy to help with this release :)
>
> BTW can someone take a look at this
> https://issues.apache.org/jira/browse/MPIR-374 issue?
> Or maybe any pointer on where to look? I spent some time trying to fix it,
> but no luck :(
>
> On Tue, 18 Jun 2019 at 00:31, Peter Somogyi wrote:
>
> > Hi,
> >
> > I'd like to ask if there are any plans to release version 3.0.1 from Maven
> > Project Info Reports Plugin in the near future.
> >
> > There is a bug in 3.0.0 which generates the email links incorrectly. As a
> > result, the mailing list subscribe URLs give 404 error message on HBase's
> > Mailing Lists page. This bug was fixed in MPIR-380
> > https://issues.apache.org/jira/browse/MPIR-380> back in February but there
> > was no release since then.
> >
> > HBase is using this version via the latest ASF parent pom
> >
> > https://github.com/apache/maven-apache-parent/blob/apache-21/pom.xml#L210-L214
> > >
> >
> > Thanks,
> > Peter
> >
>
>
> --
> WBR
> Maxim aka solomax



-- 
WBR
Maxim aka solomax

---------------------------------------------------------------------
To unsubscribe, e-mail: dev-unsubscribe@maven.apache.org
For additional commands, e-mail: dev-help@maven.apache.org


Mime
View raw message