Re: NPE in AbstractMavenReport.execute, because 'this.siteTool' is null

2023-10-11 Thread Herve Boutemy
ok, Draft PR created https://github.com/dev-aspectj/aspectj-maven-plugin/pull/137 to easily reproduce and describe the issue then I analyzed more in depth, as commented in the PR: it seems you found a bug in Plugin Tools on inherited @Component injection we'll need to open a MPLUGIN issue and

Re: NPE in AbstractMavenReport.execute, because 'this.siteTool' is null

2023-10-10 Thread Alexander Kriegisch
Hello Hervé. I am sorry, if my explanations were too complicated for you. I just repeated everything from scratch for you, directly from my Git Bash console: $ git clone

Re: NPE in AbstractMavenReport.execute, because 'this.siteTool' is null

2023-10-10 Thread Herve Boutemy
I tried to build the project and see the failure, but could not and reading explanations did not help me what we need is a basic commit that works, followed by a PR upgrading Doxia and that does not work with the NPE that you are saying happens Regards, Hervé On 2023/10/10 02:44:08 Alexander

Re: NPE in AbstractMavenReport.execute, because 'this.siteTool' is null

2023-10-09 Thread Alexander Kriegisch
Reverting back to my original question, also summarised in the subject of this thread: Why is 'this.siteTool' null and not injected properly? If I knew how to fix that configuratively or via coding, I would no longer need to override 'exceute()', which relies on that field being set, as you can

Re: NPE in AbstractMavenReport.execute, because 'this.siteTool' is null

2023-10-09 Thread Alexander Kriegisch
Actually, my initial message already explained the situation: 1. With 1.x, I have no issues. That was my starting point. 2. With 2.0, I had no desire to verride 'execute()'. It was simply the only thing that helped my project to function properly again. I would be glad to do

Re: NPE in AbstractMavenReport.execute, because 'this.siteTool' is null

2023-10-09 Thread Michael Osipov
I had these NPEs when I started to migrate reporting plugins and both the plugin and the super class contained the same fields which created conflicts. After I have removed them from the plugin and used the super ones all went fine. In your case I would really do what Hervé proposes: * remove

Re: NPE in AbstractMavenReport.execute, because 'this.siteTool' is null

2023-10-05 Thread Michael Osipov
On 2023/10/05 06:46:02 Hervé Boutemy wrote: > improvement proposed: review appreciated > https://github.com/apache/maven-reporting-impl/pull/24 Will try to look at the issue here and the proposed PR in the next couple of days...

Re: NPE in AbstractMavenReport.execute, because 'this.siteTool' is null

2023-10-05 Thread Hervé Boutemy
improvement proposed: review appreciated https://github.com/apache/maven-reporting-impl/pull/24 Le mercredi 4 octobre 2023, 08:30:04 CEST Hervé Boutemy a écrit : > Hi Xander, > > Sorry, I lost track, having a look now. > > When I see that you are implementing Mojo's "execute()" [1] while using

Re: NPE in AbstractMavenReport.execute, because 'this.siteTool' is null

2023-10-04 Thread Hervé Boutemy
Hi Xander, Sorry, I lost track, having a look now. When I see that you are implementing Mojo's "execute()" [1] while using maven-reporting-impl, you are in fact defeating the objective of maven-reporting-impl: do that for you. maven-reporting-impl would probably need better documentation, I

Re: NPE in AbstractMavenReport.execute, because 'this.siteTool' is null

2023-10-03 Thread Alexander Kriegisch
Hello Hervé. Did the reproducer help you in any way? Regards -- Alexander Kriegisch https://scrum-master.de Alexander Kriegisch schrieb am 23.09.2023 10:00 (GMT +07:00): > Hello Hervé. > >>> I tried to upgrade those >>> dependencies to the most recent Doxia and Sitetools versions. > >> by

Re: NPE in AbstractMavenReport.execute, because 'this.siteTool' is null

2023-09-22 Thread Alexander Kriegisch
Hello Hervé. >> I tried to upgrade those >> dependencies to the most recent Doxia and Sitetools versions. > by "most recent", do you mean most recent from 1.x or 2.0.0-M*? I mean 2.0.0-M*. Actually, the project works nicely and I would have ignored the Dependabot suggestions, but all those

Re: NPE in AbstractMavenReport.execute, because 'this.siteTool' is null

2023-09-22 Thread Hervé Boutemy
> I tried to upgrade those > dependencies to the most recent Doxia and Sitetools versions. by "most recent", do you mean most recent from 1.x or 2.0.0-M*? 1.x should not cause issues 2.0.0-M*, as expected from the version number, is more risky and not yet fully stable Such reporting plugin

Re: NPE in AbstractMavenReport.execute, because 'this.siteTool' is null

2023-09-21 Thread Alexander Kriegisch
Hello. I was asking this question two weeks ago. Can somebody tell me if maybe I asked on the wrong list or in the wrong way and possible direct me elsewhere? Thank you all. Best regards -- Alexander Kriegisch https://scrum-master.de Alexander Kriegisch schrieb am 07.09.2023 09:35 (GMT