[Lldb-commits] [PATCH] D74636: [lldb-vscode] Add inheritEnvironment option
walter erquinigo via Phabricator via lldb-commits
lldb-commits at lists.llvm.org
Fri Mar 20 14:07:14 PDT 2020
wallace updated this revision to Diff 251754.
wallace added a comment.
I added some tests cases to show why I used "settings set target.inherit-env".
There are currently two ways to launch a process. Either with the plain "program" argument,
or with the "launchCommands" argument. The latter is assumed to create a target by executing
arbitrary commands, which may go through CommandObjectProcess.
As by default target.inherit-env is true, if we first set its value to what we got from the
inheritEnvironment argument, then both kinds of launchers would behave the same way.
I haven't found an API for changing this setting, so I ended up invoking the command.
Repository:
rG LLVM Github Monorepo
CHANGES SINCE LAST ACTION
https://reviews.llvm.org/D74636/new/
https://reviews.llvm.org/D74636
Files:
lldb/packages/Python/lldbsuite/test/tools/lldb-vscode/lldbvscode_testcase.py
lldb/packages/Python/lldbsuite/test/tools/lldb-vscode/vscode.py
lldb/test/API/tools/lldb-vscode/environmentVariables/Makefile
lldb/test/API/tools/lldb-vscode/environmentVariables/TestVSCode_environmentVariables.py
lldb/test/API/tools/lldb-vscode/environmentVariables/main.cpp
lldb/tools/lldb-vscode/lldb-vscode.cpp
lldb/tools/lldb-vscode/package.json
-------------- next part --------------
A non-text attachment was scrubbed...
Name: D74636.251754.patch
Type: text/x-patch
Size: 10803 bytes
Desc: not available
URL: <http://lists.llvm.org/pipermail/lldb-commits/attachments/20200320/efb617a6/attachment-0001.bin>
More information about the lldb-commits
mailing list