[merge] FTP now has and passes regression tests
Martin Pool
mbp at canonical.com
Mon May 22 04:17:51 BST 2006
On 18 May 2006, John A Meinel <john at arbash-meinel.com> wrote:
> -from bzrlib.transport import Transport
> -from bzrlib.errors import (TransportNotPossible, TransportError,
> - NoSuchFile, FileExists, DirectoryNotEmpty)
> +try:
> + import medusa
> + import medusa.filesys
> + import medusa.ftp_server
> +except ImportError:
> + _have_medusa = False
> +else:
> + _have_medusa = True
This will import medusa and asyncore every time the ftp module is loaded, which
seems a bit unnecessary.
More generally it seems a bit wasteful to define the server code in
the transport modules and so load it every time when we almost always
only want the client. But you don't need to change that now.
+1 from me.
--
Martin
More information about the bazaar
mailing list