author | Sebastian Harl <sh@tokkee.org> | |
Mon, 16 Jun 2008 21:02:36 +0000 (23:02 +0200) | ||
committer | Florian Forster <octo@huhu.verplant.org> | |
Tue, 17 Jun 2008 19:57:21 +0000 (21:57 +0200) | ||
commit | 43b3f7d1ce3195c93c5357bcafb60421699aab7d | |
tree | 2a0ede82c93689e8f07eaec4472649330a3188cc | tree | snapshot |
parent | 05b684693fe88f6adc58cfd0f1c95fa811d72bfc | commit | diff |
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>
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 | diff | blob | history | |
src/netlink.c | diff | blob | history | |
src/notify_desktop.c | diff | blob | history | |
src/plugin.c | diff | blob | history | |
src/rrdtool.c | diff | blob | history | |
src/snmp.c | diff | blob | history | |
src/utils_cache.c | diff | blob | history |