On mer, mag 07, 2014 at 12:37:41 +0200, Alessandro Ghedini wrote:
> Hi,
> 
> I've been working on merging the GIF encoder from ffmpeg, since libav's 
> doesn't
> seem to work very well (in my tests anyway), and my initial approach has been 
> to
> simply copy/paste lavc/gif.{c,h} and lavf/gif.c, make them compile and see how
> it goes (I'm using libav.git).

Sooo, thanks to Anton I've been able to make this work. I went ahead and split
the changes in smaller commits (I cherry-picked the commits from ffmpeg
directly), and while I've been able to squash a few commits together, there are
still 27 separate patches.

Now what? I didn't send the patches yet to avoid spamming the list with so many
mails, but I can go ahead and do it if you are ok with it. In any case you can
see all the commits at [0]. To merge in a local branch do:

 $ git checkout -b gif
 $ git pull https://github.com/ghedo/libav gif

There may still be some cleaning work to do. Also, the test (make fate-lavf-gif)
doesn't work yet, I'll need to look into it.

> Basically, when I'm using ffmpeg (ffmpeg -i test.mkv test.ffmpeg.gif) it 
> selects
> "bgr8" and everything goes fine.
> 
> > Output #0, gif, to 'test.ffmpeg.gif':
> >     Stream #0:0: Video: gif, bgr8, 1920x1080 [SAR 1:1 DAR 16:9], q=2-31, 
> > 200 kb/s, 100 tbn, 30 tbc
> 
> but when I use avconv, it selects "gray" (which is not what I want):
> 
> > Output #0, gif, to 'test.libav.gif':
> >     Stream #0.0: Video: gif, gray, 1920x1080 [PAR 1:1 DAR 16:9], q=2-31, 
> > 200 kb/s, 100 tbn, 1k tbc

This still happens, but it's not a big problem for the moment.

Cheers

[0] https://github.com/ghedo/libav/tree/gif

Attachment: signature.asc
Description: Digital signature

_______________________________________________
libav-devel mailing list
[email protected]
https://lists.libav.org/mailman/listinfo/libav-devel

Reply via email to