From 0b650165a13afc8b518b1ff0c82fa58000732cf5 Mon Sep 17 00:00:00 2001 From: bescoto Date: Sat, 1 Nov 2003 21:04:28 +0000 Subject: Quoted RPaths are now transmitted correctly (no conversion to RPaths) git-svn-id: http://svn.savannah.nongnu.org/svn/rdiff-backup/trunk@484 2b77aa54-bcbc-44c9-a7ec-4f6cf2b41109 --- rdiff-backup/rdiff_backup/connection.py | 16 ++++++++++++++++ rdiff-backup/testing/connectiontest.py | 11 ++++++++++- 2 files changed, 26 insertions(+), 1 deletion(-) diff --git a/rdiff-backup/rdiff_backup/connection.py b/rdiff-backup/rdiff_backup/connection.py index 3961a54..e92402f 100644 --- a/rdiff-backup/rdiff_backup/connection.py +++ b/rdiff-backup/rdiff_backup/connection.py @@ -103,6 +103,7 @@ class LowLevelPipeConnection(Connection): b - string q - quit signal R - RPath + Q - QuotedRPath r - RORPath only c - PipeConnection object @@ -127,6 +128,8 @@ class LowLevelPipeConnection(Connection): log.Log.conn("sending", obj, req_num) if type(obj) is types.StringType: self._putbuf(obj, req_num) elif isinstance(obj, connection.Connection):self._putconn(obj, req_num) + elif isinstance(obj, FilenameMapping.QuotedRPath): + self._putqrpath(obj, req_num) elif isinstance(obj, rpath.RPath): self._putrpath(obj, req_num) elif isinstance(obj, rpath.RORPath): self._putrorpath(obj, req_num) elif ((hasattr(obj, "read") or hasattr(obj, "write")) @@ -164,6 +167,12 @@ class LowLevelPipeConnection(Connection): rpath.index, rpath.data) self._write("R", cPickle.dumps(rpath_repr, 1), req_num) + def _putqrpath(self, qrpath, req_num): + """Put a quoted rpath into the pipe (similar to _putrpath above)""" + qrpath_repr = (qrpath.conn.conn_number, qrpath.base, + qrpath.index, qrpath.data) + self._write("Q", cPickle.dumps(qrpath_repr, 1), req_num) + def _putrorpath(self, rorpath, req_num): """Put an rorpath into the pipe @@ -235,6 +244,7 @@ class LowLevelPipeConnection(Connection): result = iterfile.FileToRORPIter(VirtualFile(self, int(data))) elif format_string == "r": result = self._getrorpath(data) elif format_string == "R": result = self._getrpath(data) + elif format_string == "Q": result = self._getqrpath(data) else: assert format_string == "c", header_string result = Globals.connection_dict[int(data)] @@ -252,6 +262,12 @@ class LowLevelPipeConnection(Connection): return rpath.RPath(Globals.connection_dict[conn_number], base, index, data) + def _getqrpath(self, raw_qrpath_buf): + """Return QuotedRPath object from raw buffer""" + conn_number, base, index, data = cPickle.loads(raw_qrpath_buf) + return FilenameMapping.QuotedRPath( + Globals.connection_dict[conn_number], base, index, data) + def _close(self): """Close the pipes associated with the connection""" self.outpipe.close() diff --git a/rdiff-backup/testing/connectiontest.py b/rdiff-backup/testing/connectiontest.py index 265862a..9b1d202 100644 --- a/rdiff-backup/testing/connectiontest.py +++ b/rdiff-backup/testing/connectiontest.py @@ -1,7 +1,7 @@ import unittest, types, tempfile, os, sys from commontest import * from rdiff_backup.connection import * -from rdiff_backup import Globals, rpath +from rdiff_backup import Globals, rpath, FilenameMapping class LocalConnectionTest(unittest.TestCase): """Test the dummy connection""" @@ -144,6 +144,15 @@ class PipeConnectionTest(unittest.TestCase): assert self.conn.reval("lambda rp: rp.data", rp) == rp.data assert self.conn.reval("lambda rp: rp.conn is Globals.local_connection", rp) + def testQuotedRPaths(self): + """Test transmission of quoted rpaths""" + qrp = FilenameMapping.QuotedRPath(self.conn, + "testfiles/various_file_types/regular_file") + assert self.conn.reval("lambda qrp: qrp.data", qrp) == qrp.data + assert qrp.isreg(), qrp + qrp_class_str = self.conn.reval("lambda qrp: str(qrp.__class__)", qrp) + assert qrp_class_str.find("QuotedRPath") > -1, qrp_class_str + def testExceptions(self): """Test exceptional results""" self.assertRaises(os.error, self.conn.os.lstat, -- cgit v1.2.1