[Nagiosplug-devel] [ nagiosplug-Bugs-1480574 ] check_disk is missing -lm on Solaris (rel. 1.4.3)

[SourceForge.net]

Bugs item #1480574, was opened at 2006-05-02 19:03
Message generated for change (Comment added) made by tonvoon
You can respond by visiting: 
https://sourceforge.net/tracker/?func=detail&atid=397597&aid=1480574&group_id=29880

Please note that this message will contain a full copy of the comment thread,
including the initial issue submission, for this request,
not just the latest update.
Category: Compilation
Group: Release (specify)
>Status: Pending
Resolution: None
Priority: 5
Submitted By: Ralph Rößner (ralph_roessner)
>Assigned to: Ton Voon (tonvoon)
Summary: check_disk is missing -lm on Solaris (rel. 1.4.3)

Initial Comment:
The common.h include defines, on Solaris only, a
replacement definition for floorf(), which Solaris (up
to and including version 8) does not have. Thus, on
Solaris only, all plugins including common.h need a
definition of floor() (on Linux that would be necessary
only for those plugins that actually use floorf()),
i.e. they need to link libm.so . The Makefile.in does
not respect that need, and consequently compiling
check_disk on Solaris fails. I would expect all other
plugins that include common.h to have the same problem,
but strangely that appears not to be the case.

For my needs it was sufficient to add MATHLIBS to the
dependency line for check_disk in the
plugins/Makefile.in, and I include a patch that does
exactly that. However, this should probably be resolved
more generally.

Regards,
   Ralph Rößner

----------------------------------------------------------------------

>Comment By: Ton Voon (tonvoon)
Date: 2006-10-19 20:47

Message:
Logged In: YES 
user_id=664364

Ralph,

Can you please try the snapshot at http://nagiosplug.sf.net/snapshot. The 
check_disk code has been re-written without the use of floorf anymore.

Ton

----------------------------------------------------------------------

You can respond by visiting: 
https://sourceforge.net/tracker/?func=detail&atid=397597&aid=1480574&group_id=29880