[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