[MPlayer-dev-eng] [PATCH] update ad_mpg123 in preparation to replace mp3lib
compn
tempn at twmi.rr.com
Thu Mar 15 00:26:59 CET 2012
On Wed, 14 Mar 2012 10:35:08 +0100, Diego Biurrun wrote:
>On Tue, Mar 13, 2012 at 10:24:16PM -0400, compn wrote:
>> On Tue, 13 Mar 2012 23:47:55 +0100, Thomas Orgis wrote:
>> >Am Tue, 13 Mar 2012 12:44:38 +0100
>> >schrieb Diego Biurrun <diego at biurrun.de>:
>> >
>> >> Committed, thanks.
>> >
>> >Cool. How long should I wait to submit a patch that eliminates
>> >mp3lib, then, or do you manage that yourselves? ;-)
>>
>> the first patch should be to put mpg123 ahead of ac mp3 in codecs.conf
>> by default. just to make sure it doesnt cause problems with any
>> samples or demuxers.
>>
>> would be nice if we made it svn external or included a copy. i'm just
>> complaining since mac and mingw wont have a system mpg123...
>
>Nothing prevents those systems from having a system mpg123.
" 1. New installation via GNU autotools "
installing autotools is never fun. or easy.
(i wonder why step 2 is in the INSTALL at all?)
and 0. Prerequesites
does not list autotools crap at all!
>> before mp3lib is retired:
>> 1. make sure mpg123 builds on all systems mplayer builds on (amiga,
>> os/2 , arm, bsd, etc).
>>
>> 2. run it on all of our mp3 samples , with ao pcm + md5 checking of
>> output. on all of those systems...
>
>Ugh, no, let's just dump it.
a better idea would be to have a distro dump it, so we can get bug
reports without affecting all users.
-compn
More information about the MPlayer-dev-eng
mailing list