commit eeba5ea49b89dfc6204876fd85247348fc0f9e27
author Carbo Kuo <carbokuo@fb.com> Fri Sep 20 02:32:18 2013 -0700
committer Michael Bolin <mbolin@fb.com> Wed Sep 25 11:10:07 2013 -0700
tree 6a023d4567cb42882ab3fba226a92eced6b6f0e0
parent 50572babef12103de8d76969af277b2a2db3a951

Fix BuckRunner of IntelliJ plugin.

Summary: Should remove `process.destroy()` instead of `process.waitFor()`.

Test Plan: Build plugin.

plugin/src/com/facebook/buck/plugin/intellij/commands/BuckRunner.java

diff --git a/plugin/src/com/facebook/buck/plugin/intellij/commands/BuckRunner.java b/plugin/src/com/facebook/buck/plugin/intellij/commands/BuckRunner.java
index 251e41a..616b684 100644
--- a/plugin/src/com/facebook/buck/plugin/intellij/commands/BuckRunner.java
+++ b/plugin/src/com/facebook/buck/plugin/intellij/commands/BuckRunner.java
@@ -109,7 +109,7 @@
       int exitCode;
       Future<String> stdoutFuture = readStream(process.getInputStream());
       Future<String> stderrFuture = readStream(process.getErrorStream());
-      process.destroy();
+      exitCode = process.waitFor();
       stdout = stdoutFuture.get();
       stderr = stderrFuture.get();
       return exitCode;