tokkee.org

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

onewire plugin: fix one more compiler warning

comparison of integers of different signs: 'int' and 'unsigned long'

IPC: hopefully fix compiling under AIX

This code couldn't possibly have compiled.

Found by cppcheck:
[src/ipc.c:285]: (error) Invalid number of character '{' when these
macros are defined: 'KERNEL_AIX'.

onewire plugin: fix two compiler warnings

onewire.c:418:31: warning: comparison of integers of different signs:
'int' and 'unsigned long' [-Wsign-compare]
  if ((status > 0) && (status < sizeof (subpath)))
                       ~~~~~~ ^ ~~~~~~~~~~~~~~~~
onewire.c:422:31: warning: comparison of integers of different signs:
'int' and 'unsigned long' [-Wsign-compare]
  if ((status > 0) && (status < sizeof (subpath)))
                       ~~~~~~ ^ ~~~~~~~~~~~~~~~~

utils_subst_test.c: constify

ceph_test.c: constify

battery plugin: constify

Fix some typos in the collectd.conf manpage

Fix some typos in collectd.conf

Revert "Fix building with older versions of libtool"

While this fixed building from git on EL5, it broke
building from tarballs. Since the latter is more important,
revert this until I can get to the bottom of it.

This reverts commit 5a911ceb65165a4293830503afd257a9bd474bc7.

Revert "Turbostat: add missing types in configure.ac"

These autoconf tests were introduced in autoconf 2.60.
We still support older versions.

We've assumed for a while now that any platform we build
on has the types and functionality tested here available, and supports
a reasonable subset of the C99 and Posix standards.

This fixes building from git on RHEL5.

This reverts commit 45687863426b21f854711248258a13c345371843.

Fix building with older versions of libtool

The libtool on RHEL5 doesn't set LTDEPS, therefore
we failed to compile the builtin ltdl lib.

It does however set LIBLTDL to the path of the if it is
going to build an internal one. Use that.

Fixes building from git on RHEL5

build.sh: explicitly check for bison

We don't work with other yaccs.

python plugin: fix some typos in doc strings

curl_json plugin: zero-initialize struct correctly

Seen on FreeBSD:
curl_json.c: In function 'cj_sock_perform':
curl_json.c:864:10: warning: missing initializer for field 'sun_len' of 'struct sockaddr_un' [-Wmissing-field-initializers]
   struct sockaddr_un sa_unix = {};
          ^
In file included from curl_json.c:32:0:
/usr/include/sys/un.h:48:16: note: 'sun_len' declared here
  unsigned char sun_len; /* sockaddr len including null */
                ^

virt plugin: missed one old-style definition

Travis: build python plugin

The Travis image has various python versions installed in /opt,
none of which have development headers, and they are first in
the PATH. Explicitly use the python in /usr/bin instead.

Travis: build iptables plugin

Merge branch 'collectd-5.5'

Merge branch 'collectd-5.4' into collectd-5.5

virt plugin: fix old-style function definitions

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

Revert "perl plugin: mark file variable as const"

It breaks the build on RHEL5

This reverts commit 43d7245c20930161284e5784fbf115d1221a5951.

cu_mount_getoptionvalue: mark keyword as const

tests: fix old-style function definition

sigrok plugin: constify

processes plugin: fix one more old-style prototype

swap plugin: fix building on OpenBSD

configure.ac: fix comment

perl plugin: mark file variable as const

collectdmon: mark pidfile as const

collectdctl: mark as exit_usage as noreturn

zfs_arc plugin: constify

tokyotyrant plugin: constify

smart plugin: constify

utils_rrdcreate.c: constify

vserver plugin: constify

Mark functions that exit as noreturn

configfile.c: constify

postgresql plugin: constify

parser.y: mark text as const

cu_mount_checkoption: mark keyword as const

nfs plugin: constify

netlink plugin: constify

ipvs plugin: constify

memory plugin: constify

mysql plugin: fix indentation

mysql plugin: constify

nginx plugin: constify

ntpd plugin: constify

write_tsdb plugin: constify

openvpn plugin: constify

unixsock plugin: constify

plugin.c: constify

syslog plugin: constify

email plugin: constify

ted plugin: constify

zookeeper plugin: constify

memcached plugin: constify

mbmon plugin: constify

ascent plugin: constify

apcups plugin: constify

Merge pull request #1602 from rubenk/powerdns-constify

powerdns: constify some stuff

Merge pull request #1600 from rubenk/ceph-make-some-variables-static

ceph plugin: mark variable as static and const

Merge pull request #1579 from rubenk/fix-typo-in-comment

Fix a typo in a comment

Merge pull request #1601 from rubenk/thermal-make-some-variables-static

thermal plugin: mark some variables as static

Merge pull request #1603 from rubenk/tcpconns-make-variable-static

tcpconns plugin: mark variable as static

Merge pull request #1604 from rubenk/sensors-make-variable-static

sensors plugin: mark variable as static

Merge pull request #1605 from rubenk/tail-mark-variables-as-static

tail plugin: mark a few variables as static

Merge pull request #1606 from rubenk/utils-dns-remove-unused-code

utils-dns: remove unused code

utils-dns: remove unused code

tail plugin: mark a few variables as static

sensors plugin: mark variable as static

tcpconns plugin: mark variable as static

ceph plugin: mark variable as static and const

thermal plugin: mark some variables as static

powerdns: constify some stuff

make some variables static while we're at it

Fix the remainging -Wshadow issues on master

Merge branch 'collectd-5.5'

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.5'

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.5'

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)

Merge branch 'collectd-5.5'

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));
                    ^~