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

FIX: extension activate failed if there is a space in clangd path #715

Merged
merged 1 commit into from
Nov 6, 2024

Conversation

quanzhuo
Copy link
Contributor

@quanzhuo quanzhuo commented Nov 4, 2024

The last change 0c2d40d7d6b8092309bd2c880a32e59c71472a80 add shell: true to clangd options, which cause the extension activate failed if there is a space in clangd's path.

Especially on windows, clangd installed under 'C:\Program Files'

@quanzhuo quanzhuo closed this Nov 4, 2024
@quanzhuo quanzhuo reopened this Nov 4, 2024
@JVApen
Copy link
Contributor

JVApen commented Nov 4, 2024

Thanks for finding and fixing!

Copy link
Contributor

@HighCommander4 HighCommander4 left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Thanks for the patch!

src/clangd-context.ts Show resolved Hide resolved
@HighCommander4 HighCommander4 merged commit 4f64ca7 into clangd:master Nov 6, 2024
1 check passed
HighCommander4 added a commit to HighCommander4/vscode-clangd that referenced this pull request Nov 13, 2024
HighCommander4 added a commit that referenced this pull request Nov 13, 2024
JVApen pushed a commit to JVApen/vscode-clangd that referenced this pull request Nov 18, 2024
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants