On 03/26/2010 03:31 PM, Daniel Stenberg wrote:
> On Fri, 26 Mar 2010, Ben Greear wrote:
>
>>> $ make CFLAGS='<your flags here>'
>>
>> I was hoping to have it be a configure option. I got something working
>> (see patch I posted a few minutes ago), but the logic that adds
>> -Werror by cat'ing a file is pretty ugly.
>
> It's rather ugly and very very specific for your idea. I don't see
> what's wrong with Marco Maggi's suggestion, or by simply watching for
> the warnings in the output! I don't think it's worth adding code to
> support such an extreme edge-case.

Ok, I can carry that particular -Werror hack in my own tree..but are you ok
with the rest of the patch that fixes the warnings?

Thanks,
Ben

-- 
Ben Greear <greearb_at_candelatech.com>
Candela Technologies Inc  http://www.candelatech.com
-------------------------------------------------------------------
List admin: http://cool.haxx.se/list/listinfo/curl-library
Etiquette:  http://curl.haxx.se/mail/etiquette.html