[Merge] lp:~clint-fewbar/ubuntu/natty/squid/no-expect-fork into lp:ubuntu/squid
Clint Byrum
clint at fewbar.com
Tue Jan 11 21:34:49 UTC 2011
Clint Byrum has proposed merging lp:~clint-fewbar/ubuntu/natty/squid/no-expect-fork into lp:ubuntu/squid.
Requested reviews:
Ubuntu Sponsors Team (ubuntu-sponsors)
Related bugs:
#573853 Cannot control squid "Unknown instance"
https://bugs.launchpad.net/bugs/573853
For more details, see:
https://code.launchpad.net/~clint-fewbar/ubuntu/natty/squid/no-expect-fork/+merge/45916
Fixes the bug by dropping expect fork
This does extend the already existing race between squid's started event and it listening for connections. There don't seem to be any blocking operations before the fork, so this race condition is already as long as its going to be, also I'm not aware of any job files that depend on squid's started event before connecting to it.
--
https://code.launchpad.net/~clint-fewbar/ubuntu/natty/squid/no-expect-fork/+merge/45916
Your team Ubuntu Sponsors Team is requested to review the proposed merge of lp:~clint-fewbar/ubuntu/natty/squid/no-expect-fork into lp:ubuntu/squid.
-------------- next part --------------
A non-text attachment was scrubbed...
Name: review-diff.txt
Type: text/x-diff
Size: 961 bytes
Desc: not available
URL: <https://lists.ubuntu.com/archives/ubuntu-sponsors/attachments/20110111/8754df26/attachment.diff>
More information about the Ubuntu-sponsors
mailing list