[Lldb-commits] [PATCH] D68258: [Windows] Introduce a switch for the `lldb-server` mode on Windows

Wed Oct 2 08:40:00 PDT 2019

amccarth added a comment.

In D68258#1690757 <https://reviews.llvm.org/D68258#1690757>, @aleksandr.urakov wrote:

> In D68258#1690756 <https://reviews.llvm.org/D68258#1690756>, @aleksandr.urakov wrote:
>
> > I've made it in the way similar to Zachary have made for the `SymbolFileNativePDB` plugin. An environment variable could be more convenient e.g. to run a bunch of tests using the `lldb-test` option.
>

The environment variable for using the native PDB was always intended to be temporary, since the native PDB reader would eventually be the only PDB reader.

In general, I find environment variables harder to manage (especially on Windows) than command line options.  They're invisible global variables that compound all the environmental issues that make building and testing so flaky and hard to diagnose.

Repository:
  rLLDB LLDB

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

https://reviews.llvm.org/D68258