You are viewing a plain text version of this content. The canonical link for it is here.
Posted to scm@geronimo.apache.org by jd...@apache.org on 2006/09/03 02:17:31 UTC

svn commit: r439693 - /geronimo/server/trunk/maven-plugins/selenium-maven-plugin/src/main/java/org/apache/geronimo/mavenplugins/selenium/StartServerMojo.java

Author: jdillon
Date: Sat Sep  2 17:17:31 2006
New Revision: 439693

URL: http://svn.apache.org/viewvc?rev=439693&view=rev
Log:
Added some info logging
Using SimpleLog instead of JDK 1.4 logging for now

Modified:
    geronimo/server/trunk/maven-plugins/selenium-maven-plugin/src/main/java/org/apache/geronimo/mavenplugins/selenium/StartServerMojo.java

Modified: geronimo/server/trunk/maven-plugins/selenium-maven-plugin/src/main/java/org/apache/geronimo/mavenplugins/selenium/StartServerMojo.java
URL: http://svn.apache.org/viewvc/geronimo/server/trunk/maven-plugins/selenium-maven-plugin/src/main/java/org/apache/geronimo/mavenplugins/selenium/StartServerMojo.java?rev=439693&r1=439692&r2=439693&view=diff
==============================================================================
--- geronimo/server/trunk/maven-plugins/selenium-maven-plugin/src/main/java/org/apache/geronimo/mavenplugins/selenium/StartServerMojo.java (original)
+++ geronimo/server/trunk/maven-plugins/selenium-maven-plugin/src/main/java/org/apache/geronimo/mavenplugins/selenium/StartServerMojo.java Sat Sep  2 17:17:31 2006
@@ -152,8 +152,16 @@
                 try {
                     ExecTask exec = (ExecTask)createTask("exec");
                     exec.setExecutable(executable);
+                    
+                    //
+                    // HACK: Use Simple log instead of evil JDK 1.4 logging
+                    //       Should change to Java task and setup log4j.properties in the cp
+                    //
+                    exec.createArg().setValue("-Dorg.apache.commons.logging.Log=org.apache.commons.logging.impl.SimpleLog");
+                    
                     exec.createArg().setValue("-jar");
                     exec.createArg().setFile(artifact.getFile());
+                    
                     exec.createArg().setValue("-port");
                     exec.createArg().setValue(String.valueOf(port));
                     
@@ -162,18 +170,27 @@
                     }
                     
                     if (timeout > 0) {
+                        log.info("Timeout after: " + timeout + " seconds");
+                        
                         exec.createArg().setValue("-timeout");
                         exec.createArg().setValue(String.valueOf(timeout));
                     }
                     
                     if (userExtensions != null) {
+                        log.info("Using user extensions: " + userExtensions);
+                        
                         exec.createArg().setValue("-userExtensions");
                         exec.createArg().setFile(userExtensions);
                     }
                     
                     exec.setLogError(true);
                     exec.execute();
-
+                }
+                catch (Exception e) {
+                    log.error("Failed to start server", e);
+                }
+                
+                try {
                     synchronized(this) {
                         wait();
                     }