[FFmpeg-devel] [PATCH] Adding support for OSX bundles to ffmpeg command line

[Robert Swain]

On 21/3/09 14:20, Chris Marrin wrote:
> On Mar 21, 2009, at 6:53 AM, M?ns Rullg?rd wrote:
>> vmrsss <vmrsss at gmail.com> writes:
>>
>>> Wouldn't it be simpler and more flexible to have an env var FFPRESETS
>>> which, if set, overrides the standard ~/.ffmpeg and /usr/local/share/
>>> ffmpeg preset locations?
>>
>> Not overrides, precedes. The search order should be same as for any
>> typical Unix application:
>>
>> 1. $FFPRESETS
>> 2. ~/.ffmpeg
>> 3. $prefix/share/ffmpeg
>>
>> I'm sure there some reason this wouldn't work on macos of course, it
>> being from apple and all. I don't know what these bundles are, let
>> alone how they work.
>
> Ah. So that's it. You hate Apple, so anything that helps ffmpeg work
> better on OSX is bad? That's a dangerous attitude for a tool that
> purports to be a universal solution to video encoding. It's unfortunate
> you felt the need to bring prejudice into a technical discussion.
>
> Being as this is my first attempt at submitting a patch to ffmpeg, I
> have to say that the response has been pretty disappointing and
> surprising. FFmpeg is a great tool with an active support community. I
> would have though the environment would be a bit less hostile. Oh, well.
> I still like the tool.

People are free to hate whatever they like (oxymoronic phrasing but you 
know what I mean) and to be vocal about it.

Most of your interactions throughout your submission have been with M?ns 
and he is known to be one of the more abrasive people within the 
community. This doesn't make him less valuable as an FFmpeg developer, 
though it may make his responses less helpful and better taken with a 
pinch of salt. :)

Personally, I don't like the hostility and if you read back through the 
mailing lists I have campaigned against it previously. The main things 
that one can note when making submissions to FFmpeg are that some people 
here are abrasive, comments should not be taken personally and 
perseverance may well be necessary to succeed both because we have 
exceptionally high standards, that often seem to border on unnecessary 
pedantry, and because sometimes it takes perseverance to get useful 
information out of people to attain the high standards demanded.

> For the record, using an environment variable is a better solution
> (thanks to vmrsss for that). Maybe I'll try it at some point. But I
> don't really have any more time for patch submission, so for now I'll
> continue just using my existing customized version which is fine for my
> use case.

A patch for this would definitely be welcome and then you could script 
something in your bundle to dynamically set the environment variable 
depending on from where the .app is run.

Regards,
Rob