[collectd] memcached:// support in curl plugin

[Doug MacEachern]

Hi Florian,

On Mar 19, 2009, at 1:00 AM, Florian Forster wrote:

> Hi Doug,
>
> On Mon, Mar 16, 2009 at 03:37:11PM -0700, Doug MacEachern wrote:
>>> Too bad the Debian developer didn't turn to `libmemcached' when
>>> `libmemcache' became abandoned. To date, there is not Debian package
>>> for the new `libmemcached'.
>
> apparently there is a wishlist bug report for this in Debian:
> <http://bugs.debian.org/cgi-bin/bugreport.cgi?bug=499480>

ok, time has been flying by these days, so it'll be here before we  
know it ;)

>>> In fact, I would simply remove the partial implementation of the
>>> protocol in the existing. There is no real need to implement the
>>> protocol yet again and incomplete, and we've had problems with it in
>>> the past ([0])..
>>
>> ok, so you'd want to require libmemcached then?
>
> Let's say I wouldn't object. *I* will not change the memcached  
> plugin to
> use libmemcached instead of the current implementation, because I'm  
> sure
> I'd only break things. But if someone puts in the effort for whatever
> reason, I'd certainly apply the patch.

I can probably work on it later, although the current impl has been  
working great.

>
>>> All things considered, I think I like the separate `memcachec'
>>> plugin best, with one big `memcached' plugin with all things to do
>>> with `memcached' (the daemon) in it being second place.
>>
>> ok, sounds good to me.
>
> I have to admit I've lost track a bit.. I changed your patch to
> implement that, right? Does it work? Should I pull it into the master
> branch?

Yes, please do!

Thanks,
-Doug

> Regards,
> -octo
> -- 
> Florian octo Forster
> Hacker in training
> GnuPG: 0x91523C3D
> http://verplant.org/