Gentoo Websites Logo
Go to: Gentoo Home Documentation Forums Lists Bugs Planet Store Wiki Get Gentoo!
Bug 361723 - app-admin/collectd: ebuild&init script improvements
Summary: app-admin/collectd: ebuild&init script improvements
Status: RESOLVED FIXED
Alias: None
Product: Gentoo Linux
Classification: Unclassified
Component: [OLD] Unspecified (show other bugs)
Hardware: All Linux
: Normal normal (vote)
Assignee: Andreas K. Hüttel
URL:
Whiteboard:
Keywords:
Depends on:
Blocks:
 
Reported: 2011-04-02 22:51 UTC by Bruno
Modified: 2011-04-24 17:11 UTC (History)
0 users

See Also:
Package list:
Runtime testing required: ---


Attachments
Patch providing suggested improvements (collectd-ebuild.patch,6.44 KB, patch)
2011-04-02 22:51 UTC, Bruno
Details | Diff

Note You need to log in before you can comment on or make changes to this bug.
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!