[Lldb-commits] [lldb] r179831 - Change TargetList::CreateTarget to realpath the c-string filename of the

Greg Clayton gclayton at apple.com
Fri Apr 19 10:30:40 PDT 2013


FYI, this might break the argument zero stuff where people use a symlink to their program and want that to be used as the first argument. You might look at the history of this function to make sure...

On Apr 18, 2013, at 7:36 PM, Jason Molenda <jmolenda at apple.com> wrote:

> Author: jmolenda
> Date: Thu Apr 18 21:36:24 2013
> New Revision: 179831
> 
> URL: http://llvm.org/viewvc/llvm-project?rev=179831&view=rev
> Log:
> Change TargetList::CreateTarget to realpath the c-string filename of the
> executable -- if I do "(lldb) file ~/bin/exe" that tilde expansion needs
> to take place here.
> 
> Modified:
>    lldb/trunk/source/Target/TargetList.cpp
> 
> Modified: lldb/trunk/source/Target/TargetList.cpp
> URL: http://llvm.org/viewvc/llvm-project/lldb/trunk/source/Target/TargetList.cpp?rev=179831&r1=179830&r2=179831&view=diff
> ==============================================================================
> --- lldb/trunk/source/Target/TargetList.cpp (original)
> +++ lldb/trunk/source/Target/TargetList.cpp Thu Apr 18 21:36:24 2013
> @@ -157,7 +157,7 @@ TargetList::CreateTarget (Debugger &debu
>     if (!arch.IsValid())
>         arch = specified_arch;
> 
> -    FileSpec file (user_exe_path, false);
> +    FileSpec file (user_exe_path, true);
>     bool user_exe_path_is_bundle = false;
>     char resolved_bundle_exe_path[PATH_MAX];
>     resolved_bundle_exe_path[0] = '\0';
> 
> 
> _______________________________________________
> lldb-commits mailing list
> lldb-commits at cs.uiuc.edu
> http://lists.cs.uiuc.edu/mailman/listinfo/lldb-commits




More information about the lldb-commits mailing list