Skip to content

Commit 3b3aeb5

Browse files
committed
revert
1 parent a413823 commit 3b3aeb5

File tree

1 file changed

+6
-3
lines changed

1 file changed

+6
-3
lines changed

src/main/java/org/apache/maven/plugins/jlink/AbstractJLinkToolchainExecutor.java

Lines changed: 6 additions & 3 deletions
Original file line numberDiff line numberDiff line change
@@ -114,7 +114,7 @@ private String getJLinkExecutable() {
114114
getLog().error("Either JDK9+ or a toolchain "
115115
+ "pointing to a JDK9+ containing a jlink binary is required.");
116116
getLog().info("See https://maven.apache.org/guides/mini/guide-using-toolchains.html "
117-
+ "for more information.");
117+
+ "for mor information.");
118118
throw new IllegalStateException("Running on JDK8 and no toolchain found.");
119119
}
120120

@@ -162,7 +162,9 @@ private int executeCommand(Commandline cmd) throws MojoExecutionException {
162162

163163
if (exitCode != 0) {
164164

165-
if (output != null) {
165+
if (output != null && !output.isEmpty()) {
166+
// Reconsider to use WARN / ERROR ?
167+
// getLog().error( output );
166168
for (String outputLine : output.split("\n")) {
167169
getLog().error(outputLine);
168170
}
@@ -179,7 +181,8 @@ private int executeCommand(Commandline cmd) throws MojoExecutionException {
179181
throw new MojoExecutionException(msg.toString());
180182
}
181183

182-
if (output != null) {
184+
if (output != null && !output.isEmpty()) {
185+
// getLog().info( output );
183186
for (String outputLine : output.split("\n")) {
184187
getLog().info(outputLine);
185188
}

0 commit comments

Comments
 (0)