Rev 2589: Increase comment clarity from review. in http://people.ubuntu.com/~robertc/baz2.0/external_url

Robert Collins robertc at robertcollins.net
Thu Jul 5 08:00:31 BST 2007


At http://people.ubuntu.com/~robertc/baz2.0/external_url

------------------------------------------------------------
revno: 2589
revision-id: robertc at robertcollins.net-20070705070026-5bfnbk99qfv6yv9i
parent: robertc at robertcollins.net-20070705023829-5l21034oalccsz7h
committer: Robert Collins <robertc at robertcollins.net>
branch nick: external_url
timestamp: Thu 2007-07-05 17:00:26 +1000
message:
  Increase comment clarity from review.
modified:
  bzrlib/smart/server.py         server.py-20061110062051-chzu10y32vx8gvur-1
=== modified file 'bzrlib/smart/server.py'
--- a/bzrlib/smart/server.py	2007-07-05 02:38:29 +0000
+++ b/bzrlib/smart/server.py	2007-07-05 07:00:26 +0000
@@ -69,17 +69,20 @@
         # for hooks we are letting code know that a server has started (and
         # later stopped).
         # There are three interesting urls:
-        # The URL the server can be contacted on.
+        # The URL the server can be contacted on. (e.g. bzr://host/)
         # The URL that a commit done on the same machine as the server will
-        # have within the servers space.
+        # have within the servers space. (e.g. file:///home/user/source)
         # The URL that will be given to other hooks in the same process -
-        # The URL of the backing transport itself.
+        # the URL of the backing transport itself. (e.g. chroot+:///)
         # We need all three because:
         #  * other machines see the first
         #  * local commits on this machine should be able to be mapped to
         #    this server 
         #  * commits the server does itself need to be mapped across to this
         #    server.
+        # The latter two urls are different aliases to the servers url,
+        # so we group those in a list - as there might be more aliases 
+        # in the future.
         backing_urls = [self.backing_transport.base]
         try:
             backing_urls.append(self.backing_transport.external_url())



More information about the bazaar-commits mailing list