Rev 5931: Tsk, tsk, that's not the issue at all, the race is in the test framework. in file:///home/vila/src/bzr/bugs/789167-test-server/

[Vincent Ladeuil]

At file:///home/vila/src/bzr/bugs/789167-test-server/

------------------------------------------------------------
revno: 5931
revision-id: v.ladeuil+lp at free.fr-20110528091036-c82o6sozffi0q6ey
parent: v.ladeuil+lp at free.fr-20110528084017-1t8dsk8r10x5qu0u
committer: Vincent Ladeuil <v.ladeuil+lp at free.fr>
branch nick: 789167-test-server
timestamp: Sat 2011-05-28 11:10:36 +0200
message:
  Tsk, tsk, that's not the issue at all, the race is in the test framework.
-------------- next part --------------
=== modified file 'bzrlib/smart/server.py'
--- a/bzrlib/smart/server.py	2011-05-28 08:40:17 +0000
+++ b/bzrlib/smart/server.py	2011-05-28 09:10:36 +0000
@@ -136,10 +136,10 @@
 
     def serve(self, thread_name_suffix=''):
         self._should_terminate = False
-        self._started.set()
         # for hooks we are letting code know that a server has started (and
         # later stopped).
         self.run_server_started_hooks()
+        self._started.set()
         try:
             try:
                 while not self._should_terminate: