"Address already in use " crashes smart server

[RLVJ]

To reproduce:
bzr serve&
bzr serve

Crashes Bazaar 1.2.0, which issues a request that a bug be filed.

[Dan Watkins (oddbloke)]

I can reproduce here, and it seems like we should be giving a more friendly error.

[John A Meinel (jameinel)]

Yeah, it just needs a clearer error than a traceback.

[Andrea Corbellini (andrea.corbellini)]

Here's a patch to fix this bug.

[Alexander Belchenko (bialix)]

Please, don't touch in the patch the lines you don't intend to change. I suppose your editor is set to delete all trailing whitespaces in lines when you save file, but it's really bad for patches for open-source projects, because this produces many unneeded changes unrelated to your actual bugfix. So, please rework your patch.

And after you'll have clean patch, please send it to bazaar mailing list with [MERGE] prefix in the subject line.

Thanks.

[Alexander Belchenko (bialix)]

If you are not subscribed to bzr ML I can forward your patch there.