skin: fixed launch ecp (removed processbuild redirection)
authorJinhyung Choi <jinhyung2.choi@samsung.com>
Wed, 2 Jul 2014 10:03:35 +0000 (19:03 +0900)
committerJinhyung Choi <jinhyung2.choi@samsung.com>
Wed, 2 Jul 2014 10:03:35 +0000 (19:03 +0900)
processbuilder redirection is not supported by java 1.6

Change-Id: Ibf5e88844f13dca62f21dfe4be1b00f171567822
Signed-off-by: Jinhyung Choi <jinhyung2.choi@samsung.com>
tizen/src/skin/client/src/org/tizen/emulator/skin/EmulatorSkin.java

index b2ec2a5448ee8eaa5b2da982df20b3bdaad41f6a..752b75ead0e5d835e911fa8e04e294eecd34c21e 100755 (executable)
 
 package org.tizen.emulator.skin;
 
+import java.io.BufferedReader;
 import java.io.File;
 import java.io.IOException;
+import java.io.InputStream;
+import java.io.InputStreamReader;
 import java.util.ArrayList;
 import java.util.Iterator;
 import java.util.LinkedList;
@@ -104,6 +107,7 @@ import org.tizen.emulator.skin.log.SkinLogger;
 import org.tizen.emulator.skin.menu.KeyWindowKeeper;
 import org.tizen.emulator.skin.menu.PopupMenu;
 import org.tizen.emulator.skin.screenshot.ScreenShotDialog;
+import org.tizen.emulator.skin.util.IOUtil;
 import org.tizen.emulator.skin.util.SkinUtil;
 import org.tizen.emulator.skin.util.SwtUtil;
 
@@ -1842,44 +1846,38 @@ public class EmulatorSkin {
                                        try {
                                                SkinUtil.openMessage(shell, null,
                                                                "Control Panel file does not exist in the following path.\n"
-                                                               + ecpFile.getCanonicalPath(),
+                                                                               + ecpFile.getCanonicalPath(),
                                                                SWT.ICON_ERROR, config);
                                        } catch (IOException ee) {
                                                logger.log(Level.SEVERE, ee.getMessage(), ee);
                                        }
 
                                        return;
-                }
+                               }
 
-                String emulName = SkinUtil.getVmName(config);
-                int portSdb = config.getArgInt(ArgsConstants.VM_BASE_PORT);
+                               String emulName = SkinUtil.getVmName(config);
+                               int portSdb = config.getArgInt(ArgsConstants.VM_BASE_PORT);
 
                                ProcessBuilder procEcp = new ProcessBuilder();
                                procEcp.redirectErrorStream(true);
-                               File nullFile = null;
                                // FIXME: appropriate running binary setting is necessary.
                                if (SwtUtil.isWindowsPlatform()) {
                                        procEcp.command("java.exe", "-jar", ecpPath, "vmname="
                                                        + emulName, "base.port=" + portSdb);
-                                       nullFile = new File("NUL:");
                                } else if (SwtUtil.isMacPlatform()) {
                                        procEcp.command("java", "-jar", "-XstartOnFirstThread",
                                                        ecpPath, "vmname=" + emulName, "base.port="
                                                                        + portSdb);
-                                       nullFile = new File("/dev/null");
                                } else { /* Linux */
                                        procEcp.command("java", "-jar", ecpPath, "vmname="
                                                        + emulName, "base.port=" + portSdb);
-                                       nullFile = new File("/dev/null");
-                               }
-                               if (nullFile != null) {
-                                       procEcp.redirectOutput(ProcessBuilder.Redirect.to(nullFile));
                                }
 
                                logger.info(procEcp.command().toString());
 
+                               Process proc = null;
                                try {
-                                       procEcp.start(); /* open ECP */
+                                       proc = procEcp.start(); /* open ECP */
                                } catch (Exception ee) {
                                        logger.log(Level.SEVERE, ee.getMessage(), ee);
                                        SkinUtil.openMessage(
@@ -1888,6 +1886,26 @@ public class EmulatorSkin {
                                                        "Fail to open control panel : \n" + ee.getMessage(),
                                                        SWT.ICON_ERROR, config);
                                }
+
+                               if (proc != null) {
+                                       final InputStream in = proc.getInputStream();
+                                       final BufferedReader br = new BufferedReader(
+                                                       new InputStreamReader(in));
+                                       new Thread(new Runnable() {
+                                               @Override
+                                               public void run() {
+                                                       try {
+                                                               while (br.readLine() != null) {
+                                                                       // do nothing
+                                                               }
+                                                       } catch (IOException e) {
+                                                               logger.log(Level.SEVERE, e.getMessage(), e);
+                                                       } finally {
+                                                               IOUtil.close(in);
+                                                       }
+                                               }
+                                       }).start();
+                               }
                        }
                };