Closes #1304. You can view, comment on, or merge this pull request online at:
https://github.com/geany/geany-plugins/pull/1309
-- Commit Summary --
* Add missing sign for cppcheck --template argument
-- File Changes --
M build/cppcheck.mk (2)
-- Patch Links --
https://github.com/geany/geany-plugins/pull/1309.patch https://github.com/geany/geany-plugins/pull/1309.diff
@elextr I couldn't resist to credit this great and very sophisticated change to you as you already solved the mystery in https://github.com/geany/geany-plugins/issues/1304#issuecomment-1912600670!
I just RTFineM for cppcheck, but I don't use it, so thats why I referred it to @b4n who wrote it originally or anybody else who has a new enough version that `--template` is an official option, not UU (undocumented unofficial).
@elextr approved this pull request.
@eht16, To perhaps be more clear, I don't have a new enough cppcheck to test this, but if it works for you approved although I'm not sure why the output format needs to be forced to some specific format.
@eht16 pushed 1 commit.
146abc696b2d40ba6178328df4184794e40ce106 Add more custom macro definitions for recent cppcheck versions
I tested the `--template=gcc` format locally with the latest cppcheck version and it works.
The idea by using the specific "gcc" template was probably that the output can be parsed by Geany out of the box using the default error message regex for C/C++ files. But this is just a guess.
I noticed new "undefined macro" errors while testing and added those as well (same story as in #1196).
With these changes, "make check" ran fine on my system.
See also #1310 for an improved CI build with using the current "cppcheck" version instead of the dated version from the CI runner image.
Merged #1309 into master.
github-comments@lists.geany.org