tokkee.org

build system (Perl bindings): Make Makefile.PL's PREFIX/INSTALL_BASE optional.

Both options may cause problems:

 - PREFIX might clash with INSTALL_BASE being set in $PERL_MM_OPT.
 - INSTALL_BASE without any further options set will install into non-standard
   directories in many cases.

Rather, use PREFIX=${prefix} as default but overwrite that if
--with-perl-bindings=... has been used.

Thanks to faxm0dem for reporting the INSTALL_BASE problems on IRC and
bvarner and nirik for reporting this in GH #177 and #196.

This is a follow-up / fix for 97e8f346a88eccf4d1608c21d5cc6ee9620c1c41.

Fixes Github issue #290.

Signed-off-by: Florian Forster <octo@collectd.org>

Merge branch 'collectd-5.1' into collectd-5.2

src/libcollectdclient/network_buffer.c check for htonll exitence
before defining it. I have to to this to get it compiled on AIX6

Signed-off-by: Florian Forster <octo@collectd.org>

configure.in: Yet another attempt at fixing the mess that is AC_LANG_PROGRAM().

Merge branch 'collectd-5.1' into collectd-5.2

configure.in: Quote AC_LANG_PROGRAM and remove AC_INCLUDES_DEFAULT.

The latter isn't expanded anymore when quoted. There's no easy way
around this :\

Merge branch 'collectd-5.1' into collectd-5.2

Merge branch 'collectd-4.10' into collectd-5.1

Conflicts:
src/curl_json.c

curl_xml plugin: Check for a curl_easy_perform() error first

The value of CURLINFO_RESPONSE_CODE isn't valid otherwise.
Also, use the symbolic name CURLE_OK in all plugins where
appropriate.

Change-Id: I17ae9c7eea393ee4641296b5484c93809a662dd9
Signed-off-by: Florian Forster <octo@collectd.org>

Fix memory leaks when plugin gets no results

Signed-off-by: Florian Forster <octo@collectd.org>

exec plugin: Fix a copy'n'paste mistake in a comment.

Thanks to Robert Riches Jr for the hint.

Add upstart configuration file for Ubuntu, Debian and RedHat distros

Signed-off-by: Florian Forster <octo@collectd.org>

Merge branch 'mf/redhat' into collectd-5.2

Fixes Github issue #220.

curl_json plugin: Call yajl_complete_parse() / yajl_parse_complete() only once.

The cURL callback may be called several times, so we need to call the
complete() function after perform() returns.

Add protection from infinite redirect loops to curl-using plugins

Signed-off-by: Florian Forster <octo@collectd.org>

curl_xml plugin: Fix potential dereference of NULL pointer.

I managed to get collectd to segfault in a couple of places while
playing with it a bit. The first is in the curl_xml module when the
XPATH expression doesn't quite match the input. The crash occurs on line
407 when instance_node->nodeTab[0] is dereferenced. At this point, all
members of instance_node are 0, so dereferencing the array isn't a good
idea. This patch fixes the problem, although I'm not sure if this
particular case actually deserves its own error message.

Signed-off-by: Florian Forster <octo@collectd.org>

src/plugin.c: Initialize "next_read" field of read callbacks.

Fixes Github issue #230.

src/Makefile.am: Only add path to lcc_features.h to binaries linked against libcollectdclient.

Allow out-of-tree builds

The generated header file lcc_features.h and collectd.h cause problems
otherwise.

Signed-off-by: Florian Forster <octo@collectd.org>

Merge branch 'collectd-5.1' into collectd-5.2

Merge branch 'collectd-4.10' into collectd-5.1

Conflicts:
src/curl_json.c

curl's numeric options are always at minimum long, never int

This can affect portability to some architectures.

Signed-off-by: Florian Forster <octo@collectd.org>

postgresql plugin: Fix cast (was breaking aliasing rules).

postgresql.c: In function 'c_psql_flush':
postgresql.c:974: warning: dereferencing type-punned pointer will break strict-aliasing rules

pf plugin: Fix checking for <net/pfvar.h>.

The header is not self-contained, so we need to include appropriate
headers in the configure check as well.

pf plugin: Define {F,S}CNT_NAMES.

Apparently they are only defined in the pfctl sources, not in the
system's header files for some reason.

pf plugin: Fix a typo.

s/status/state/g

cpu plugin: Remove broken debug message.

The message was uninformative and did cast a pointer to an int,
resulting in a compile error.

Merge remote-tracking branch 'github/collectd-5.2' into collectd-5.2

ChangeLog: Fix typo.

ChangeLog: Fixed version number for 5.2.1 ;-)

src/utils_tail.c: Remove multiple newlines at EOL.

Updated ChangeLog.

Merge remote-tracking branch 'origin/collectd-5.2' into collectd-5.2

Bumped version to 5.2.1; Updated ChangeLog.

Merge branch 'collectd-5.1' into collectd-5.2

Conflicts:
ChangeLog
src/write_graphite.c
version-gen.sh

Bumped version to 5.1.2; Updated ChangeLog.

Added "tar-pax" (bugfix for long filenames with make distcheck)

Signed-off-by: Florian Forster <octo@collectd.org>

src/Makefile.am: Fix "make distcheck".

protoc-c has a weird handling of files not in the CWD: They have to be
specified with the "-I" option _and_ you need to specify the full path
to the file.

bugfix ("cannot find collectd.h") for branch master (and 5.2)

Signed-off-by: Florian Forster <octo@collectd.org>

make distcheck : perllocal.pod was missing. Fixed

Signed-off-by: Florian Forster <octo@collectd.org>

Fix make distcheck (against release 5.1.1)

Signed-off-by: Florian Forster <octo@collectd.org>

logfile plugin: Make "do_close" a boolean.

If the file isn't closed such as when logging to stdout or stderr the file must be flushed for it to get out of the IO buffers right away.

Signed-off-by: Florian Forster <octo@collectd.org>

postgresql plugin: Don't access realloc'ed memory.

D'oh! This is the most stupidest bug: all database connection objects are
stored in a plugin-global array and a pointer to each object was passed around
to the worker functions. This array used to be an array of objects (rather
than pointers) and was realloc'ed every time a new connection was added. Now,
the realloc caused the old pointer to not be valid any more, thus, any but the
last database connection were later accessing memory out of range. This caused
the plugin to segfault if more than one database connection was used.

Thanks to Greg Swift for reporting this!
Fixes Github issue #238.

src/utils_dns.c: Improve ordering of includes.

sys, followed by net, followed by netinet, followed by arpa
that should be the most logical order

Signed-off-by: Florian Forster <octo@collectd.org>

write_graphite plugin: Remove two more redundant error messages.

The failing functions already print an error message; don't spam the logs
by duplicating this.

Further fixes Github issue #236.

src/utils_complain.[ch]: Fix the complaint mechanism after the sub-second change.

write_graphite: Use the complain mechanism to report connection errors.

Fixes Github issue #236. Thanks to @pyr for reporting it!

Merge remote-tracking branch 'origin/collectd-5.2' into collectd-5.2

write_graphite: Decrease a buffer size.

If the buffer is too large, it is possible that the message cannot be
send using the send buffer, leading to a failed assertion in
wg_send_message().

Fixes Github issue #226. Thanks to @bwhaley for reporting this.

Merge branch 'collectd-5.1' into collectd-5.2

Merge branch 'collectd-4.10' into collectd-5.1

dbi plugin: Replace cdbi_config_set_string() with cf_util_get_string().

Fixed a small typo in the ethstat plugin

Signed-off-by: Florian Forster <octo@collectd.org>

ethstat plugin: Fix the map lookup.

Previously, a pointer into the configuration structure was used to look
up the mappings. Because the configuration structure is freed or
otherwise modified, this does not work as intended (and may actually
lead to a segmentation fault when unlucky).

For completeness sake, a shutdown callback was added to free the used
memory on exit.

Fixes Github issue 224.

RedHat RPM spec: remove unnecessary dependency on libstatgrab

RedHat RPM spec: bump release number

RedHat RPM spec: package php-collection/collection3 independently

redhat/collectd.spec - typo.

RedHat specfile: updated for 5.2.0

Also includes a couple of cleanups mentioned in github#186.

src/types.db: Remove the maximum value from "timeleft".

format_graphite: Implement the "separate instances" and "always append ds" options.

Fixes Github issue #214.

Merge remote-tracking branch 'github/collectd-5.2' into collectd-5.2

Fix when gcrypt.h is missing (libcollectdclient)

Signed-off-by: Florian Forster <octo@collectd.org>

Fix for nanosleep missing on Solaris

Signed-off-by: Florian Forster <octo@collectd.org>

postgresql plugin: Fixed a memory leak occurring on every (successful) write.

zfs_arc plugin: Fix the type used for mutex misses.

Thanks to Yves Mettier for pointing this out!

Updating data source for collectd v5, see https://collectd.org/wiki/index.php/V4_to_v5_migration_guide

Signed-off-by: Florian Forster <octo@collectd.org>

Merge remote-tracking branch 'github/collectd-5.2' into collectd-5.2

varnish plugin: Make sure the argument passed to VSM_n_Arg() is not NULL.

Fixes Github issue 202.

Add missing "Graphite" to the amqp format list.

Signed-off-by: Florian Forster <octo@collectd.org>

collectd.conf: Added example PostgreSQL writer configuration.

collectd.conf: Added a short explanation to the 'Interval' setting.

… mentioning the per-plugin interval setting.

README: Added 'pf' and 'aggregation' plugins.

fix redis default node initialization

unbreak redis plugin build

Signed-off-by: Florian Forster <octo@collectd.org>

collectd-tg(1): Add manpage.

Allow for conditional compilation of plugins

To enable/disable a plugin you can toggle the
%define for that plugin at the top of the specfile.
You can also pass the plugin as argument to rpmbuild,
for example:
rpmbuild -ba --without rrdtool
or:
rpmbuild -ba --with rrdcached

Quote arguments

No need to edit collectd.conf anymore.

This was fixed by in commit 25fa4ad

Follow Fedora Packaging Guidelines in java subpackage

Cleanup redhat specfile

Fixed a few rpmlint warnings
Updated (mostly) to Fedora Packaging Guidelines

RedHat RPM spec: update for 5.1 + improvements

  * Enabled all buildable plugins based on libraries available on EL6 +
    EPEL
  * All plugins requiring external libraries are now shipped in seperate
    packages.
  * No longer treat Java plugin as an exception, correctly set
    $JAVA_HOME during the build process + ensure build deps are installed.
  * Dropped per-plugin configuration files, as they tend to diverge from
    upstream defaults.
  * Moved perl stuff to /usr/share/perl5/
  * Don't alter Interval and ReadThreads by default, let the user change
    this himself.
  * Drop collection.cgi from main package, as it's been obsoleted by
    collection3
  * Moved contrib/ to its own package, to avoid cluttering the main
    package with non-essential stuff.
  * Replaced BuildPrereq by BuildRequires

RedHat initscript: small improvements

  * checks configuration before (re)starting, based on debian's
    initscript
  * use /etc/sysconfig instdead of /etc/default
  * include optional $ARGS in arguments passed to collectd.

collectd.conf(5): Add the per-plugin interval to the synopsis.

Fix java detection as unprivileged user.

If you have an inaccessible directory inside
JAVA_HOME, find returns an error and the test fails.

I found this on Fedora, where /usr/lib64/audit is only
readable by root, and my JAVA_HOME was set to /usr

Signed-off-by: Florian Forster <octo@collectd.org>

Fix building of java binding

Caused by d327beed4d1541d883c4823485b381d1dff79039

Signed-off-by: Florian Forster <octo@collectd.org>

src/libcollectdclient/Makefile.am: Fix typo.

src/libcollectdclient/Makefile.am: Add "collectd/network.h".

Bump version to 5.2.0; Update ChangeLog.

src/libcollectdclient/network.c: Fix the build on FreeBSD.

On FreeBSD, we have to include <netinet/in.h> to get IN_MULTICAST(). We
don't need to include anything extra, as according to POSIX,
<netinet/in.h> also exposes htonl(). There is no need to include
<arpa/inet.h>.

Signed-off-by: Florian Forster <octo@collectd.org>

src/libcollectdclient/network_buffer.c: Fix gcrypt build on FreeBSD.

As with src/network.c, we need to ensure that we define
GCRYPT_NO_DEPRECATED on FreeBSD to get rid of the compiler warnings
emitted by the header file.

Signed-off-by: Florian Forster <octo@collectd.org>

Merge branch 'collectd-5.1'

Merge remote-tracking branch 'origin/master'

Merge branch 'collectd-5.0' into collectd-5.1

Merge branch 'collectd-4.10' into collectd-5.0

fixed paths in example configuration file

These paths previously got expanded to:
  #BaseDir "/usr/var/lib/collectd"
instead of:
  #BaseDir "/var/lib/collectd"

And on systems which put libs in /usr/lib64:
  #PluginDir   "/usr/lib/collectd"
instead of:
  #PluginDir   "/usr/lib64/collectd"

Signed-off-by: Florian Forster <octo@collectd.org>

lcc_features.h: Relicense under the MIT license.

src/libcollectdclient/network_buffer.c: Copy gcrypt import magic from the network plugin.

libcollectdclient: Update to the high-resolution time format.

Merge branch 'sr/pf'