Fix for header-only compile actions by Bazel

[mikael-s-persson]

Fix for header-only compile actions by Bazel

This PR fixes, as far as I can tell, the issues related to #199 where Bazel adds header-only / syntax-only compilation actions that produce dummy output files (.processed). I believe this is part of Bazel open-sourcing more of its capabilities for preprocessing headers and handling pre-compiled modules.

The fix is based on #209 and makes the following changes:

• Adds '.h', '.inl', '.hh', '.hpp', '.hxx' to the list of expected source files for C and C++.
• Add '.processed' to the list of expected output file extensions for compiler actions.
• Removes '-fsyntax-only' or '/Zs' from the compilation arguments when running the preprocessor-only compilation step to silence any warnings about unused compiler arguments.

This fix was tested in this repo: https://github.com/mikael-s-persson/evdevpp
And also tested and used for a while in other private repos.
This seems to produce clean, non-racy outputs. AFAICT, the resulting compile_commands.json file contains everything it should, including the headers.

Fixes #199

[davidzchen]

Is ready to be committed? Can someone please review this?

[xFile3160]

LGTM

[mtanida]

Is this ready to be merged? We are running into the same issue...

[LaicepS]

This worked fine to produce a compilation database on a Abseil library. But I had to patch the refresh.template.py file in the bazel cache by hand. Is there another way (other than waiting for this to be merged)?

[davidzchen]

@cpsauer could you take a look at this when you have a chance? It would be great to finally get this merged. Thanks in advance.

[axbycc-mark]

Simply using @mikael-s-persson 's branch in your MODULE.bazel is a workaround until this gets merged into the main branch.

# Hedron's Compile Commands Extractor for Bazel
# https://github.com/hedronvision/bazel-compile-commands-extractor
bazel_dep(name = "hedron_compile_commands", dev_dependency = True)
git_override(
    module_name = "hedron_compile_commands",
    remote = "https://github.com/mikael-s-persson/bazel-compile-commands-extractor",
    commit = "f5fbd4cee671d8d908f37c83abaf70fba5928fc7"
)

[silversword05]

When is this going to be merged? Keep facing the same problems and figuring out these work-arounds.

[mrmarkwell]

Please consider merging this fix. Thanks!

[silversword05]

bazel_dep(name = "hedron_compile_commands", dev_dependency = True)
git_override(
module_name = "hedron_compile_commands",
remote = "https://github.com/mikael-s-persson/bazel-compile-commands-extractor",
commit = "f5fbd4cee671d8d908f37c83abaf70fba5928fc7"
)

Please merge this.

[hhkblogi]

LGTM

[mikael-s-persson]

FYI, people on this thread might be interested in checking out my new repo "bazel_cc_meta" which combines the functionality of this compile-commands-extractor and the depend-on-what-you-use tool into a much more efficient solution (benefits from bazel caching through an aspect-based implementation). It is still highly experimental, needless to say, but we've been using it for a few weeks in a relatively complicated cross-compilation setup. And it should be less brittle to issues like the one that spurred this PR because it can rely on Bazel's own logic to classify sources and headers rather than inferring them externally.

https://github.com/mikael-s-persson/bazel_cc_meta

[skrassiev]

FYI, people on this thread might be interested in checking out my new repo "bazel_cc_meta" which combines the functionality of this compile-commands-extractor and the depend-on-what-you-use tool into a much more efficient solution (benefits from bazel caching through an aspect-based implementation). It is still highly experimental, needless to say, but we've been using it for a few weeks in a relatively complicated cross-compilation setup. And it should be less brittle to issues like the one that spurred this PR because it can rely on Bazel's own logic to classify sources and headers rather than inferring them externally.

https://github.com/mikael-s-persson/bazel_cc_meta

I tried it and unfortunately it does not generate the proper compile db for the simplest abseil hello world example, at least for clangd based LSP in vim. I.e. firing up vim would immediately report plenty of undefined symbols.