You are viewing a plain text version of this content. The canonical link for it is here.
Posted to commits@maven.apache.org by sl...@apache.org on 2020/12/30 00:47:09 UTC
[maven-enforcer] 03/04: [MENFORCER-351] Added check to avoid NPE
with old Eclipse versions
This is an automated email from the ASF dual-hosted git repository.
slachiewicz pushed a commit to branch master
in repository https://gitbox.apache.org/repos/asf/maven-enforcer.git
commit 086204bc20e44989928b5106140e7247775dddb3
Author: Sylwester Lachiewicz <sl...@apache.org>
AuthorDate: Wed Dec 30 00:54:52 2020 +0100
[MENFORCER-351] Added check to avoid NPE with old Eclipse versions
---
.../java/org/apache/maven/plugins/enforcer/RequireMavenVersion.java | 5 +++++
1 file changed, 5 insertions(+)
diff --git a/enforcer-rules/src/main/java/org/apache/maven/plugins/enforcer/RequireMavenVersion.java b/enforcer-rules/src/main/java/org/apache/maven/plugins/enforcer/RequireMavenVersion.java
index 50ba869..733f0cd 100644
--- a/enforcer-rules/src/main/java/org/apache/maven/plugins/enforcer/RequireMavenVersion.java
+++ b/enforcer-rules/src/main/java/org/apache/maven/plugins/enforcer/RequireMavenVersion.java
@@ -42,6 +42,11 @@ public class RequireMavenVersion
MavenSession mavenSession = (MavenSession) helper.evaluate( "${session}" );
String mavenVersion = mavenSession.getSystemProperties().getProperty( "maven.version" );
helper.getLog().debug( "Detected Maven Version: " + mavenVersion );
+ if ( mavenVersion == null )
+ {
+ throw new EnforcerRuleException(
+ "Unable to detect Maven Version - missing system property - maven.version" );
+ }
DefaultArtifactVersion detectedVersion = new DefaultArtifactVersion( mavenVersion );
enforceVersion( helper.getLog(), "Maven", getVersion(), detectedVersion );
}