[geode] branch feature/GEODE-5212 updated: Does the restore.bat script really need cmd.exe?

[jensdeppe]

This is an automated email from the ASF dual-hosted git repository.

jensdeppe pushed a commit to branch feature/GEODE-5212
in repository https://gitbox.apache.org/repos/asf/geode.git


The following commit(s) were added to refs/heads/feature/GEODE-5212 by this 
push:
     new b2d94f1  Does the restore.bat script really need cmd.exe?
b2d94f1 is described below

commit b2d94f1df791017e1a5225dd4aa6bca06e648ed9
Author: Jens Deppe <jde...@pivotal.io>
AuthorDate: Thu Jul 26 11:21:05 2018 -0700

    Does the restore.bat script really need cmd.exe?
---
 .../apache/geode/internal/cache/backup/BackupIntegrationTest.java | 8 ++++----
 1 file changed, 4 insertions(+), 4 deletions(-)

diff --git 
a/geode-core/src/integrationTest/java/org/apache/geode/internal/cache/backup/BackupIntegrationTest.java
 
b/geode-core/src/integrationTest/java/org/apache/geode/internal/cache/backup/BackupIntegrationTest.java
index 0a2b6ce..9960421 100644
--- 
a/geode-core/src/integrationTest/java/org/apache/geode/internal/cache/backup/BackupIntegrationTest.java
+++ 
b/geode-core/src/integrationTest/java/org/apache/geode/internal/cache/backup/BackupIntegrationTest.java
@@ -391,10 +391,10 @@ public class BackupIntegrationTest {
     scriptContent.forEach(i -> System.out.println("CONTENT: " + i));
 
     boolean isWindows = script.getName().endsWith("bat");
-    if (isWindows) {
-      command.add("cmd.exe");
-      command.add("/c");
-    }
+    // if (isWindows) {
+    // command.add("cmd.exe");
+    // command.add("/c");
+    // }
 
     command.add(script.getCanonicalPath());
     ProcessBuilder pb = new ProcessBuilder(command);