Ticket #344 (closed defect: wontfix)
neo doesn't build on Jaunty
Reported by: | jdreed | Owned by: | |
---|---|---|---|
Priority: | normal | Milestone: | Upstream Utopia |
Component: | -- | Keywords: | |
Cc: | Fixed in version: | ||
Upstream bug: |
Description
In its sleeping code, neo uses the obsolete CLK_TCK constant. /usr/include/time.h and time.h(7posix) claim that CLOCKS_PER_SEC should be used instead, but go on to warn that CLOCKS_PER_SEC should not be assumed to be a compile time constant. CLOCKS_PER_SEC is required to be one million (1000000l) on XSI-compliant systems, and indeed is on both RHEL4 and Ubuntu. However, on RHEL4, CLK_TCK is 100, and is undefined on Ubuntu. Defining CLK_TCK as CLOCKS_PER_SEC makes neo sleep for significantly longer (1000 times longer, in fact) than it should between SNMP queries when getting device stats. Defining CLK_TCK as 100 allows neo to function correctly, but this is almost certainly the wrong answer.
I've reported this "upstream" to bug-ktools, but can't even reference a transaction number in the discuss meeting because it's private. Yay.
Bug report resent to kretch@… at kchen's suggestion.