Add environment var LLDBVSCODE_SKIP_INIT_FILES to lldb-vscode

[wallace]

Summary

lldb has the -x option but this doesn't exist at all for lldb-vscode. To be honest my main goal is the ability to run the lldb-vscode tests without an lldbinit file polluting it.
The DAP doesn't seem to have it (https://microsoft.github.io/debug-adapter-protocol/specification#Requests_Initialize) so we're adding it as an env var.

[aadsm]

Oops, wrnog default

[clayborg]

If MS VS Code IDE itself will never pass this to the "initialize" packet, then we might just want to use an env var instead? You should probably get rid of https://reviews.llvm.org/D70886 and just modify this patch?

[aadsm]

Yes, I'll change this to env it makes more sense.

[aadsm]

Use an env var instead

[clayborg]

lldb/tools/lldb-vscode/lldb-vscode.cpp

64

make singleton function:

bool GetSkipInitFiles() {
  static const bool skip_init = getenv("LLDBVSCODE_SKIP_INIT_FILES") != NULL;
  return skip_init;
}

[aadsm]

Put the logic into a function

[wallace]

not needed anymore because of D99974

lldb/packages/Python/lldbsuite/test/tools/lldb-vscode/vscode.py

def request_testGetTargetBreakpoints(self):
         return self.send_recv(command_dict)
 
     def terminate(self):
         self.send.close()
         # self.recv.close()
 
 
 class DebugAdaptor(DebugCommunication):
-    def __init__(self, executable=None, port=None, init_commands=[]):
+    def __init__(
+        self, executable=None, port=None, init_commands=[], skip_init_files=True
+    ):
         self.process = None
         if executable is not None:
+            env = os.environ.copy()
+            if skip_init_files:
+                env['LLDBVSCODE_SKIP_INIT_FILES'] = "1"
             self.process = subprocess.Popen([executable],
                                             stdin=subprocess.PIPE,
                                             stdout=subprocess.PIPE,
-                                            stderr=subprocess.PIPE)
+                                            stderr=subprocess.PIPE,
+                                            env=env)
             DebugCommunication.__init__(self, self.process.stdout,
                                         self.process.stdin, init_commands)
         elif port is not None:
             s = socket.socket(socket.AF_INET, socket.SOCK_STREAM)
             s.connect(('127.0.0.1', port))
             DebugCommunication.__init__(self, s.makefile('r'), s.makefile('w'),
                 init_commands)
 

lldb/tools/lldb-vscode/lldb-vscode.cpp

 typedef int socklen_t;
 constexpr const char *dev_null_path = "nul";
 
 #else
 constexpr const char *dev_null_path = "/dev/null";
 
 #endif
 
 using namespace lldb_vscode;
 
 namespace {
 
 typedef void (*RequestCallback)(const llvm::json::Object &command);
 
 enum LaunchMethod { Launch, Attach, AttachForSuspendedLaunch };
 
 enum VSCodeBroadcasterBits { eBroadcastBitStopEventThread = 1u << 0 };
if (sourceMap) {
     strm << "\".\" \"" << sourcePath << "\"";
   }
   strm.flush();
   if (!sourceMapCommand.empty()) {
     g_vsc.RunLLDBCommands("Setting source map:", {sourceMapCommand});
   }
 }
 
+const bool GetSkipInitFiles() {
+  static const bool skip_init_files = getenv("LLDBVSCODE_SKIP_INIT_FILES") != NULL;
+  return skip_init_files;
+}
+
 // "AttachRequest": {
 //   "allOf": [ { "$ref": "#/definitions/Request" }, {
 //     "type": "object",
 //     "description": "Attach request; value of command field is 'attach'.",
 //     "properties": {
 //       "command": {
 //         "type": "string",
 //         "enum": [ "attach" ]

 //       "body": {
 //         "$ref": "#/definitions/Capabilities",
 //         "description": "The capabilities of this debug adapter."
 //       }
 //     }
 //   }]
 // }
 void request_initialize(const llvm::json::Object &request) {
-  g_vsc.debugger = lldb::SBDebugger::Create(true /*source_init_files*/);
+  const bool skip_init_files = GetSkipInitFiles();
+  g_vsc.debugger =
+      lldb::SBDebugger::Create(!skip_init_files /*source_init_files*/);
   // Create an empty target right away since we might get breakpoint requests
   // before we are given an executable to launch in a "launch" request, or a
   // executable when attaching to a process by process ID in a "attach"
   // request.
   FILE *out = llvm::sys::RetryAfterSignal(nullptr, fopen, dev_null_path, "w");
   if (out) {
     // Set the output and error file handles to redirect into nothing otherwise
     // if any code in LLDB prints to the debugger file handles, the output and