[MPlayer-cvslog] CVS: main/libmpdemux ebml.c,1.4,1.5

[Rich Felker]

On Wed, Jul 20, 2005 at 10:35:57AM +0200, Alexander Strasser wrote:
> Hi all,
> 
> Reimar Döffinger wrote:
> > On Wed, Jul 20, 2005 at 04:05:44AM +0200, Diego Biurrun wrote:
> > > On Tue, Jul 19, 2005 at 11:14:46PM +0200, Reimar Döffinger wrote:
> > > > Since internal libavcodec still is not mandatory
> > > 
> > > I would consider that news...
> > 
> > Well, we are still testing for it, aren't we? And I can't see any other
> > reason why we have all these
> > #if #if LIBAVCODEC_BUILD < ...
> > things around.
> > Also I said _internal_ specifically, using the .so might give a few
> > problems as well (mostly because of non-matching versions).
> > I think Ivan was suggested making it mandatory some time ago, but
> > somebody else disagreed.
> 
>   I don't consider it news, it is reality and mplayer does really
> `work' without libavcodec! I was suggesting making it mandatory
> short time ago, and had a short discussion with some devels on IRC
> about it, but it didn't came to a real conclusion anyways.
>   Maybe we should reconsider this again, as more and more parts of
> libavcodec would be needed in mplayer to avoid code duplication.
>   Though i still wonder sometimes if it wouldn't be possible to
> split some of the more generic parts of libavcodec into e.g. an
> independent subdir, so we could just make that part mandatory.
> Comments welcome. (BTW mandatory means configure fails when the
> object in question could not be found.)

I'm mildly against this. If we want to make parts of lavc mandatory we
should just make the whole thing mandatory. MPlayer is basically
useless without it anyway..

Rich