[Lldb-commits] [PATCH] D61578: [Driver] Add command line option to allow loading local lldbinit file

[Jim Ingham via Phabricator via lldb-commits]

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