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

Merged branch 'collectd-4.10' into collectd-5.4.

perl plugin: Be more consistent about capitalization.

perl plugin: Removed commented code

perl plugin: Added call_pv_locked() wrapper to handle thread locking flags.

perl plugin: Switched to PTHREAD_MUTEX_RECURSIVE locking

perl plugin: Fixes for #1706
* Fix coredump due to destroying interpreter on threads running perl.
* Fix deadlock when perl_log() is called from perl_init()

perl plugin: lock base thread interpreter in perl_init() too.
Avoid race conditions with c_ithread_create() called from threads of already-initialized plugins.
Fix for https://github.com/collectd/collectd/issues/1706.

Merge pull request #1710 from rpv-tomsk/perl-plugin-fixes

perl plugin: Synchronize access to thread information.

Cf. #1706

perl plugin: Removed commented code

perl plugin: Added call_pv_locked() wrapper to handle thread locking flags.

perl plugin: Switched to PTHREAD_MUTEX_RECURSIVE locking

perl plugin: Fixes for #1706
* Fix coredump due to destroying interpreter on threads running perl.
* Fix deadlock when perl_log() is called from perl_init()

perl plugin: lock base thread interpreter in perl_init() too.
Avoid race conditions with c_ithread_create() called from threads of already-initialized plugins.
Fix for https://github.com/collectd/collectd/issues/1706.

Merge pull request #1708 from rpv-tomsk/plugin-postgresql-plugininstancefrom-public

Postgresql plugin: set plugin instance from query results.

utils_db_query (postgresql/dbi/oracle plugins): Documented PluginInstanceFrom option

Merge branch 'collectd-5.5'

plugin_mock.c: Declare "kc".

This hopefully fixes "make check" on Solaris, which currently fails
with:

  CC       common_test.o
  CCLD     test_common
Undefined first referenced
 symbol       in file
kc                                  ./.libs/libcommon.a(common.o)
ld: fatal: symbol referencing errors. No output written to test_common
collect2: error: ld returned 1 exit status

(cherry picked from commit 4fb43017b4b585256627b9503ce0a5cf08ecdefe)

Do not link test_plugin_ceph with libcommon

Linking with libplugin_mock is enough since
that one already links with libcommon

Set subdir-objects globally instead of per Makefile

Makefile.am: Link tests with required libraries, too.

"make check" fails on Solaris because libraries are missing:

      CCLD     test_common
    Undefined first referenced
     symbol   in file
    kc                          ./.libs/libcommon.a(common.o)
    kstat_data_lookup           ./.libs/libcommon.a(common.o)
    kstat_lookup                ./.libs/libcommon.a(common.o)
    kstat_read                  ./.libs/libcommon.a(common.o)
    getaddrinfo                 ./.libs/libcommon.a(common.o)
    freeaddrinfo                ./.libs/libcommon.a(common.o)
    gai_strerror                ./.libs/libcommon.a(common.o)
    ld: fatal: symbol referencing errors. No output written to test_common
    collect2: error: ld returned 1 exit status

(cherry picked from commit 6213557b1d5c79a60e9c4322263836ca6066aa0e)

build: add libavltree, libcommon & libheap dependencies

Otherwise it can break on very parallel builds since collectd link time
arrives before one or more of these were built.

Signed-off-by: Gustavo Zacarias <gustavo@zacarias.com.ar>
(cherry picked from commit 780e6a76021a240e95007a04b723d827120afa95)

Fix a typo that snuck in with 0c8d275

Always test before committing...

Pointy hat to: me

Link libcommon.la with $(COMMON_LIBS).

This hopefully fixes link errors on Solaris:

  CCLD     test_utils_vl_lookup
Undefined first referenced
 symbol       in file
kstat_data_lookup                   daemon/.libs/libcommon.a(common.o)
kstat_lookup                        daemon/.libs/libcommon.a(common.o)
kstat_read                          daemon/.libs/libcommon.a(common.o)
getaddrinfo                         daemon/.libs/libcommon.a(common.o)
freeaddrinfo                        daemon/.libs/libcommon.a(common.o)
gai_strerror                        daemon/.libs/libcommon.a(common.o)
ld: fatal: symbol referencing errors. No output written to test_utils_vl_lookup
collect2: error: ld returned 1 exit status

(cherry picked from commit fe2eee98f4b1cf8c5a5df6ed6cd03d8487c695be)

Conflicts:
src/daemon/Makefile.am

Merge branch 'collectd-5.5'

Conflicts:
configure.ac

Merge branch 'collectd-5.4' into collectd-5.5

Merge pull request #1726 from rubenk/udev

Udev

processes: re-add <sys/user.h> inclusion

cf #1725

Stop hardcoding libudev

Add missing udev flags to Makefile.am

Merge branch 'collectd-5.4' into collectd-5.5

processes: MAXCOMLEN is not exposed on Solaris

MAXCOMLEN in <sys/user.h> is only exposed to kernel
code. I think it's safe to assume that it isn't going
to change, so just hardcode it.

Merge pull request #1724 from rpv-tomsk/smart-plugin-udev-dep

Fix for #1688 The SMART plugin does not build/load, v.5.5.0

smart plugin: Check for libudev presence

Merge pull request #1723 from rpv-tomsk/disk-plugin

Disk plugin: UdevNameAttr feature improvements

disk plugin: Initialize udev context only once, not per each read

disk plugin: Call disk_udev_attr_name() only when UdevNameAttr configured

collectd.conf.in: remove extraneous pound sign

Merge branch 'collectd-5.5'

Merge branch 'collectd-5.4' into collectd-5.5

utils_cache.c: fix FORMAT_VL failed ERROR messages

kill correct pid on fdopen failure