Skip to content
This repository has been archived by the owner on Dec 18, 2021. It is now read-only.

Fix NPE due to missing MVNW_VERBOSE #65

Merged
merged 1 commit into from
Sep 12, 2017
Merged

Conversation

ttddyy
Copy link

@ttddyy ttddyy commented Sep 12, 2017

With the latest, 0.2.2

./mvnw --version
Exception in thread "main" java.lang.NullPointerException
	at org.apache.maven.wrapper.MavenWrapperMain.main(MavenWrapperMain.java:42)

This was due to System.getenv(MVNW_VERBOSE).equalsIgnoreCase("true").
Updated to handle when env value is null.

@mosabua
Copy link
Member

mosabua commented Sep 12, 2017

Dang .. I only tested for empty but not null. Thanks! I will cut another release shortly.

@mosabua mosabua merged commit 02a4464 into takari:master Sep 12, 2017
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants