[MERGE][0.90][Bug 133965] PathNotChild, port mismatch with "bzr info" for bzr:// smartserver

[Vincent Ladeuil]

>>>>> "robert" == Robert Collins <robertc at robertcollins.net> writes:

    robert> On Tue, 2007-08-28 at 15:08 +1000, Robert Collins wrote:
    >> On Mon, 2007-08-27 at 08:51 +0200, Vincent Ladeuil wrote:
    >> > >>>>> "robert" == Robert Collins <robertc at robertcollins.net> writes:
    >> > 
    >> >     robert> Why does only ConnectedTransport get this? udp style protocols have
    >> >     robert> url's too...
    >> > 
    >> > Well the class was named 'ConnectedTransport' because
    >> > 'RemoteTransport' was already used :-)
    >> > 
    >> > But if udp schemes have to be handled (which ones are you
    >> > refering to ?) they may need to keep track of credentials too so
    >> > some sort of connection sharing may still be needed.
    >> 
    >> Well, Connected to me implies 'TCP'. But looking at it looks like it
    >> really is for 'Transports which need to share state'.
    >> 
    >> MemoryTransport needs to do this, and its clearly neither 'Remote' nor
    >> 'Connected'.
    >> 
    >> I'd like to see this renamed and everything that is not related to the
    >> sharing of state moved back up to the root transport.

    robert> Should I open a bug on this?

Yes please.

    Vincent