Use -Wall -Werror (AM_CFLAGS) when building any module.
authorSebastian Harl <sh@tokkee.org>
Mon, 16 Jun 2008 21:02:36 +0000 (23:02 +0200)
committerFlorian Forster <octo@huhu.verplant.org>
Tue, 17 Jun 2008 19:57:21 +0000 (21:57 +0200)
commit43b3f7d1ce3195c93c5357bcafb60421699aab7d
tree2a0ede82c93689e8f07eaec4472649330a3188cc
parent05b684693fe88f6adc58cfd0f1c95fa811d72bfc
Use -Wall -Werror (AM_CFLAGS) when building any module.

A few plugins have to set custom CFLAGS. When using "<plugin>_la_CFLAGS =",
$(AM_CFLAGS) will no longer be used when compiling <plugin> as that resets the
CFLAGS to the empty string. So, in that case, we have to explicitly add
$(AM_CFLAGS) to the custom CFLAGS of that plugin.

After doing so, a couple of problems have been identified which have been
fixed in this patch as well. That includes a possible SEGFAULT in the snmp
plugin which was missing an argument to WARNING().

Signed-off-by: Sebastian Harl <sh@tokkee.org>
Signed-off-by: Florian Forster <octo@huhu.verplant.org>
src/Makefile.am
src/netlink.c
src/notify_desktop.c
src/plugin.c
src/rrdtool.c
src/snmp.c
src/utils_cache.c