Skip to content

Add a make target for compile-commands.json #2271

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

Open
wants to merge 3 commits into
base: main
Choose a base branch
from

Conversation

MrAlias
Copy link
Contributor

@MrAlias MrAlias commented May 3, 2025

Use bear to generate compile_commands.json. This is used for editor LSPs (i.e. clangd).

The compile-commands.json file itself is going to contain local absolute paths. Therefore, add a make target to generate it locally for each developer.

Use [bear](https://github.com/rizsotto/Bear) to generate
compile_commands.json. This is used for editor's LSPs (i.e. clangd).

The compile-commands.json file itself is going to contain local absolute
paths. Therefore, add a make target to generate it locally for each
developer.
@MrAlias MrAlias force-pushed the compile-commands branch from cf59b35 to 7294cf2 Compare May 3, 2025 16:26
@MrAlias MrAlias marked this pull request as ready for review May 5, 2025 14:21
@MrAlias MrAlias requested a review from a team as a code owner May 5, 2025 14:21
Copy link
Contributor

@grcevski grcevski left a comment

Choose a reason for hiding this comment

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

LGTM!

@RonFed
Copy link
Contributor

RonFed commented May 6, 2025

can you please give some more context about this? what are we planning to use it for?

@MrAlias
Copy link
Contributor Author

MrAlias commented May 6, 2025

This is used for editor LSPs (i.e. clangd).

That's my whole intention.

@RonFed
Copy link
Contributor

RonFed commented May 6, 2025

Got it. Do you think it's worth adding a mention to this in the contributing doc under Development?

@MrAlias
Copy link
Contributor Author

MrAlias commented May 6, 2025

Sounds like a good idea to me. I'll add a note.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants