From 2008da00a357accb2783147acd0b6bd67580f261 Mon Sep 17 00:00:00 2001 From: Robert von Burg Date: Wed, 2 Dec 2015 22:03:22 +0100 Subject: [PATCH] [Minor] added checking that working dir is a dir in ProcessHelper --- .../java/ch/eitchnet/utils/helper/ProcessHelper.java | 12 ++++++------ 1 file changed, 6 insertions(+), 6 deletions(-) diff --git a/src/main/java/ch/eitchnet/utils/helper/ProcessHelper.java b/src/main/java/ch/eitchnet/utils/helper/ProcessHelper.java index a4e3f3294..020422deb 100644 --- a/src/main/java/ch/eitchnet/utils/helper/ProcessHelper.java +++ b/src/main/java/ch/eitchnet/utils/helper/ProcessHelper.java @@ -40,8 +40,8 @@ public class ProcessHelper { final int[] returnValue = new int[1]; try (final BufferedReader errorStream = new BufferedReader(new InputStreamReader(process.getErrorStream())); - final BufferedReader inputStream = new BufferedReader(new InputStreamReader( - process.getInputStream()));) { + final BufferedReader inputStream = new BufferedReader( + new InputStreamReader(process.getInputStream()));) { Thread errorIn = new Thread("errorIn") { //$NON-NLS-1$ @Override @@ -79,8 +79,8 @@ public class ProcessHelper { public static ProcessResult runCommand(File workingDirectory, String... commandAndArgs) { - if (!workingDirectory.exists()) { - String msg = "Working directory does not exist at {0}"; //$NON-NLS-1$ + if (!workingDirectory.isDirectory()) { + String msg = "Working directory does not exist or is not a directory at {0}"; //$NON-NLS-1$ msg = MessageFormat.format(msg, workingDirectory.getAbsolutePath()); throw new RuntimeException(msg); } @@ -99,8 +99,8 @@ public class ProcessHelper { int[] returnValue = new int[1]; try (final BufferedReader errorStream = new BufferedReader(new InputStreamReader(process.getErrorStream())); - final BufferedReader inputStream = new BufferedReader(new InputStreamReader( - process.getInputStream()));) { + final BufferedReader inputStream = new BufferedReader( + new InputStreamReader(process.getInputStream()));) { Thread errorIn = new Thread("errorIn") { //$NON-NLS-1$ @Override