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/04 22:57:34 UTC

svn commit: r941040 - /geronimo/sandbox/build-support/libraries/system/1/groovy/gbuild/system/util/BastardChildReaper.groovy

Author: jawarner
Date: Tue May  4 20:57:34 2010
New Revision: 941040

URL: http://svn.apache.org/viewvc?rev=941040&view=rev
Log:
Reverting previous change since it didn't work and added log statement to gain more info

Modified:
    geronimo/sandbox/build-support/libraries/system/1/groovy/gbuild/system/util/BastardChildReaper.groovy

Modified: geronimo/sandbox/build-support/libraries/system/1/groovy/gbuild/system/util/BastardChildReaper.groovy
URL: http://svn.apache.org/viewvc/geronimo/sandbox/build-support/libraries/system/1/groovy/gbuild/system/util/BastardChildReaper.groovy?rev=941040&r1=941039&r2=941040&view=diff
==============================================================================
--- geronimo/sandbox/build-support/libraries/system/1/groovy/gbuild/system/util/BastardChildReaper.groovy (original)
+++ geronimo/sandbox/build-support/libraries/system/1/groovy/gbuild/system/util/BastardChildReaper.groovy Tue May  4 20:57:34 2010
@@ -95,6 +95,8 @@ class BastardChildReaper extends LogSupp
             if (it.PPID == pid) {
                 list << it
                 
+                log.info "Added process to the list of children: $it"
+                
                 // Include children's children if any
                 def l = children(procs, it.PID)
                 if (l) {
@@ -129,7 +131,7 @@ class BastardChildReaper extends LogSupp
     def reap() {
         log.info 'Checking for bastard children...'
         
-        String pid = ShellExecutor.execute('$(sh -c \'echo $PPID\')')
+        String pid = ShellExecutor.execute('echo $PPID')
         
         log.info "Reaping bastard children of $pid"