Merge pull request #1742 from tokkee/sh/suppress

Valgrind: Suppress a bogus invalid read on FreeBSD.

Merge pull request #1741 from rubenk/fix-leak-in-disk-plugin

Plug a memory leak in disk plugin

Valgrind: Suppress a bogus invalid read on FreeBSD.

Valgrind reports an invalid read in common.c:parse_value. The error is
supposed to happen in strlen() on a just previously strdup()ed string.

Possibly, this is some sort of alignment issue. The read size is reported to
be 4 on a buffer of size 3.

https://bugzilla.redhat.com/show_bug.cgi?id=678518 may also be related.

Plug a memory leak in disk plugin

Introduced by bdbc066 so no need to backport.
CID 116311

Makefile: Add testwrapper.sh to dist tarballs.

Merge pull request #1740 from rubenk/correct-am_cflags

Append -Werror to the right flags

chrony plugin: constify 2 function arguments

Fixes a couple of dozen of these warnings:
chrony.c:889:20: warning: passing 'const char [14]' to parameter of type
'char *' discards qualifiers
[-Wincompatible-pointer-types-discards-qualifiers]
  chrony_push_data("clock_stratum", "chrony", (__extension__ ({ unsigned
short int __v, __x = (unsigned short int)
(chrony_resp.body.tracking.f_stratum); if (__builtin_constant_p (__x))
__v = ((unsigned short int) ((((__x) >> 8) & 0xff) | (((__x) & 0xff) <<
8))); else __asm__ ("rorw $8, %w0" : "=r" (__v) : "0" (__x) : "cc");
__v; })));
                   ^~~~~~~~~~~~~~~

chrony plugin: fix clang warning

chrony.c:745:11: warning: declaration shadows a local variable
[-Wshadow]
      int fh = open("/dev/random", 00);
          ^

collectd.spec: add missing %endif

configure.ac: fix autoreconf warnings

configure.ac:804: warning: AC_LANG_CONFTEST: no AC_LANG_SOURCE call
detected in body
../../lib/autoconf/lang.m4:193: AC_LANG_CONFTEST is expanded from...
configure.ac:804: the top level

Append -Werror to the right flags

testwrapper.sh: Use /bin/sh instead of /bin/bash.

We don't use any bashisms.

Merge pull request #1739 from tokkee/sh/memcheck

unit tests: Run all tests through Valgrind's memcheck, if available.

Actually add the testwrapper.sh script.

… it's now used for running all tests.

configure: Drop unneeded AC_SUBST() calls for AC_PATH_PROG() results.

AC_PATH_PROG calls AC_SUBST for us.

unit tests: Run all tests through Valgrind's memcheck, if available.

collectd.spec: add new gRPC plugin

Disable it for now as the version of protobuf-compiler
in RHEL and Fedora is too old.

ceph_test: Free the yajl handle.

utils_mount_test: Free dynamically allocated strings.

testing: Evaluate expressions only once in EXPECT_EQ_STR().

utils_vl_lookup: Free regexen when destroying class lists.

Else, we'll leak any memory used by the regex.

Merge pull request #1738 from tokkee/sh/silent

src/Makefile: Support silent build rules for pod2man.

src/Makefile: Support silent build rules for pod2man.

Merge pull request #1699 from tokkee/sh/grpc

gRPC server plugin

gRPC doc: Fixed typos and mention C++11 requirement.

src/Makefile: Don't unnecessarily set plugin specific CXXFLAGS.

Also, drop unneeded -lphread / -ldl from grpc's libraries.

src/Makefile: Don't set subdir-objects.

That's now done globally via configure.

collectd.spec: fix rpmlint warning

collectd.src: W: summary-not-capitalized C statistics collection and
monitoring daemon

collectd.spec: make scriptlets work on Fedora

collectd.spec: only require python26 on RHEL < 6

Otherwise the else clause in the condition matches on Fedora too.

This makes the spec build on Fedora.

collectd.spec: disable plugins on older distro

Instead of keeping a list per RHEL version of which plugins to enable,
invert the logic and keep a list of plugins to disable when the distro
is too old. This way we only have to specify a plugin once, and this
also opens up the possibility to build on Fedora.

Merge pull request #1737 from rubenk/pthread-cleanup

Pthread cleanup

collectd.spec: sort list

collectd.spec: EPEL7 has a recent ganglia-devel

collectd.spec: sort plugin lists

collectd.spec: turn TODO into changelog entry

Add new Chrony plugin to rpm spec file

Remove HAVE_PTHREAD_H include guard

We can't build without pthread.h so guarding it is not necessary
anymore.

configure.ac: move AC_SUBST down to after define

Sort types.db

Merge pull request #1548 from bbczeuz/zseng_dev_chrony

[plugin] Chrony NTP service check

src/Makefile.am: merge EXTRA_DIST variables

src/Makefile.am: remove commented out line

src/Makefile.am: remove useless assigment

zfs_arc: simplify build logic

Use BUILD_SOLARIS instead of assuming that any OS not FreeBSD or Linux is Solaris.

src/Makefile.am: remove useless assignment

xencpu: add missing cppflags and ldflags

write_redis: append to CPPFLAGS, not CFLAGS

openvpn plugin: remove useless assignment

oracle plugin: use CPPFLAGS instead of CFLAGS

openldap plugin: use CPPFLAGS instead of CFLAGS

openldap plugin: use $PLUGIN_LDFLAGS

Just like the other plugins.

src/Makefile.am: remove useless assignment

mysql plugin: simplify build logic

The MySQL plugin can't be build without libmysql support.
./configure already takes care of that.

lvm: add missing cppflags and ldflags to Makefile

src/Makefile.am: remove useless assignment

ipvs: simplify logic a bit

src/Makefile.am: remove useless assignment

simplify Makefile.am for nginx and write_http

They can't be build without libcurl support,
./configure will take care of that.

Remove the now unused BUILD_WITH_LIBCURL conditional

Simplify Makefile.am for curl plugins

They can't be build without libcurl support,
./configure will take care of that.

aquaero: add flags to the right variable

src/Makefile.am: merge 2 lines

src/Makefile.am: simplify Apache plugin

It can't be build without libcurl support and
configure will take care of that.

configure.ac: break a few long lines up

Always build with pthread support

Until now you could run ./configure --with-libpthread=no.
This didn't actually compile though.

The daemon has assumed for a while now that pthread support
is available. Bail out in ./configure if this is not the case.

configure.ac: remove unused HAVE_LIBPTHREAD define

unixsock plugin: stop linking with libpthread

The collectd daemon already links with it.

statsd plugin: stop linking with libpthread

The collectd daemon already links with it.

snmp plugin: stop linking with libpthread

The collectd daemon already links with it.

nut plugin: stop linking with libpthread

The collectd daemon already links with it.

notify_email plugin: stop linking with libpthread

The collectd daemon already links with it.

network plugin: stop linking with libpthread

The collectd daemon already links with it.

java plugin: stop linking with libpthread

The collectd daemon already links with it.

exec plugin: stop linking with libpthread

The collectd daemon already links with it.

email plugin: stop linking with libpthread

The collectd daemon already links with it.

dns plugin: stop linking with libpthread

The collectd daemon already links with it.

all plugins: remove pthread.h include

plugin.h already includes pthread.h

Don't hardcode libpthread

collectd_tg: let automake compute dependencies

Automake already does the right thing based on the library added with _LDADD.
This also restores a missing dependency on libheap.la.

collectd_tg: simplify logic a bit

We always link collectdctl with libcollectdclient/libcollectdclient.la.
Make this more obvious.

collectdctl: let automake compute dependencies

Automake already does the right thing based on the library added with _LDADD

collectdctl: simplify logic a bit

We always link collectdctl with libcollectdclient/libcollectdclient.la.
Make this more obvious.

src/Makefile.am: let automake compute dependencies

Automake already does the right thing based on the library
added with _LDADD

src/Makefile.am: simplify logic a bit

We always link collectd_nagios with libcollectdclient/libcollectdclient.la.
Make this more obvious.

Removed braces on single line 'if'. Always using inet_ntop.

Macro fixes, Implemented comments from code review (style, defines)

src/Makefile.am: remove CPPFLAGS assignment

AM_CPPFLAGS are added by default, unless you override _CPPFLAGS

Merge branch 'collectd-5.5'

Conflicts:
src/perl.c

Merge branch 'collectd-5.4' into collectd-5.5

src/Makefile.am: remove unused variable

configure: Add some more information about protoc/gRPC in the summary.

src/Makefile: Move proto-generated sources to nodist_*_SOURCES.

collectd.conf(5): Briefly document grpc's SSL options.

collectd.conf: Update grpc sample configuration.

grpc plugin: Add options to enable SSL protected connections.

grpc plugin: Use std::vector to store listener configs.

grpc plugin: Replace the ListValues RPC with QueryValues.

This is a more powerful function to query values from collectd's cache.

grpc plugin: Split out identifier into it's own message type.

grpc plugin: Use plugin_thread_create instead of pthread_create.

grpc plugin: Always populate *instance fields in a value-list.

That is, ensure a (null-terminated!) empty string if the field is empty.