I am pretty sure /bin/sh is supposed to be a posix shell.  We could probably 
use that as well if we felt that there might be an OS X without /bin/bash.  

But we aren't using any but the most basic properties of the shell along this 
code path.  It isn't being done for shell expansion or anything like that,  So 
there isn't much point in making it overridable.

Jim

> On Oct 18, 2016, at 4:26 PM, Todd Fiala via lldb-commits 
> <lldb-commits@lists.llvm.org> wrote:
> 
> Actually that's a good point. We can default to bash but add a setting to 
> override. The issue we have to solve is that the current approach fails with 
> non-POSIX shells.
> 
> -Todd
> 
>> On Oct 18, 2016, at 4:09 PM, Joerg Sonnenberger <jo...@netbsd.org> wrote:
>> 
>> joerg added a comment.
>> 
>> OK, this is OSX specific, but I still think that hardcoding bash is a bad 
>> idea. Does it need anything not in `/bin/sh`?
>> 
>> 
>> https://reviews.llvm.org/D25750
>> 
>> 
>> 
> _______________________________________________
> lldb-commits mailing list
> lldb-commits@lists.llvm.org
> http://lists.llvm.org/cgi-bin/mailman/listinfo/lldb-commits

_______________________________________________
lldb-commits mailing list
lldb-commits@lists.llvm.org
http://lists.llvm.org/cgi-bin/mailman/listinfo/lldb-commits

Reply via email to