[FFmpeg-devel] [PATCH] Matroska Muxer

David Conrad umovimus
Wed Sep 5 03:15:04 CEST 2007


On Sep 3, 2007, at 1:49 PM, Aurelien Jacobs wrote:

> On Mon, 3 Sep 2007 12:55:20 -0400
> David Conrad <umovimus at gmail.com> wrote:
>
>> On Aug 30, 2007, at 4:30 AM, Diego Biurrun wrote:
>>
>>> On Thu, Aug 30, 2007 at 12:10:24AM -0400, David Conrad wrote:
>>>>
>>>> New patch attached, with regression tests updated.
>>>
>>> Documentation and build system parts are OK, I'm not sure if I  
>>> already
>>> said this.
>>>
>>> Some unimportant nits below.
>>>
>>> [...]
>>>
>>> There are some more instances of lowercase ID, I prefer uppercase.
>>> Feel
>>> free to ignore me.
>>
>> I changed the two lowercase IDs I in comments; were there others?
>
> I see no others. So the code should now be ready for ffmpeg svn.
> You can start committing...

It took me a while to get git-svn to work with my repository that I  
cloned from git.mplayerhq.hu. I wound up having to use git-svn to  
redownload all of the svn history into the same repository, then  
rebasing my matroskaenc branch onto the new git-svn history, which I  
guess wasn't the optimal way...




More information about the ffmpeg-devel mailing list