[Lldb-commits] [PATCH] Fix some posixy assumptions that were used during RunShellCommand
Greg Clayton
clayborg at gmail.com
Fri Dec 5 15:30:04 PST 2014
Seems like we should either get this from a call like the one that gets "cmd.exe" (HostInfoWindows::GetDefaultShell()). In Host::RunShellCommand() we already do:
```
launch_info.SetShell(HostInfo::GetDefaultShell());
```
We should be asking the HostInfo class to be setting the shell arguments for us. Then we HostInfoPosix and HostInfoWindows could "do the right thing".
So I vote to remove ProcessLaunchInfo::ConvertArgumentsForLaunchingInShell(), then create a function in the HostInfo class that can do this correctly for each system. This function is only used in Platform::LaunchProcess() when "IsHost()" is true and in Host::RunShellCommand().
http://reviews.llvm.org/D6553
More information about the lldb-commits
mailing list