Testing that 'bzr serve' actually installs SIGHUP and responds to it showed some problems.

Namely, using 'register(id(self), self.func', doesn't work correctly.
Because 'self.func' creates a temporary object BoundMethod, which ends up
only referenced by the WeakValueDictionary.
Instead, we have to assign the attribute to a local variable, and then
register that variable.

We still haven't solved the 'wait for clients' issue yet.