Merge pull request #16 from alexkrauss/proper-shell

Invoke Tslint without new shell, which used to fail on Unix.
Pablissimo · Sep 30, 2015 · 44480a2 · 44480a2
2 parents a29b34f + 3449cc4
commit 44480a2
Showing 1 changed file with 7 additions and 5 deletions.
diff --git a/src/main/java/com/pablissimo/sonar/TsLintExecutorImpl.java b/src/main/java/com/pablissimo/sonar/TsLintExecutorImpl.java
@@ -15,11 +15,13 @@ public class TsLintExecutorImpl implements TsLintExecutor {
     public String execute(String pathToTsLint, String configFile, String file) {
         LOG.info("TsLint executing for " + file);
         Command command = Command.create("node");
-
-        command
-            .addArgument("\"" + pathToTsLint + "\" --config \"" + configFile + "\" --format json \"" + file.trim() + "\"");
-
-        command.setNewShell(true);
+        command.addArgument(pathToTsLint);
+        command.addArgument("--format");
+        command.addArgument("json");
+        command.addArgument("--config");
+        command.addArgument(configFile);
+        command.addArgument(file.trim());
+        command.setNewShell(false);
 
         this.stdOut = new StringBuilder();
         this.stdErr = new StringBuilder();