manager: rich logging
authorSeokYeon Hwang <syeon.hwang@samsung.com>
Fri, 12 Jun 2015 09:05:10 +0000 (18:05 +0900)
committerSeokYeon Hwang <syeon.hwang@samsung.com>
Fri, 12 Jun 2015 09:05:10 +0000 (18:05 +0900)
Change-Id: Iab8c9d61964af70ab7d86c5bd89ed707a0da49eb
Signed-off-by: SeokYeon Hwang <syeon.hwang@samsung.com>
src/org/tizen/emulator/manager/job/CheckSDCard.java
src/org/tizen/emulator/manager/job/Overseer.java
src/org/tizen/emulator/manager/vms/helper/HelperClass.java

index e70eb17..c3a4263 100644 (file)
@@ -67,10 +67,12 @@ public class CheckSDCard implements Job {
 
                                        // copy sdcard images if only new sdcard images are provided.
                                        if (sourceAttr.lastModifiedTime().compareTo(targetAttr.lastModifiedTime()) <= 0) {
-                                               // FIXME: if newer images are exist,
-                                               // we should warn to user before replace legacy images.
                                                continue;
                                        }
+
+                                       // FIXME: if newer images are exist,
+                                       // we should warn to user before replace legacy images.
+                                       EMLogger.getLogger().log(Level.INFO, "Newer SDCARD image is provided: " + source.getFileName());
                                }
 
                                Files.copy(source, target, StandardCopyOption.COPY_ATTRIBUTES, StandardCopyOption.REPLACE_EXISTING);
index 3134c86..147d1ea 100644 (file)
@@ -71,7 +71,6 @@ public class Overseer {
        }
 
        public void startWork() throws InterruptedException {
-               EMLogger.getLogger().log(Level.INFO, "Overseer [" + name + "] is starting...");
                if (overseerThread != null) {
                        // already started
                        assert false : "Overseer [" + name + "] already started !!!"; // for debugging...
@@ -83,7 +82,9 @@ public class Overseer {
                        @Override
                        public void run() {
                                try {
+                                       EMLogger.getLogger().log(Level.INFO, "Overseer [" + name + "] is starting...");
                                        Overseer.this.startWorkInternal();
+                                       EMLogger.getLogger().log(Level.INFO, "Overseer [" + name + "] is done...");
                                } catch (InterruptedException e) {
                                        e.printStackTrace();
                                }
@@ -92,7 +93,6 @@ public class Overseer {
 
                overseerThread.start();
 
-               EMLogger.getLogger().log(Level.INFO, "Overseer [" + name + "] is done...");
        }
 
        public void join() throws InterruptedException {
index 6345493..27e2953 100644 (file)
@@ -43,6 +43,7 @@ import java.net.SocketException;
 import java.util.Arrays;
 import java.util.Enumeration;
 import java.util.List;
+import java.util.logging.Level;
 
 import org.tizen.emulator.manager.EmulatorManager;
 import org.tizen.emulator.manager.logging.EMLogger;
@@ -245,6 +246,8 @@ public class HelperClass {
                        pb.directory(directory);
                }
 
+               EMLogger.getLogger().log(Level.INFO, "Try to run external process: " + cmd.get(0));
+
                try {
                        process = pb.start();
                        List<String> stdOut = ProcessOutputReader.readStdOut(process, cmd.toString());