Merge branch 'collectd-5.4' into collectd-5.5

Conflicts:
src/Makefile.am

rrdtool plugin: fix thread-safety detection

librrd 1.6.0 is now threadsafe and librrd_th is gone.

Since there doesn't seem to be a way to detect that librrd
is threadsafe, use pkg-config to check for 1.6.0 or newer instead.

The logic is now as follows:
Check for librrd >= 1.6.0 with pkg-config
If not found, look for rrd_update_r in librrd_th.
If not found, look for rrd_update in librrd and assume librrd is not
thread safe.

(cherry picked from commit ef43260cd901847220c2a9de400579ff903ca13e)

Conflicts:
src/Makefile.am

rrdtool check: stop linking in libm

(cherry picked from commit 32c0ce39f786c56e9d13f3615253a7ae55b578e6)

rrdtool check: no need to save CPPFLAGS

There's no need to save CPPFLAGS when we're just
checking a library.

(cherry picked from commit f7e09269879b74e61324de9178503221b2df9136)

rrdtool check: no need to save LDFLAGS

There's no need to save LDFLAGS when we're just
checking a header.

(cherry picked from commit 38068ecc7922ace29bb6af3f8ee5568c3fe96c08)

statsd plugin: Fix deadlock on plugin shutdown (Issue #1703)

More missing kstat usages

Add missing kstat definitions

Take out explicit definitions as they are already checked via enable_standards in configure.ac.

Fixes: #978

collectd-tg: Move _POSIX_C_SOURCE and friends to the Makefile.

Apparently defining this in the source files confuses Solaris' libc.
Thanks to @dago for looking into this.

Hopefully fixes: #978

processes: remove initialisation of unexisting variables

Partial revert of 1f0156c, which broke the build on Solaris and OpenBSD.
The context-switch reporting code is only part of master at the moment.

Fixes: #1674

Merge pull request #1546 from mfournier/processname_length_1284

processes: warn about process names above OS limit

fix potential curl plugin curl_slist_append memory leak

make sure that we dont leak memory if curl_slist_append returns null

Merge branch 'collectd-5.4' into collectd-5.5

Fix process plugin memory leak [solaris]

Merge pull request #1627 from rubenk/remove-old-filter-syntax-from-manpage

Remove broken filter syntax from collectd.conf(5)

Bump version to latest 5.5.1 #1536
(cherry picked from commit 238419ac27bcd991cebd021d25706b78d7edd182)

Merge branch 'collectd-5.4' into collectd-5.5

configure.ac: break long line and sort

This will make future merges easier.

Merge branch 'collectd-5.4' into collectd-5.5

Fix building with xfsprogs 4.5.0

xfsprogs 4.5.0 started to use off64_t in its headers,
which is hidden behind _GNU_SOURCE

Fixes #1637

mysql plugin: remove unused declaration

This was left behind after commit c17582c

Remove broken filter syntax from collectd.conf(5)

fb67910d documented the new syntax.
The old syntax does no longer work, remove it.

Fix conflict markers

Merge branch 'collectd-5.4' into collectd-5.5

Fix indentation in previous commit

Merge pull request #1620 from skinowski/write_http_mem_leak_II

write_http: in cleanup handler free http headers

write_http: in cleanup handler free http headers

slist of http headers require cleanup to avoid
memory leak. See libcurl example:

https://curl.haxx.se/libcurl/c/httpcustomheader.html

Merge branch 'collectd-5.4' into collectd-5.5

Fix build on Solaris 10 i386 part 2

utils_latency.c needs the same fix as applied in a41f312

(cherry picked from commit e7f7ecc1cac73841cdebd7e0462af4989532a107)

Fix build on Solaris 10 i386

AC_SYS_LARGEFILE defines _FILE_OFFSET_BITS in src/config.h,
so we need this header first before including system headers.

Thanks to @dago for the report.

Fixes:
  CC       collectd-utils_random.o
In file included from collectd.h:31:0,
                 from utils_random.c:29:
../../src/config.h:1646:0: error: "_FILE_OFFSET_BITS" redefined
[-Werror]
 #define _FILE_OFFSET_BITS 64
 ^
In file included from
/opt/csw/lib/gcc/i386-pc-solaris2.10/5.2.0/include-fixed/pthread.h:20:0,
                 from utils_random.c:27:
/opt/csw/lib/gcc/i386-pc-solaris2.10/5.2.0/include-fixed/sys/feature_tests.h:196:0:
note: this is the location of the previous definition
 #define _FILE_OFFSET_BITS 32
 ^
cc1: all warnings being treated as errors

(cherry picked from commit a41f312a8d48a989dfba00a8b43532c60f67caba)

Merge branch 'collectd-5.4' into collectd-5.5

 RPM specfile: fix xmms plugin build condition

Merge branch 'collectd-5.4' into collectd-5.5

RPM specfile: update changelog

RPM specfile: enable xmms plugin on EL7

Merge branch 'collectd-5.4' into collectd-5.5

RPM specfile: re-enable memcachec plugin on EL7

cf https://bugzilla.redhat.com/show_bug.cgi?id=1116949

RPM specfile: enable modbus plugin on EL7

Merge pull request #1617 from michaelsalmon/putval-time-exec-pod

added definition of N to collectd-exec

added definition of N to collectd-exec

Merge branch 'collectd-5.4' into collectd-5.5

Merge pull request #1575 from rubenk/dragonflybsd-utils-mount

Fix build on DragonFlyBSD

Merge pull request #1580 from rubenk/freebsd-zfs-arc

zfs_arc: 'stolen' was removed after FreeBSD 10.2

Merge branch 'collectd-5.4' into collectd-5.5

Merge pull request #1597 from rubenk/turbostat-fix-prototypes

Turbostat: fix a few old-style prototypes

Merge pull request #1595 from rubenk/turbostat-fix-shadowing-issue

Turbostat: fix shadowing issue

Merge pull request #1594 from rubenk/ceph-fix-shadowing-issue

Ceph: fix shadowing issue

Merge pull request #1596 from rubenk/fix-a-few-more-prototypes

Fix a few more prototypes

Turbostat: fix a few old-style prototypes

Fix a few more prototypes

Found with -Wstrict-prototypes and -Wold-style-definition

Merge branch 'collectd-5.4' into collectd-5.5

Turbostat: fix shadowing issue

turbostat.c: In function ‘delta_thread’:
turbostat.c:460:26: error: declaration of ‘core_delta’ shadows a global
declaration [-Werror=shadow]
  const struct core_data *core_delta)
                          ^~~~~~~~~~
turbostat.c:163:4: note: shadowed declaration is here
 } *core_delta, *core_even, *core_odd;
    ^~~~~~~~~~

Ceph: fix shadowing issue

Merge pull request #1584 from rubenk/fix-wrong-logical-op

utils_avltree.c: fix compiler warning

Merge branch 'collectd-5.4' into collectd-5.5

Merge branch 'Wshadow' into collectd-5.4

Define _DEFAULT_SOURCE in addition to _BSD_SOURCE

This enables forward compatibility with the ongoing
deprecation of _BSD_SOURCE.

(cherry picked from commit 3bc1a46bebfa53ec0f0e12d6406ca126a3ad6bf3)

Fix memory leak in redis.c

Fix a few issues found by -Wshadow

utils_avltree.c: fix compiler warning

make[3]: Entering directory '/home/ruben/src/collectd/src/daemon'
  CC       utils_avltree.lo
utils_avltree.c: In function ‘rebalance’:
utils_avltree.c:248:20: warning: logical ‘or’ of collectively exhaustive
tests is always true [-Wlogical-op]
    assert ((b_bottom >= -1) || (b_bottom <= 1));
                    ^~
utils_avltree.c:258:20: warning: logical ‘or’ of collectively exhaustive
tests is always true [-Wlogical-op]
    assert ((b_bottom >= -1) || (b_bottom <= 1));
                    ^~

write_sensu: define _GNU_SOURCE for asprintf

On Linux, asprintf and vasprintf are only available when _GNU_SOURCE is defined.

write_sensu: remove custom asprintf implementation

This was guarded by HAVE_ASPRINTF, but we never had a configure check
for asprintf (or vasprintf).

This only turned up when I made the function static, since they now
clash with the prototypes in stdio.h on FreeBSD.

As far as I know all the platforms we build on have these functions.
If this turns out to be not the case, we can always add compat functions
to our utilies later.

For now, this fixes the build on FreeBSD.

write_riemann: add new header to Makefile.am

Fixes 'make distcheck'

write_sensu: mark asprintf as static

Commit straigh to 5.5. This plugin didn't exist in 5.4.

write_sensu: mark vasprintf as static

Commit straight to 5.5. This plugin didn't exist in 5.4

write_riemann: move prototype to header file

redis: mark redis_handle_query as static

Commit straight to 5.5. This plugin didn't exist in 5.4.

redis: mark redis_handle_info as static

Commit straight to 5.5. This plugin didn't exist in 5.4.

barometer: mark Detect_sensor_type as static

Make function start with lowercase 'd' while we're at it.
Commit straight to 5.5.

collectd.c: mark notify_systemd as static

Commit straight to 5.5. This function didn't exist in 5.4.

collectd.c: mark notify_upstart as static

Commit straight to 5.5. This function didn't exist in 5.4.

sensu: mark replace_str as static

Commit straight to 5.5. This plugin doesn't exist in 5.4

sensu: mark sensu_format_name2 as static

Commit straight to 5.5, this plugin doesn't exist in 5.4

Fix prototype of uc_get_size

Commit straight to 5.5, this function doesn't exist in 5.4.

Fix prototype of plugin_log_available_writers

Commit straight to 5.5, this function doesn't exist in 5.4

Merge branch 'collectd-5.4' into collectd-5.5

Merge pull request #1582 from rubenk/missing-prototypes

Missing prototypes

xmms: mark cxmms_read as static

sensors: mark sensors_free_features as static

processes: mark ps_read_process as static

utils_db_query.c: make udb_query_free_one static

battery: mark dict_get_double as static

types_list.c: add header for prototypes

utils_subst.c: add header for prototypes

utils_random.c: add header for prototypes

utils_cmd_getval.c: add header for prototypes

utils_cmd_flush.c: add header for prototypes

threshold.c: mark ut_config as static

latency_counter_create: fix signature

utils_latency.c: mark change_bin_width as static

utils_cmd_putval.c: include header for prototypes

Include header for prototype of handle_putnotif

Add prototype for module_register

liboconfig: mark oconfig_free_all as static

zfs_arc: 'stolen' was removed after FreeBSD 10.2

Patch taken from FreeBSD port

Fix build on DragonFlyBSD

DragonFlyBSD has both vfsstat and fsstat apis.
It lacks ST_NOWAIT however.

The easiest fix is to swap the checks and use getfsstat first.

Merge branch 'collectd-5.4' into collectd-5.5

processes: fix pid format string

Follow up to 7c495a5.

Merge branch 'collectd-5.4' into collectd-5.5