[FFmpeg-devel] [PATCH] List enabled code in configure output

Ramiro Polla ramiro
Wed Sep 26 21:39:26 CEST 2007


  wrote:
> Ramiro Polla <ramiro at lisha.ufsc.br> writes:
> 
>> M?ns Rullg?rd wrote:
>>> Ramiro Ribeiro Polla <ramiro at lisha.ufsc.br> writes:
>>>
>>>
>>>> Hello,
>>>>
>>>>  wrote:
>>>>
>>>>> Ramiro Ribeiro Polla wrote:
>>>>>
>>>>>> Hello,
>>>>>>
>>>>>> I uploaded a patch [1] to issue17 [2] on the roundup tracker. Is it ok
>>>>>> to apply?
>>>>>>
>>>>> No, it's not POSIX shell compatible.
>>>>>
>>>> Ok, I'll try again with: mans --verbose 2 ./configure
>>>>
>>>> Could you enlighten us where it's not POSIX compatible?
>>>>
>>> The ${space:0:$length}
>> Replaced with a while loop that adds one space at a time.
> 
> I'm not having anything of the kind.  What's this patch good for
> anyway?

As has already been discussed before in [1], some people would like for 
configure to print out which codecs and formats have been selected. I 
would like to have this feature, and Michael said he likes it too.

I didn't like the fact that it would output one really long unreadable 
line. I also wouldn't like if it was one entry per line, since that 
would make configure's output really, really long. I suggest something 
in between: using show_list, print the output tabulated with 8 spaces. 
But I didn't find a way to make the code portable and elegant at the 
same time.

show_list_1.diff makes show_list much faster, so I think it should be 
applied independent of the rest of this discussion.

These are 3 possibilities of implementing listing the selected output:
show_list_2.diff prints one entry per line (long output).
show_list_3.diff prints them sequentially (long lines).
And another option is tabulating the output. Would you be against this 
kind of output, even if we managed to get the source code to be portable 
and clean?

Ramiro Polla

[1] http://thread.gmane.org/gmane.comp.video.ffmpeg.devel/49478
-------------- next part --------------
An embedded and charset-unspecified text was scrubbed...
Name: show_list_1.diff
URL: <http://lists.mplayerhq.hu/pipermail/ffmpeg-devel/attachments/20070926/62453e77/attachment.asc>
-------------- next part --------------
An embedded and charset-unspecified text was scrubbed...
Name: show_list_2.diff
URL: <http://lists.mplayerhq.hu/pipermail/ffmpeg-devel/attachments/20070926/62453e77/attachment.txt>
-------------- next part --------------
An embedded and charset-unspecified text was scrubbed...
Name: show_list_3.diff
URL: <http://lists.mplayerhq.hu/pipermail/ffmpeg-devel/attachments/20070926/62453e77/attachment-0001.asc>



More information about the ffmpeg-devel mailing list