[Ffmpeg-cvslog] r5670 - trunk/libavcodec/Makefile

[Måns Rullgård]

"Guillaume POIRIER" <poirierg at gmail.com> writes:

> Hi,
>
> On 7/8/06, Jan Knutar <jknutar at nic.fi> wrote:
>> On Saturday 08 July 2006 22:59, M?ns Rullg?rd wrote:
>>
>> > Then the ARM maintainers will have to fix their configure script.  We
>> > can't be expected to use a bunch of make variables just in case
>> > somebody else's build system sets them.
>>
>> indeed
>
> Sorry, I was probably unclear. IPP is AFAIK used in lavc. I believed
> it's used in IDCT and other decoding-related routines.
> I think the correct the would be to add a config test to test for the
> presence of IPP instead of compiling it whenever.
>
> When I was talking about MPlayer's ARM port, it's mainly because I
> have in mind that AFAIK IPP support was added in MPlayer's fork
> MPlayer-W100.
>
> So I guess removing IPP_INC doesn't only impact MPlayer but also all
> projects that uses lavc.

The IPP_INC variable was never set by anyone, not even the mplayer
configure script.  Adding it to CFLAGS was a no-op.  If other projects
compile ffmpeg with private build systems, they should set whatever
they need in CFLAGS, not hope that we will pick up whatever FOO_INC
flags they happen to be using.

-- 
M?ns Rullg?rd
mru at inprovide.com