[Lldb-commits] [lldb] r154943 - in /lldb/branches/lldb-platform-work: include/lldb/Target/ source/Commands/ source/Plugins/Platform/MacOSX/ source/Plugins/Platform/POSIX/ source/Plugins/Platform/gdb-server/ source/Plugins/Process/gdb-remote/ source/Target/

[Enrico Granata]

Hi.

Thanks for your suggestions :)
Yes, it is a work in progress. And there are still a lot of TODOs. Making sure we keep stuff up-to-date is one of these.

The plan I have is to actually rely on UUIDs so that we push the file on the remote Mac if it does not exist OR if it does exist but has a different UUID.
My guess is that it is much easier to have wrong timestamps than have two different files share the same UUID.

As concerns the timestamp, there should be a complete stat() call on the Platform so that you could get all sorts of useful details on a remote file. Currently, we have size and exists as special cases, but they need to be merged along with other elements such as timestamp into a general-purpose stat() call.

Sincerely,
Enrico Granata
✉ egranata@.com
✆ (four oh eight) 862-7683

On Apr 17, 2012, at 4:06 PM, Filipe Cabecinhas wrote:

> Hi Enrico 
> 
> I know this is a work in progress, but I have a suggestion:
> 
> When checking the file, shouldn't we check if the local copy is newer and, in that case, upload it to the platform?
> As I see it, when doing remote debugging, you may want to always keep everything in you dev machine as much as possible (e.g: if developing on embedded platforms). In that case, Even if you have the file in the remote platform, it could be out of date.
> 
> But we would also need to flag if we got the file from the platform (because it didn't exist locally). In that case we don't want to push it there again.
> 
> This would require adding methods to get the timestamp for a file, at the very least.
> 
> Regards, 
> 
>  Filipe
> 
> 
> On Tuesday, April 17, 2012 at 8:39 PM, Enrico Granata wrote:
> 
>> + if (!platform_sp->GetFileExists (remote_file))
>> + {
>> + Error err = platform_sp->PutFile(file_spec, remote_file);
>> + if (err.Fail())
>> + {
> 
> 
> 

-------------- next part --------------
An HTML attachment was scrubbed...
URL: <http://lists.llvm.org/pipermail/lldb-commits/attachments/20120417/9e54d14a/attachment.html>