[FFmpeg-devel] Bug report : build problem with "configure --disable-encoders"

[David Geldreich]

Hello Diego,

Le 26 nov. 08 ? 01:48, Diego Biurrun a ?crit :

> On Tue, Nov 25, 2008 at 09:29:28AM +0100, David Geldreich wrote:
>>
>> Only in /Users/dgeld/src/ffmpeg/: all.diff
>> Only in /Users/dgeld/src/ffmpeg/: all.diff~
>> diff -r -up -x .svn ./libavcodec/aandcttab.c /Users/dgeld/src/ 
>> ffmpeg/libavcodec/aandcttab.c
>> --- ./libavcodec/aandcttab.c	2008-11-25 09:18:43.000000000 +0100
>> +++ /Users/dgeld/src/ffmpeg/libavcodec/aandcttab.c	2008-11-25  
>> 09:18:50.000000000 +0100
>
> 'svn diff' will make your life easier.

But you are asking for multiple successive patches. I could make the  
first patch with "svn diff" but then I should simulate a commit by  
applying the patch on a first source hierarchy. Then on a second  
source hierarchy, I apply the first patch and apply my second  
modification. Then I "diff -r -up -x .svn" the 2 hierarchy to get the  
second patch.
Or perhaps you have an "SVN sandbox" where I could commit ?

So I propose that we apply the move patch and then discuss the  
variable names.

I have chosen these "not looking good names" regarding what it is  
found in aactab.h and ac3tab.h. But in other places you have names  
that do not follow the ff_* rules so do what you want with the renaming.

Applying the move patch will solve the "configure --disable-encoders"  
problem.

Best regards.