From: Paul W. <pa...@bo...> - 2008-09-28 05:50:11
|
> On None, Paul Wise wrote: > > synfig-core/trunk/src/modules/mod_libavcodec/trgt_av.cpp, line 38 > > <http://patches.synfig.org/r/6/diff/1/?file=20#file20line38> > > > > test > > wrote: > Given the inherent incomprehensibility of the configure script, it is probably a good idea to document this code. Particularly why you use both PKG_CHECK_MODULES and AC_CHECK_HEADERS. The reason is basically that ffmpeg development practices aren't great, but are getting better. Part of that process means they have to move things around, change API/ABI to be the "right way" and so on. Unfortunately that isn't good for people who like stability (like packages that depend on ffmpeg and the distros that package their stuff) I put a new patch up to cope with some of these changes, inspired by how VLC have coped with the changes. - Paul ----------------------------------------------------------- This is an automatically generated e-mail. To reply, visit: http://patches.synfig.org/r/6/#review10 ----------------------------------------------------------- On 2008-09-27 22:43:59.125774, Paul Wise wrote: > > ----------------------------------------------------------- > This is an automatically generated e-mail. To reply, visit: > http://patches.synfig.org/r/6/ > ----------------------------------------------------------- > > (Updated 2008-09-27 22:43:59.125774) > > > Review request for Synfig. > > > Summary > ------- > > Cope with the headers for libavformat and libswscale moving around. > > > Diffs > ----- > > > Diff: http://patches.synfig.org/r/6/diff > > > Testing > ------- > > Built with older ffmpeg, benG built with newer ffmpeg. > > > Thanks, > > Paul > > |