Rev 2725: Remove stale close_file_stream methods on Transport. in http://people.ubuntu.com/~robertc/baz2.0/repository

Robert Collins robertc at robertcollins.net
Wed Aug 22 09:09:08 BST 2007


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

------------------------------------------------------------
revno: 2725
revision-id: robertc at robertcollins.net-20070822080905-w1cvp6e5cubk0rai
parent: robertc at robertcollins.net-20070822063025-cbnp2jeeshfx62mc
committer: Robert Collins <robertc at robertcollins.net>
branch nick: repository
timestamp: Wed 2007-08-22 18:09:05 +1000
message:
  Remove stale close_file_stream methods on Transport.
modified:
  bzrlib/transport/__init__.py   transport.py-20050711165921-4978aa7ce1285ad5
  bzrlib/transport/chroot.py     chroot.py-20061011104729-0us9mgm97z378vnt-1
  bzrlib/transport/decorator.py  decorator.py-20060402223305-e913a0f25319ab42
  bzrlib/transport/ftp.py        ftp.py-20051116161804-58dc9506548c2a53
  bzrlib/transport/local.py      local_transport.py-20050711165921-9b1f142bfe480c24
  bzrlib/transport/memory.py     memory.py-20051016101338-cd008dbdf69f04fc
  bzrlib/transport/remote.py     ssh.py-20060608202016-c25gvf1ob7ypbus6-1
  bzrlib/transport/sftp.py       sftp.py-20051019050329-ab48ce71b7e32dfe
=== modified file 'bzrlib/transport/__init__.py'
--- a/bzrlib/transport/__init__.py	2007-08-22 06:23:52 +0000
+++ b/bzrlib/transport/__init__.py	2007-08-22 08:09:05 +0000
@@ -361,15 +361,6 @@
         """
         raise NotImplementedError(self.clone)
 
-    def close_file_stream(self, relpath):
-        """Close a file stream at relpath.
-
-        :raises: NoSuchFile if there is no open file stream for relpath.
-        :seealso: open_file_stream.
-        :return: None
-        """
-        raise NotImplementedError(self.close_file_stream)
-
     def ensure_base(self):
         """Ensure that the directory this transport references exists.
 

=== modified file 'bzrlib/transport/chroot.py'
--- a/bzrlib/transport/chroot.py	2007-08-22 06:23:52 +0000
+++ b/bzrlib/transport/chroot.py	2007-08-22 08:09:05 +0000
@@ -95,9 +95,6 @@
     def clone(self, relpath):
         return ChrootTransport(self.server, self.abspath(relpath))
 
-    def close_file_stream(self, relpath):
-        return self._call('close_file_stream', relpath)
-
     def delete(self, relpath):
         return self._call('delete', relpath)
 

=== modified file 'bzrlib/transport/decorator.py'
--- a/bzrlib/transport/decorator.py	2007-08-22 06:23:52 +0000
+++ b/bzrlib/transport/decorator.py	2007-08-22 08:09:05 +0000
@@ -75,10 +75,6 @@
         return self.__class__(
             self._get_url_prefix() + decorated_clone.base, decorated_clone)
 
-    def close_file_stream(self, relpath):
-        """See Transport.close_file_stream."""
-        return self._decorated.close_file_stream(relpath)
-
     def delete(self, relpath):
         """See Transport.delete()."""
         return self._decorated.delete(relpath)

=== modified file 'bzrlib/transport/ftp.py'
--- a/bzrlib/transport/ftp.py	2007-08-22 06:23:52 +0000
+++ b/bzrlib/transport/ftp.py	2007-08-22 08:09:05 +0000
@@ -107,10 +107,6 @@
             self._set_connection(connection, credentials)
         return connection
 
-    def close_file_stream(self, relpath):
-        """See Transport.close_file_stream."""
-        del _file_streams[self.abspath(relpath)]
-
     def _create_connection(self, credentials=None):
         """Create a new connection with the provided credentials.
 

=== modified file 'bzrlib/transport/local.py'
--- a/bzrlib/transport/local.py	2007-08-22 06:23:52 +0000
+++ b/bzrlib/transport/local.py	2007-08-22 08:09:05 +0000
@@ -82,11 +82,6 @@
                 abspath = self.base
             return LocalTransport(abspath)
 
-    def close_file_stream(self, relpath):
-        """See Transport.close_file_stream."""
-        handle = transport._file_streams.pop(self.abspath(relpath))
-        handle.close()
-
     def _abspath(self, relative_reference):
         """Return a path for use in os calls.
 

=== modified file 'bzrlib/transport/memory.py'
--- a/bzrlib/transport/memory.py	2007-08-22 06:23:52 +0000
+++ b/bzrlib/transport/memory.py	2007-08-22 08:09:05 +0000
@@ -91,10 +91,6 @@
         result._locks = self._locks
         return result
 
-    def close_file_stream(self, relpath):
-        """See Transport.close_file_stream."""
-        del _file_streams[self.abspath(relpath)]
-
     def abspath(self, relpath):
         """See Transport.abspath()."""
         # while a little slow, this is sufficiently fast to not matter in our

=== modified file 'bzrlib/transport/remote.py'
--- a/bzrlib/transport/remote.py	2007-08-22 06:23:52 +0000
+++ b/bzrlib/transport/remote.py	2007-08-22 08:09:05 +0000
@@ -127,10 +127,6 @@
         # No credentials
         return None, None
 
-    def close_file_stream(self, relpath):
-        """See Transport.close_file_stream."""
-        del transport._file_streams[self.abspath(relpath)]
-
     def is_readonly(self):
         """Smart server transport can do read/write file operations."""
         resp = self._call2('Transport.is_readonly')

=== modified file 'bzrlib/transport/sftp.py'
--- a/bzrlib/transport/sftp.py	2007-08-22 06:23:52 +0000
+++ b/bzrlib/transport/sftp.py	2007-08-22 08:09:05 +0000
@@ -172,11 +172,6 @@
         super(SFTPTransport, self).__init__(base,
                                             _from_transport=_from_transport)
 
-    def close_file_stream(self, relpath):
-        """See Transport.close_file_stream."""
-        handle = _file_streams.pop(self.abspath(relpath))
-        handle.close()
-
     def _remote_path(self, relpath):
         """Return the path to be passed along the sftp protocol for relpath.
         



More information about the bazaar-commits mailing list