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

add output_dir feature #210

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

Custom output path for compile_commands.json
2 participants