Skip to content

Update default for iwyu.compile_commands#10

Merged
helly25 merged 5 commits intomainfrom
feat/find_compile_commands_under_build_20240707
Jul 8, 2024
Merged

Update default for iwyu.compile_commands#10
helly25 merged 5 commits intomainfrom
feat/find_compile_commands_under_build_20240707

Conversation

@helly25
Copy link
Copy Markdown
Owner

@helly25 helly25 commented Jul 7, 2024

If iwyu.compile_commands is set to the default auto, then the extension will try:

  • ${workspaceFolder}/compile_commands.json,
  • ${workspaceFolder}/build/compile_commands.json,
  • ${fileWorkspaceFolder}/compile_commands.json, and
  • ${fileWorkspaceFolder}/build/compile_commands.json.

This should address #5 in slightly better way as it supports the common places.

This fixes #11

helly25 added 3 commits July 8, 2024 09:41
… is not found, then the extension will also try `${workspaceFolder}/build/compile_commands.json`.

This should address #5 in slightly better way as it supports both common places.
@helly25 helly25 force-pushed the feat/find_compile_commands_under_build_20240707 branch from c1a3f13 to bc65d47 Compare July 8, 2024 07:43
@helly25 helly25 requested a review from Fab-Cat July 8, 2024 07:58
helly25 added 2 commits July 8, 2024 11:45
…ng the following:

* `${workspaceFolder}/compile_commands.json`,
* `${workspaceFolder}/build/compile_commands.json`,
* `${fileWorkspaceFolder}/compile_commands.json`, and
* `${fileWorkspaceFolder}/build/compile_commands.json`.
@helly25 helly25 enabled auto-merge (squash) July 8, 2024 09:57
@helly25 helly25 merged commit ee678af into main Jul 8, 2024
@helly25 helly25 deleted the feat/find_compile_commands_under_build_20240707 branch July 8, 2024 16:20
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.

Improve finding compile_commnands.json.

2 participants