Re: Seconder for MNG-6819 NPE interpolation

classic Classic list List threaded Threaded
1 message Options
Reply | Threaded
Open this post in threaded view
|

Re: Seconder for MNG-6819 NPE interpolation

olamy
Just do it!
Not sure we need this for a simple null check
But yes +1 (for the administrative paperwork record)

On Wed, 20 May 2020 at 19:45, Sylwester Lachiewicz <[hidden email]>
wrote:

> Hi,
> I would like to merge MNG-6819 to fix nasty NullPointerException instead of
> correct information about cyclic dependency on property interpolation.
>
> 1. Anyone, to approve?
> 2. This is a regression for Maven 3.6.3 - do we want to backport this fix
> and release 3.6.4 ?
>
> BR
> Sylwester
>
> [1]  MNG-6819 -  https://jira.apache.org/jira/browse/MNG-6819
> [2] PR + UT: https://github.com/apache/maven/pull/301
> [3] branch with code: https://github.com/apache/maven/tree/MNG-6819
> [4]
>
> https://builds.apache.org/view/M-R/view/Maven/job/maven-box/job/maven/job/MNG-6819/
>


--
Olivier Lamy
http://twitter.com/olamy | http://linkedin.com/in/olamy