Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

[lldb-dap] Include npm install in the extension installation steps #92028

Merged
merged 1 commit into from
May 15, 2024

Conversation

walter-erquinigo
Copy link
Member

Otherwise the build step fails due to missing dependencies.

Otherwise the build step fails due to missing dependencies.
@llvmbot
Copy link
Collaborator

llvmbot commented May 13, 2024

@llvm/pr-subscribers-lldb

Author: Walter Erquinigo (walter-erquinigo)

Changes

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 --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
 ```

@walter-erquinigo walter-erquinigo merged commit 411bf38 into llvm:main May 15, 2024
5 of 6 checks passed
@walter-erquinigo walter-erquinigo deleted the walter/dap branch May 15, 2024 18:44
mub-at-arm pushed a commit to mub-at-arm/llvm-project that referenced this pull request May 16, 2024
…lvm#92028)

Otherwise the build step fails due to missing dependencies.
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

2 participants