[Lldb-commits] [PATCH] Fix some posixy assumptions that were used during RunShellCommand

[Zachary Turner]

>>! In D6553#13, @clayborg wrote:
> 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().

Looks like we crossed paths, but we basically said the same thing, that the function should be reworked.  Can I commit the fix as-is (since it doesn't break anything, due to the existing Architecture being uninitialized), and then follow up with the re-work into HostInfo?

The reason for comitting as-is first is because this patch contains some other stuff, like the changes in FileSpec, and the temp file paths, that would be unrelated to a refactor of this function.

http://reviews.llvm.org/D6553