From af3dceb2432a670dda4fb6deb3947134e091096c Mon Sep 17 00:00:00 2001
From: Dmitry Trofimov <dmitry.trofimov@jetbrains.com>
Date: Thu, 19 May 2011 15:20:02 +0400
Subject: [PATCH] Correct Jython termination (PY-2591).

---
 .../execution/runners/AbstractConsoleRunnerWithHistory.java   | 2 +-
 .../intellij/execution/configurations/GeneralCommandLine.java | 3 ++-
 .../src/com/intellij/execution/process/OSProcessHandler.java  | 4 ++++
 3 files changed, 7 insertions(+), 2 deletions(-)

diff --git a/platform/lang-impl/src/com/intellij/execution/runners/AbstractConsoleRunnerWithHistory.java b/platform/lang-impl/src/com/intellij/execution/runners/AbstractConsoleRunnerWithHistory.java
index 01033af29063..0492e2ef05e9 100644
--- a/platform/lang-impl/src/com/intellij/execution/runners/AbstractConsoleRunnerWithHistory.java
+++ b/platform/lang-impl/src/com/intellij/execution/runners/AbstractConsoleRunnerWithHistory.java
@@ -102,7 +102,7 @@ public abstract class AbstractConsoleRunnerWithHistory<T extends LanguageConsole
     myConsoleView = createConsoleView();
     myConsoleView.setBorder(new SideBorder(UIUtil.getBorderColor(), SideBorder.LEFT));
 
-      myProcessHandler = createProcessHandler(process, myProvider.getCommandLineString());
+    myProcessHandler = createProcessHandler(process, myProvider.getCommandLineString());
 
     myConsoleExecuteActionHandler = createConsoleExecuteActionHandler();
 
diff --git a/platform/platform-api/src/com/intellij/execution/configurations/GeneralCommandLine.java b/platform/platform-api/src/com/intellij/execution/configurations/GeneralCommandLine.java
index 15638e2ece61..3e7a4e54192e 100644
--- a/platform/platform-api/src/com/intellij/execution/configurations/GeneralCommandLine.java
+++ b/platform/platform-api/src/com/intellij/execution/configurations/GeneralCommandLine.java
@@ -19,6 +19,7 @@ import com.intellij.execution.ExecutionException;
 import com.intellij.execution.process.ProcessNotCreatedException;
 import com.intellij.ide.IdeBundle;
 import com.intellij.openapi.util.SystemInfo;
+import com.intellij.openapi.util.io.FileUtil;
 import com.intellij.openapi.vfs.CharsetToolkit;
 import com.intellij.util.containers.HashMap;
 import org.jetbrains.annotations.NonNls;
@@ -85,7 +86,7 @@ public class GeneralCommandLine {
   }
 
   public String getCommandLineString() {
-    final StringBuffer buffer = new StringBuffer(quoteParameter(myExePath));
+    final StringBuffer buffer = new StringBuffer(quoteParameter(FileUtil.toSystemDependentName(myExePath)));
     appendParams( buffer );
     return buffer.toString();
   }
diff --git a/platform/platform-api/src/com/intellij/execution/process/OSProcessHandler.java b/platform/platform-api/src/com/intellij/execution/process/OSProcessHandler.java
index 6af6faa6fa79..043b6519080a 100644
--- a/platform/platform-api/src/com/intellij/execution/process/OSProcessHandler.java
+++ b/platform/platform-api/src/com/intellij/execution/process/OSProcessHandler.java
@@ -187,6 +187,10 @@ public class OSProcessHandler extends ProcessHandler {
   }
 
   protected void destroyProcessImpl() {
+    closeStreamsAndDestroyProcess();
+  }
+
+  protected void closeStreamsAndDestroyProcess() {
     try {
       closeStreams();
     }