Merge lp:~broder/ubuntu/lucid/update-inetd/fix-601732 into lp:ubuntu/lucid/update-inetd

[Evan Broder]

[Dave Walker (davewalker)]

Patch looks like it has been cherry picked from debian's 4.37, http://bugs.debian.org/cgi-bin/bugreport.cgi?bug=589487

The patch also seems to be applied in our Maverick package. I have not reproduced the bug on Lucid, but based on it being a known issue; fixed in both Debian and Maverick - I am inclined to approve.

The package does build.

review: Approve

[Dave Walker (davewalker)]

Seems it was already in lucid-proposed... oops.

=== modified file 'DebianNet.pm'
--- DebianNet.pm	2009-09-20 16:51:37 +0000
+++ DebianNet.pm	2010-11-12 16:17:48 +0000
@@ -323,7 +323,13 @@
             &printv("About to $action inetd via invoke-rc.d\n");
             my $service = $1;
             unless ($fake_invocation) {
-                system("invoke-rc.d $service $action >/dev/null");
+		# If we were called by a shell script that also uses
+		# debconf, the pipe to the debconf frontend is fd 3 as
+		# well as fd 1 (stdout).  Ensure that fd 3 is not
+		# inherited by invoke-rc.d and inetd, as that will
+		# cause debconf to hang (bug #589487).  Don't let them
+		# confuse debconf via stdout either.
+                system("invoke-rc.d $service $action >/dev/null 3>&-");
             }
         }
     }

=== modified file 'debian/changelog'
--- debian/changelog	2009-09-27 12:15:15 +0000
+++ debian/changelog	2010-11-12 16:17:48 +0000
@@ -1,3 +1,10 @@
+update-inetd (4.35ubuntu0.1) lucid-proposed; urgency=low
+
+  * Workaround debconf hanging by closing fd 3 in invoke-rc.d invocation
+    (LP: #601732) Patch from Ben Hutchings <[e-mail]>
+
+ -- Evan Broder <[e-mail]>  Fri, 12 Nov 2010 08:13:33 -0800
+
 update-inetd (4.35) unstable; urgency=low
 
   * Add missing dependencies on libfile-temp-perl (which, is provided by the