[FFmpeg-devel] [PATCH] RTMPE support

[Howard Chu]

Michael Niedermayer wrote:
> On Fri, Mar 19, 2010 at 09:18:26AM +0200, Kostya wrote:
>> On Thu, Mar 18, 2010 at 10:56:23PM -0700, Howard Chu wrote:
>>> compn wrote:
>>>> On Thu, 18 Mar 2010 14:20:07 -0700, Howard Chu wrote:
>>>>> Carl Eugen Hoyos wrote:
>>>>>> Howard Chu<hyc<at>    highlandsun.com>    writes:
>>>>>> Before sending a new patch, the usefulness of adding a new external dependency
>>>>>> should probably be discussed.
>>>>>
>>>>> Discussion is all well and good, but at the end of the day the original
>>>>> question remains unanswered - do you want the new features or not. Now, or
>>>>> later. This patch provides everything, now.
>>>>
>>>> i havent tested or reviewed, but i think ffmpeg://rtmp:// would coexist
>>>> with this patch.
>>>>
>>>> this could be worked around kind of like how live555 tries rtsp://
>>>> first, then mplayer's realrtsp tries rtsp://. but thats a lot of work
>>>> for not a lot of gain.
>>>
>>> No gain. If the stream uses rtmp:// and SWF Verification, you just waste
>>> time going thru the old code.
>>
>> I admit that RTMP support in FFmpeg is not that good because of lack of
>> interest.
>> And I'm more and more inclined to see it supported via something that
>> does that work. Actually it's Adobe fault - if RTMP was stable we'd
>> have working implementation by now and no need of external dependencies;
>> but no, somebody decided it's a good idea to change handshake every
>> couple of months and add some commands and not document anything.
>>
>> So until it's settled down it may be good to stash away current RTMP
>> code and use rtmpdump. It has active developers and I'd rather spend my
>> time on REing some codec. Unless someone wants to beef lavf version up.
>
> i dont mind supporting external libs but i think it should not replace
> our implementation but coexist.

This patch adds configure hooks to allow choosing one or the other.

-- 
   -- Howard Chu
   CTO, Symas Corp.           http://www.symas.com
   Director, Highland Sun     http://highlandsun.com/hyc/
   Chief Architect, OpenLDAP  http://www.openldap.org/project/
-------------- next part --------------
An embedded and charset-unspecified text was scrubbed...
Name: dif2.txt
URL: <http://lists.mplayerhq.hu/pipermail/ffmpeg-devel/attachments/20100325/c2fc4e09/attachment.txt>