[Ffmpeg-devel] cross compiling ffmpeg

Måns Rullgård mru
Mon Feb 5 14:28:45 CET 2007


Marc Hoffman said:
>
> On Feb 5, 2007, at 7:59 AM, Diego Biurrun wrote:
>
>> On Mon, Feb 05, 2007 at 06:49:19AM -0500, Marc Hoffman wrote:
>>>
>>> On Feb 5, 2007, at 3:57 AM, Diego Biurrun wrote:
>>>
>>>> I don't much like the echos and we don't add our names to comments.
>>>> Also, you should try to hook up with the general structure of
>>>> configure and not hack in your own parts. There is one place where
>>>> we print output, there is one place where we handle options. Your
>>>> current patch is a quick hack that works for you but unacceptable
>>>> for
>>>> us unless you address the above issues.
>>>
>>> Thanks for helping me and sorry for the top posting.
>>>
>>> I'm not sure if I understand so help me through this ok.  I made the
>>> change associated with my initials and echos.  Which I agree with
>>> however.  If you look at my change which is spread across the
>>> configure script.  It was obvious to me that I need to:
>>>
>>> 1. modify the help
>>> 3. disable the default behavior of using uname to get targetos if
>>> targetos is not set
>>
>> OK
>>
>>> 2. insert the new argument handling
>>> 4. add default ignore handling to the main argument parsing loop.
>>
>> Not OK.  After your patch there are two places where arguments are
>> handled.  This is unacceptable.
>
> So would you prefer me moving all of the argument parsing to this new
> place this is fine by me but its a much larger change with lots of
> dependencies.   And if I look at the configuration script it seems
> that there are 6 places 7 with my change that actually march over the
> command line in some way or another.
>
> 7 lines matching "for opt" in buffer configure.
>      237:    for opt; do
>      243:    for opt; do
>      249:    for opt; do
>      255:    for opt; do

Those are doing something else.  They only happen to use the same variable name.

>      632:for opt do

This one you added.

>      979:for opt do

This could, and probably should, be merged with the main loop.

>      991:for opt do

The main loop.

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




More information about the ffmpeg-devel mailing list