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

Bug 351265

Summary: app-admin/collectd-4.10.2 perl plugin default config
Product: Gentoo Linux Reporter: Daniel Robbins <drobbins>
Component: [OLD] ServerAssignee: Andreas K. Hüttel <dilfridge>
Status: RESOLVED FIXED    
Severity: normal CC: jmbsvicetto
Priority: High    
Version: unspecified   
Hardware: All   
OS: Linux   
Whiteboard:
Package list:
Runtime testing required: ---

Description Daniel Robbins 2011-01-10 01:31:32 UTC
Hi there,

collectd-4.10.2+ has new syntax for loading the perl plugin, which works around a serious bug related to dynamic loading that existed in previous versions, documented here:

http://mailman.verplant.org/pipermail/collectd/2008-March/001616.html 

However, this new syntax is buried in a man page. The collectd configuration file generated by default does not include the correct syntax. This situation can be improved (a work-around, until upstream addresses it better) by adding the following as the last line to src_install():

 sed -i -e 's:^.*LoadPlugin perl$:# The new, correct way to load the perl plugin -- \n# <LoadPlugin perl>\n#   Globals true\n# </LoadPlugin>:' ${D}/etc/collectd.conf || die "sed fix fail"

I have not experimented too much with the python plugin, but according to the man page, it seems like this syntax may fix some dynamic loading issues with the python plugin as well. Spent several hours trying to get the perl plugin working until I discovered this new config option inside a git commit. I'm including this in the Funtoo Linux temp. fork of collectd.

Reproducible: Always

Steps to Reproduce:
Using the old configuration option will prevent any Perl plugin from loading symbols correctly, so it will fail. Using the correct configuration option makes things work.
Comment 1 Jorge Manuel B. S. Vicetto (RETIRED) Gentoo Infrastructure gentoo-dev 2011-01-10 05:18:17 UTC
Sorry, wrong field.
Comment 2 Andreas K. Hüttel archtester gentoo-dev 2011-01-28 21:34:52 UTC
Thanks, I've added this for both perl and python to the -r2 ebuild.