> +
> +%C%_spellcheck_la_CFLAGS = \
> + $(AM_CFLAGS) \
> + $(ENCHANT_CFLAGS)
> +
> +if HAVE_ENCHANT_1_5
> +%C%_spellcheck_la_CFLAGS += -DHAVE_ENCHANT_1_5
> +endif
> +
> +%C%_spellcheck_la_LIBADD = \
> + $(COMMONLIBS) \
> + $(ENCHANT_LIBS)
> +
> +%C%_spellcheck_la_CPPCHECKFLAGS = \
> + $(AM_CPPCHECKFLAGS) \
> + -DSCE_PAS_DEFAULT=0
That's cute, but `AM_CPPCHECKFLAGS` is not a real Automake variable, and we don't support per-target versions of it. I have no good solution to propose here, but maybe having the cppcheck snippet define plugin-specific targets that get added to check-local, something like
```make
check-cppcheck-$(plugin): $(something clever) ...
check-local: check-cppcheck-$(plugin)
```
That'd probably work just fine for both recursive and non-recursive versions, and "only" requires additionally having `$(plugin)` defined -- but for the `$(something clever)` part I'm not certain how to handle… maybe defining `reldir=%D%` in the caller and using that somehow?
---
Reply to this email directly or view it on GitHub:
https://github.com/geany/geany-plugins/pull/277/files#r46152993