>>! In D6553#16, @clayborg wrote:
> Fine to commit as long as you don't set the architecture in the launch_info 
> in Host::RunShellCommand() since that can break MacOSX.

Ok, I will take out the checks I added against the triple in 
ConvertArgumentsForLaunchingInShell for now and commit, then re-add them once 
the logic is properly separated in HostInfo.

Just so I can understand the code better, can you point me to the code that 
would break if the Architecture were set?  I see some in 
ConvertArgumentsForLaunchingInShell, but it's only exercised if will_debug == 
true, and RunShellCommand() uses will_debug == false.  So I'm assuming its 
something deep inside of Host::LaunchProcess.

http://reviews.llvm.org/D6553



_______________________________________________
lldb-commits mailing list
[email protected]
http://lists.cs.uiuc.edu/mailman/listinfo/lldb-commits

Reply via email to