jingham added a comment.

In D61578#1493875 <https://reviews.llvm.org/D61578#1493875>, @labath wrote:

> In D61578#1493863 <https://reviews.llvm.org/D61578#1493863>, @jingham wrote:
>
> > The dotest.py tests all disable reading the global .lldbinit file.  Do the 
> > lit tests not do that as well?  For tests that actually test reading the 
> > user's .lldbinit file we will need to do something fancy (can we change the 
> > HOME environment variable to some other place so that it gets read from 
> > there?)
>
>
> Yes, they do. However, this test specifically disables that behavior because 
> otherwise we wouldn't even read the CWD lldbinit. Changing the HOME variable 
> might be enough to address this though (and I see Jonas already did that).


Ah, excellent.  That's probably good enough.

The notion of being able to specify to the driver where you store the global 
.lldbinit file is not 100% crazy - for instance if you had several different 
configurations you wanted to run lldb in, you could just switch global lldb 
dirs (e.g. using various shell aliases for the different global locations).  So 
if this does cause problems adding that wouldn't be a totally silly way to fix 
the tests.  But I'm not sure it's worth adding on its own.


Repository:
  rL LLVM

CHANGES SINCE LAST ACTION
  https://reviews.llvm.org/D61578/new/

https://reviews.llvm.org/D61578



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

Reply via email to