cURL / Mailing Lists / curl-library / Single Mail

curl-library

Re: sukender: curl/include/curl curl.h,1.380,1.381

From: Sukender <suky0001_at_free.fr>
Date: Thu, 09 Apr 2009 09:33:58 +0200

Daniel Stenberg wrote:
> On Thu, 9 Apr 2009, Yang Tse wrote:
>
>> Could this
>> http://cool.haxx.se/cvs.cgi/curl/include/curl/curl.h.diff?r1=1.380&r2=1.381
>> change be rolled back now that I've made test #558 and #559 being also
>> skipped when testing a Win32 libcurl DLL?
>
> It should be reverted. It is substandard.
>
> We ONLY add dependencies on custom variables if they are properly
> prefixed and this USES_DIRECT_CURL_UTILITIES one isn't.
>
> Not to mention that there's no comment or other explanation for the
> define which should be a requirement as well.
>

Alright, sorry for the inconvinience. Hopefully this is a very short commit. About the comment: I was a bit in a hurry, sorry again.

So, how could we do to avoid the linking problem on tests/server targets then?

-- 
Sukender
PVLE - Lightweight cross-platform game engine - http://pvle.sourceforge.net/
Received on 2009-04-09