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

Zachary Turner zturner at google.com
Mon Dec 8 13:40:00 PST 2014


I resubmitted this in r223695.  My original patch was actually ok, the
issue happened when I removed the code for setting the Architecture in the
ProcessLaunchInfo, I inadvertently removed the code that added -c to the
command line of /bin/sh.  Since toward the end of the thread we determined
that setting the architecture was actually ok (since it's checked inside of
a will_debug block, I added this code back in.

I built and ran on both Linux and Mac, and did not see any failures.  On
the off chance it does fail again though, let me know.

On Fri Dec 05 2014 at 5:43:03 PM Jim Ingham <jingham at apple.com> wrote:

> I did it (r223568.)
>
> Jim
>
> REPOSITORY
>   rL LLVM
>
> http://reviews.llvm.org/D6553
>
>
>
> _______________________________________________
> lldb-commits mailing list
> lldb-commits at cs.uiuc.edu
> http://lists.cs.uiuc.edu/mailman/listinfo/lldb-commits
>
-------------- next part --------------
An HTML attachment was scrubbed...
URL: <http://lists.llvm.org/pipermail/lldb-commits/attachments/20141208/e3f56660/attachment.html>


More information about the lldb-commits mailing list