This is an archive of the discontinued LLVM Phabricator instance.

[clangd] Add an option to specify a workspece-level config file
Needs ReviewPublic

Authored by daiyousei-qz on Oct 15 2022, 10:06 PM.

Details

Reviewers
nridge
kadircet
Summary

Add an option "--workspace-config=<path>" to specify a workspace config file, which stands between the in-tree .clangd and the global user config file and offers per-workspace customization.

Diff Detail

Event Timeline

daiyousei-qz created this revision.Oct 15 2022, 10:06 PM
Herald added a project: Restricted Project. · View Herald TranscriptOct 15 2022, 10:06 PM
Herald added a subscriber: arphaman. · View Herald Transcript
daiyousei-qz requested review of this revision.Oct 15 2022, 10:06 PM
Herald added a project: Restricted Project. · View Herald TranscriptOct 15 2022, 10:06 PM
daiyousei-qz retitled this revision from Add an option to specify a workspece-level config file to [clangd] Add an option to specify a workspece-level config file.Oct 15 2022, 10:06 PM

I have some concerns about adding an undefined "workspace" concept, and in general extending the config system in a way that's hard to use (because of the need to customize flags per workspace). Maybe best to discuss on the bug? https://github.com/clangd/clangd/issues/1336