[Lldb-commits] [PATCH] Fix TestQuoting on remote targets.

Chaoren Lin chaorenl at google.com
Mon Mar 2 20:56:47 PST 2015

Comment at: test/settings/quoting/TestQuoting.py:74
@@ -71,1 +73,3 @@
+        dst_file_spec = lldb.SBFileSpec('stdout.txt', False)
+        platform.Get(src_file_spec, dst_file_spec);
         with open('stdout.txt', 'r') as f:
ovyalov wrote:
> chaoren wrote:
> > ovyalov wrote:
> > > Does it work in case of local run? It seems it might read and write data from/into the same file..
> > I had that concern too, but it works fine on local.
> I remember I had such issue with platform.Put when source and destination were in the same directory. 
> To be safe, please give them different names stdout-(local|remote).txt and check on OSX as well.
I guess we could just check if the file already exists, and if it does, assume local, and skip the Get(). Platform cannot tell if we're local, right? Also, platform doesn't provide any functionality to delete remote files, right?



More information about the lldb-commits mailing list