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

[Alexander Strasser]

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.)

  Alex (beastd)