Sam Brightman
Sam Brightman
The second parameter to `setenv` should be `nil` (or omitted) to unset properly. That said, any chance of getting a release with this PR merged?
The question about releasing was for @necaris. @ArmCorelin you should adjust the `setenv` parameter as described.
I feel this PR still has some legs. Whilst clangd now checks `build/compile_commands.json` there remain quite a few variations (e.g. `build_xyz/`, `build/releasemode/`, `build/generatortype`, `out/`). It feels like restricting the search...
The patches above are insufficient to solve the problem for me. I think it also extends beyond the root dir being symlinked. When compile commands are found/specified, and indexing performed,...
Cask is being maintained and as far as I know has no issues on Emacs 26 for the last few months. I believe the main issue being referenced was a...
I still prefer this package to the more popular ones, to the extent I'm copying it around all my installations after it was removed from MELPA. Will it be making...
Is the above comment going to be merged into this PR with a rebase or should a new PR be made?
I can confirm the issue but haven't had chance to look into it yet.
Seems to me that it's still worth having the best performance in all use-cases. BCF has been discussed for years, but progress is slow. Shall we either merge or close...
This seems reasonable if it works. It would be good to: - add tests to prove it works - fix the merge conflicts with master - probably remove `which` function...