svn commit: r355037 - /geronimo/gbuild/trunk/gbuild-agent/src/main/java/org/apache/geronimo/gbuild/agent/LogFailedBuildsExtension.java

[db...@apache.org]

Author: dblevins
Date: Thu Dec  8 00:40:14 2005
New Revision: 355037

URL: http://svn.apache.org/viewcvs?rev=355037&view=rev
Log:
Added better logging

Modified:
    geronimo/gbuild/trunk/gbuild-agent/src/main/java/org/apache/geronimo/gbuild/agent/LogFailedBuildsExtension.java

Modified: geronimo/gbuild/trunk/gbuild-agent/src/main/java/org/apache/geronimo/gbuild/agent/LogFailedBuildsExtension.java
URL: http://svn.apache.org/viewcvs/geronimo/gbuild/trunk/gbuild-agent/src/main/java/org/apache/geronimo/gbuild/agent/LogFailedBuildsExtension.java?rev=355037&r1=355036&r2=355037&view=diff
==============================================================================
--- geronimo/gbuild/trunk/gbuild-agent/src/main/java/org/apache/geronimo/gbuild/agent/LogFailedBuildsExtension.java (original)
+++ geronimo/gbuild/trunk/gbuild-agent/src/main/java/org/apache/geronimo/gbuild/agent/LogFailedBuildsExtension.java Thu Dec  8 00:40:14 2005
@@ -23,6 +23,7 @@
 import org.codehaus.plexus.personality.plexus.lifecycle.phase.Startable;
 import org.codehaus.plexus.personality.plexus.lifecycle.phase.StartingException;
 import org.codehaus.plexus.personality.plexus.lifecycle.phase.StoppingException;
+import org.codehaus.plexus.logging.AbstractLogEnabled;
 
 import java.io.File;
 import java.io.IOException;
@@ -34,7 +35,7 @@
 /**
  * @version $Rev$ $Date$
  */
-public class LogFailedBuildsExtension extends AbstractContinuumAgentAction implements BuildResultsExtension, Startable {
+public class LogFailedBuildsExtension extends AbstractLogEnabled implements BuildResultsExtension, Startable {
 
     /**
      * @plexus.configuration
@@ -80,6 +81,7 @@
     }
 
     public void execute(Map context) throws Exception {
+
         ContinuumStore store = AbstractContinuumAgentAction.getContinuumStore(context);
 
         int projectId = AbstractContinuumAction.getProjectId(context);
@@ -90,6 +92,8 @@
 
         int exitCode = buildResult.getExitCode();
 
+        getLogger().debug(context.get("build.name") + " " + context.get("build.id") + " - exit code (" + exitCode + ")");
+
         if (exitCode == 0) {
             return;
         }
@@ -118,10 +122,13 @@
 
         parent.mkdirs();
 
+
         try {
             GZipUtils.fileWrite(file, bytes);
         } catch (IOException e) {
             getLogger().error("Could not write to file " + file.getAbsolutePath(), e);
         }
+
+        getLogger().info("Wrote build ouput (" + file.length() + " bytes) to " + file.getAbsolutePath());
     }
 }