[collectd] curl_json fails to build against YAJL-2.0.x

[Johan Bergström]

Hey,

On Jul 4, 2011, at 11:30 , Grzegorz Nosek wrote:

> W dniu 04.07.2011 11:22, Johan Bergström pisze:
>> Thanks for the patch, looks like we're still not there though (FWIW, this is GCC 4.5.2):
>> 
>> /bin/sh ../libtool  --tag=CC   --mode=compile x86_64-pc-linux-gnu-gcc -DHAVE_CONFIG_H -I.    -Wall -Werror  -O2 -pipe -c -o curl_json_la-curl_json.lo `test -f 'curl_json.c' || echo './'`curl_json.c
>> libtool: compile:  x86_64-pc-linux-gnu-gcc -DHAVE_CONFIG_H -I. -Wall -Werror -O2 -pipe -c curl_json.c  -fPIC -DPIC -o .libs/curl_json_la-curl_json.o
>> cc1: warnings being treated as errors
>> curl_json.c:257:3: error: initialization from incompatible pointer type
>> curl_json.c:258:3: error: initialization from incompatible pointer type
>> curl_json.c:260:3: error: initialization from incompatible pointer type
>> curl_json.c: In function 'cj_curl_perform':
>> curl_json.c:699:3: error: too few arguments to function 'yajl_alloc'
>> /usr/include/yajl/yajl_parse.h:130:26: note: declared here
>> curl_json.c:732:3: error: implicit declaration of function 'yajl_complete_parse'
>> make[3]: *** [curl_json_la-curl_json.lo] Error 1
>> 
>> Not sure how arch got this through?
> 
> Not sure either, but it builds without problems on my Arch machine (collectd 5.0.0, yajl 2.0.2-1, no relevant patches other than what I sent).
> 
> Care to provide the functions with compile errors as you see them? Try reverting to vanilla curl_json.c and applying the patch again.

Never mind. Somehow mixed the paths to YAJL - so I built against 1.X in above case.

> 
> Best regards,
> Grzegorz Nosek

Kind regards,
Johan