From 5ecfd39340f70b8c49e53e2c472e92aeafde3992 Mon Sep 17 00:00:00 2001
From: luke <luke@980ebf18-57e1-0310-9a29-db15c13687c0>
Date: Sun, 18 Mar 2007 22:54:41 +0000
Subject: Looks like I already accidentally committed the switch from using
 system() to exec().  I am hoping this will fix the many problems people are
 having with processes hanging around (e.g., #509).  This change just removes
 the attempts at closing TCPServer instances, which should now be fixed from
 using exec instead of system.

git-svn-id: https://reductivelabs.com/svn/puppet/trunk@2297 980ebf18-57e1-0310-9a29-db15c13687c0
---
 lib/puppet/util.rb | 16 ----------------
 1 file changed, 16 deletions(-)

diff --git a/lib/puppet/util.rb b/lib/puppet/util.rb
index fa4d0fae4..8740c7407 100644
--- a/lib/puppet/util.rb
+++ b/lib/puppet/util.rb
@@ -297,21 +297,6 @@ module Util
             if f
                 output = f.read
             else
-                # FIXME There really should be a better way to do this,
-                # but it looks like webrick is already setting close_on_exec,
-                # and setting it myself doesn't seem to do anything.  So,
-                # not the best, but it'll have to do.
-                if Puppet[:listen]
-                    ObjectSpace.each_object do |object|
-                        if object.is_a?(TCPServer) and ! object.closed?
-                            begin
-                                object.close
-                            rescue
-                                # Just ignore these, I guess
-                            end
-                        end
-                    end
-                end
                 begin
                     $stderr.close
                     $stderr = $stdout.dup
@@ -328,7 +313,6 @@ module Util
                     else
                         Kernel.exec(command)
                     end
-                    exit!($?.exitstatus)
                 rescue => detail
                     puts detail.to_s
                     exit!(1)
-- 
cgit