You are viewing a plain text version of this content. The canonical link for it is here.
Posted to scm@geronimo.apache.org by ja...@apache.org on 2010/05/05 16:28:50 UTC

svn commit: r941311 - /geronimo/sandbox/build-support/libraries/system/1/groovy/gbuild/config/projects/Geronimo_CTS/Controller.groovy

Author: jawarner
Date: Wed May  5 14:28:50 2010
New Revision: 941311

URL: http://svn.apache.org/viewvc?rev=941311&view=rev
Log:
Add in a pkill call to kill any servers left lingering after testing.

Modified:
    geronimo/sandbox/build-support/libraries/system/1/groovy/gbuild/config/projects/Geronimo_CTS/Controller.groovy

Modified: geronimo/sandbox/build-support/libraries/system/1/groovy/gbuild/config/projects/Geronimo_CTS/Controller.groovy
URL: http://svn.apache.org/viewvc/geronimo/sandbox/build-support/libraries/system/1/groovy/gbuild/config/projects/Geronimo_CTS/Controller.groovy?rev=941311&r1=941310&r2=941311&view=diff
==============================================================================
--- geronimo/sandbox/build-support/libraries/system/1/groovy/gbuild/config/projects/Geronimo_CTS/Controller.groovy (original)
+++ geronimo/sandbox/build-support/libraries/system/1/groovy/gbuild/config/projects/Geronimo_CTS/Controller.groovy Wed May  5 14:28:50 2010
@@ -247,8 +247,12 @@ class Controller extends gbuild.system.P
             //
             showProcessesAndTemp()
             
+            // This isn't working properly.  Comment it out for now.
             // Nuke any bastard children
-            new BastardChildReaper().reap()
+            //new BastardChildReaper().reap()
+            // Still need to make sure that the server isn't left around after the test finishes
+            ShellExecutor.execute('pkill -9 -f server.jar')
+            
         }
     }