Hi Karl Heinz,

looking at the commit[1] I see that the projectBuildList.contains will prevent the NPE, but it looks weird to me that the projectBuildList does not contain a mavenProject which was marked as finished so can be built. Why is it null? If there's no clear reason yet, there should be at least an else-statement logging this.
Looks to me more like a NPE prevention instead of fixing the root cause.

-1 for me.

thanks,
Robert

[1] https://git1-us-west.apache.org/repos/asf?p=maven.git;a=commitdiff;h=238f789ad07a18c0dff7f884c9a5d3b47258fc47


On Fri, 17 Feb 2017 16:56:23 +0100, Karl Heinz Marbaise <khmarba...@gmx.de> wrote:

Hi,

I have identified a bug[1] in Maven Core based on an issue related to versions-maven-plugin:

I have made already a fix for it and all tests are running[2].

Any objection to merge that issue into master ?

Kind regards
Karl Heinz Marbaise

[1]: https://issues.apache.org/jira/browse/MNG-6170
[2]: https://builds.apache.org/view/M-R/view/Maven/job/maven-3.x-jenkinsfile/job/MNG-6170/


On 08/02/17 21:01, Stephen Connolly wrote:
I think all the important stuff is merged. I'll take a final review through
and then cut alpha-1

We can still add stuff if necessary for an alpha-2 but I'd much prefer to
focus that on shaking out bugs.

Ideally we do at most 3 alpha's before soft-code-freeze and a beta (at
which point it should only be serious bug fixes)

Then we give the beta some soak and cut the release

I toyed with spinning RCs but I'm thinking alpha and beta are better terms

Thoughts?



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

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

Reply via email to