This fixes build with recent versions of *cppcheck* with some plugins using `PLUGIN_VERSION_CHECK()` macro, like the *latex* plugin.
---
@eht16 already tested this with the nightly build environment, but I'm PRing this to make sure the CI tests here also pass. And extra review is always welcome ;) You can view, comment on, or merge this pull request online at:
https://github.com/geany/geany-plugins/pull/947
-- Commit Summary --
* cppcheck: Set Geany include path for it to find header files
-- File Changes --
M build/cppcheck.m4 (4) M build/cppcheck.mk (1)
-- Patch Links --
https://github.com/geany/geany-plugins/pull/947.patch https://github.com/geany/geany-plugins/pull/947.diff
@b4n pushed 2 commits.
10a26171e92c525f567fe0e8902f1e2779acdcb5 Define PKG_CHECK_VAR for old pkg-config (<= 0.28) e07cba1e1a8b13ab70afe69bd4b0d55a5aa3a60a Define AS_VAR_COPY for old Autoconf
I'm PRing this to make sure the CI tests here also pass.
…which was a good thing as apparently our CI has a too old pkg-config that doesn't provide `PKG_CHECK_VAR()`. I thus did similar to what cyptsetup [did in its](https://gitlab.com/cryptsetup/cryptsetup/commit/03581fc8ccf1acdaecbd808c52f4... own side](https://gitlab.com/cryptsetup/cryptsetup/commit/eb0293a6378b71e038f47440e3dd...), that is providing copies when missing.
LGBI
b4n commented on this pull request.
@@ -0,0 +1,18 @@
+dnl taken from Autoconf's m4sh.m4, GPLv3+ +m4_ifndef([AS_VAR_COPY], [ +m4_define([AS_VAR_COPY], +[AS_LITERAL_WORD_IF([$1[]$2], [$1=$$2], [eval $1=$$2])]) +])
I'm not totally sure about this part: it might not be needed, and if it is I'm not sure `AS_LITERAL_WORD_IF` would be available. This code is what Autoconf has nowadays, but the inspiration I got from cryptsetup has `AS_LITERAL_IF` instead (but it seems to be buggy in some cases if I trust the documentation). Anyway, we can fix this if a problem arises
Merged #947 into master.
github-comments@lists.geany.org