[libcxx-commits] [libcxx] WIP - [libc++][debugging] P2546R5: Debugging support & P2810R4: `is_debugger_present` `is_replaceable` (PR #81447)

[Mark de Wever via libcxx-commits]

================
@@ -500,6 +501,43 @@ def _getAndroidDeviceApi(cfg):
     ),
 ]
 
+
+# Detect whether dbx is on the system.
+def check_dbx(cfg):
+    dbx_path = shutil.which("dbx")
+    if dbx_path is None:
+        return False
+
+    return True
+
+
+DEFAULT_FEATURES += [
+    Feature(
+        name="host-has-dbx",
+        when=check_dbx,
+        actions=[AddSubstitution("%{dbx}", lambda cfg: shutil.which("dbx"))],
+    )
+]
+
+
+# Detect whether LLDB is on the system.
+def check_lldb(cfg):
----------------
mordante wrote:

This does not detect Python support in LLDB the name of the associated feature implies support `host-has-lldb-with-python`. I assume the tests require Python support so this looks like it needs more work. Maybe look at the LLDB test suite how they determine Python support.

https://github.com/llvm/llvm-project/pull/81447