On 28/03/12 10:06, Måns Rullgård wrote:
> Luca Barbato <[email protected]> writes:
> 
>> On 28/03/12 09:08, Diego Biurrun wrote:
>>> All CFLAGS settings are handled from configure; it is the proper place.
>>> ---
>>>
>>> This patch could be extended to with add_host_cflags and similar
>>> infrastructure for configure, but it's not immediately obvious that
>>> such far-reaching changes would be worth the trouble when host_cflags
>>> is not modified in many places.  In any case this minimal patch I
>>> consider an improvement over the status quo.
>>
>> The proper way is having an hostconfig.h and include it in the
>> host-specific tools sources.
> 
> That would be hugely overkill for a single flag used in one place.
> The host tools do not need any configuration beyond this.
> 

Agreed, let's drop this patch for now and reconsider the issue when we'd
get at least 3 options?

lu

-- 

Luca Barbato
Gentoo/linux
http://dev.gentoo.org/~lu_zero

_______________________________________________
libav-devel mailing list
[email protected]
https://lists.libav.org/mailman/listinfo/libav-devel

Reply via email to