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

Proposed by Evan Broder
Status: Merged
Merge reported by: Evan Broder
Merged at revision: not available
Proposed branch: lp:~broder/ubuntu/maverick/update-inetd/fix-601732
Merge into: lp:ubuntu/maverick/update-inetd
Diff against target: 33 lines (+14/-1)
2 files modified
DebianNet.pm (+7/-1)
debian/changelog (+7/-0)
To merge this branch: bzr merge lp:~broder/ubuntu/maverick/update-inetd/fix-601732
Reviewer Review Type Date Requested Status
Ubuntu Development Team Pending
Review via email: mp+40738@code.launchpad.net
To post a comment you must log in.

Preview Diff

[H/L] Next/Prev Comment, [J/K] Next/Prev File, [N/P] Next/Prev Hunk
1=== modified file 'DebianNet.pm'
2--- DebianNet.pm 2009-09-20 16:51:37 +0000
3+++ DebianNet.pm 2010-11-12 16:17:16 +0000
4@@ -323,7 +323,13 @@
5 &printv("About to $action inetd via invoke-rc.d\n");
6 my $service = $1;
7 unless ($fake_invocation) {
8- system("invoke-rc.d $service $action >/dev/null");
9+ # If we were called by a shell script that also uses
10+ # debconf, the pipe to the debconf frontend is fd 3 as
11+ # well as fd 1 (stdout). Ensure that fd 3 is not
12+ # inherited by invoke-rc.d and inetd, as that will
13+ # cause debconf to hang (bug #589487). Don't let them
14+ # confuse debconf via stdout either.
15+ system("invoke-rc.d $service $action >/dev/null 3>&-");
16 }
17 }
18 }
19
20=== modified file 'debian/changelog'
21--- debian/changelog 2010-02-13 18:50:20 +0000
22+++ debian/changelog 2010-11-12 16:17:16 +0000
23@@ -1,3 +1,10 @@
24+update-inetd (4.36ubuntu0.1) maverick-proposed; urgency=low
25+
26+ * Workaround debconf hanging by closing fd 3 in invoke-rc.d invocation
27+ (LP: #601732) Patch from Ben Hutchings <ben@decadent.org.uk>
28+
29+ -- Evan Broder <evan@ebroder.net> Fri, 12 Nov 2010 08:07:22 -0800
30+
31 update-inetd (4.36) unstable; urgency=low
32
33 * Remove "there are plans to support xinetd" from long description (see

Subscribers

People subscribed via source and target branches

to all changes: