Hi Aurélien, On Wed, Apr 28, 2010 at 10:39:42PM +0200, Aurélien Reynaud wrote: > - Solaris 8 doesn’t have setenv(), which is required for the exec > plugin. You must use putenv(), which is roughly equivalent and > available on systems where setenv() isn't yet.
thanks for the catch, I have applied your patch to the collectd-4.9 branch, since setenv(3) is not used in 4.8 yet. > - Solaris 8 has strtok_r(), but does not declare it by default. You > must at compile time define __EXTENSIONS__, or _REENTRANT or > _POSIX_SOURCE >= 199506L. Running ./configure --enable-standards may > solves this but raises a ton of other problems... Actually, there is a configure check specifically for Solaris / this problem. It checks whether or not "_REENTRANT" is required for strtok_r(3). If you look for "strtok_r" in "configure.in" you should easily find it. I'd rather fix that check then setting "__EXTENSIONS__" when on Solaris. Could you take a look at the file "config.log" generated by the configure script and try to find out why that check failed? > - Solaris 8 doesn't have <stdint.h>, which holds the declaration of > uint64_t-like types. Those are declared in <inttypes.h> Thanks, it's in the collectd-4.8 branch. Regards, —octo -- Florian octo Forster Hacker in training GnuPG: 0x91523C3D http://verplant.org/
signature.asc
Description: Digital signature
_______________________________________________ collectd mailing list [email protected] http://mailman.verplant.org/listinfo/collectd
