[Lldb-commits] [lldb] [lldb-dap] Include npm install in the extension installation steps (PR #92028)
via lldb-commits
lldb-commits at lists.llvm.org
Mon May 13 13:43:59 PDT 2024
llvmbot wrote:
<!--LLVM PR SUMMARY COMMENT-->
@llvm/pr-subscribers-lldb
Author: Walter Erquinigo (walter-erquinigo)
<details>
<summary>Changes</summary>
Otherwise the build step fails due to missing dependencies.
---
Full diff: https://github.com/llvm/llvm-project/pull/92028.diff
1 Files Affected:
- (modified) lldb/tools/lldb-dap/README.md (+2)
``````````diff
diff --git a/lldb/tools/lldb-dap/README.md b/lldb/tools/lldb-dap/README.md
index 274b1519208a1..16ce4672be71c 100644
--- a/lldb/tools/lldb-dap/README.md
+++ b/lldb/tools/lldb-dap/README.md
@@ -46,6 +46,7 @@ Installing the plug-in is very straightforward and involves just a few steps.
```bash
cd /path/to/lldb/tools/lldb-dap
+npm install
npm run package # This also compiles the extension.
npm run vscode-install
```
@@ -69,6 +70,7 @@ no effect.
```bash
# Bump version in package.json
cd /path/to/lldb/tools/lldb-dap
+npm install
npm run package
npm run vscode-install
```
``````````
</details>
https://github.com/llvm/llvm-project/pull/92028
More information about the lldb-commits
mailing list