Gentoo Websites Logo
Go to: Gentoo Home Documentation Forums Lists Bugs Planet Store Wiki Get Gentoo!

Bug 361723

Summary: app-admin/collectd: ebuild&init script improvements
Product: Gentoo Linux Reporter: Bruno <bonbons>
Component: [OLD] UnspecifiedAssignee: Andreas K. Hüttel <dilfridge>
Status: RESOLVED FIXED    
Severity: normal    
Priority: Normal    
Version: unspecified   
Hardware: All   
OS: Linux   
Whiteboard:
Package list:
Runtime testing required: ---
Attachments: Patch providing suggested improvements

Description Bruno 2011-04-02 22:51:29 UTC
Created attachment 268263 [details, diff]
Patch providing suggested improvements

Some improvement suggestions for collectd ebuild and init script:
- let collectd run as a non-root user by default
- move pidfile and unixsock socket into /var/run/collectd/
- add perl use-flag to enable/disable perl bindings (and dependency on perl)
- add static-libs useflags to control generation of libcollectdclient.a but
  always delete plugin .a files
- patch for bind plugin to parse timestamp of statistics according to UTC
  as sent by bind - can be applied for Linux but apparently non-linux systems
  are missing timegm() ... [should get properly fixed upsteam]

Note: from the patch the line:
  use collectd_plugins_ping && setcap cap_net_raw+ep ${D}/usr/sbin/collectd
can probably not be applied to tree until there is proper support for handling file-based capabilities, but that's what I do on my side to run with ping plugin.
Comment 1 Andreas K. Hüttel archtester gentoo-dev 2011-04-24 17:11:03 UTC
All added except the timestamp patch, please push that upstream. Thanks!