Merge lp://staging/~clint-fewbar/ubuntu/natty/squid/no-expect-fork into lp://staging/ubuntu/natty/squid
Proposed by
Clint Byrum
| Status: | Merged |
|---|---|
| Merged at revision: | 57 |
| Proposed branch: | lp://staging/~clint-fewbar/ubuntu/natty/squid/no-expect-fork |
| Merge into: | lp://staging/ubuntu/natty/squid |
| Diff against target: |
34 lines (+8/-3) 2 files modified
debian/changelog (+7/-0) debian/squid.upstart (+1/-3) |
| To merge this branch: | bzr merge lp://staging/~clint-fewbar/ubuntu/natty/squid/no-expect-fork |
| Related bugs: |
| Reviewer | Review Type | Date Requested | Status |
|---|---|---|---|
| Ubuntu Sponsors | Pending | ||
|
Review via email:
|
|||
Description of the change
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.
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
