[collectd] Hi, and solaris 8 compilation problems

[Florian Forster]

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/
-------------- next part --------------
A non-text attachment was scrubbed...
Name: not available
Type: application/pgp-signature
Size: 189 bytes
Desc: Digital signature
Url : http://mailman.verplant.org/pipermail/collectd/attachments/20100429/a035acd0/attachment.pgp