Skip to content

add output_dir feature #210

New issue

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

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

Already on GitHub? # to your account

Open
wants to merge 1 commit into
base: main
Choose a base branch
from
Open

add output_dir feature #210

wants to merge 1 commit into from

Conversation

ilev4ik
Copy link

@ilev4ik ilev4ik commented Aug 21, 2024

This pull request is about to provide a support of writing out compile_commands.json relatively to workspace root to provide each project means of customizing their .clangd.

By default compile-commands-extractor's refresh_compile_command produces its output into bazel-workspace root. But our monorepository is single-rooted workspace and has many subprojects as children folders. Multiple projects are maintained by one team of developers, but a single project may have its own clangd configuration.

I didn't find any customization attribute of bazel target so created this PR. If there exist any please let me know. TY

@cpsauer it would be great if you find some time to review PR :)

Closes #217

@ilev4ik
Copy link
Author

ilev4ik commented Jan 13, 2025

Please review this pr :)
@matta @garymm @manshengyang @pqn

@garymm
Copy link
Contributor

garymm commented Jan 13, 2025

@ilev4ik please don't mention people who don't even have the permissions to merge your PR (i.e. me, probably some others).

FYI I suspect there's a chance this project will no longer be maintained now that Hedron Vision was acquired, though I have no inside information.

@ilev4ik
Copy link
Author

ilev4ik commented Jan 13, 2025

@garymm how can I check if you have a permission or not? I relied on autocomplete when mentioning. It seems it was a mistake

My apologises for inconvenience

@ilev4ik ilev4ik closed this Jan 13, 2025
@ilev4ik ilev4ik reopened this Jan 14, 2025
# for free to join this conversation on GitHub. Already have an account? # to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

Custom output path for compile_commands.json
2 participants