[FFmpeg-devel] Implement missing av*_version() functions

Diego Biurrun diego
Sun Aug 3 19:02:40 CEST 2008


On Sun, Aug 03, 2008 at 06:56:18PM +0200, Stefano Sabatini wrote:
> On date Sunday 2008-08-03 17:00:47 +0200, Michael Niedermayer encoded:
> > On Sun, Aug 03, 2008 at 04:49:05PM +0200, Diego Biurrun wrote:
> [...]
> > > What is the problem with mixing the version of the headers and of the
> > > linked library anyway?  Do I understand correctly that it is for the
> > > case where ffmpeg/ffplay/ffserver is dynamically linked and the version
> > > of the linked libs differs from the version of the headers used to
> > > compile the binary?
> > 
> > yes, its kinda usefull for bugreports where the user has several libs and
> > by mistake linked to the wrong one.
> > 
> > > And thus ideally there should be a _version() function for every single
> > > library?
> > 
> > yes
> 
> partly based on some patches by Andreas ?man.

I was about to implement this, you beat me to it, nice :)

> --- libavutil/Makefile	(revision 14521)
> +++ libavutil/Makefile	(working copy)
> @@ -21,6 +21,7 @@
>         sha1.o \
>         string.o \
>         tree.o \
> +       utils.o

Add a backslash, this makes adding new entries easier.

> --- libavdevice/Makefile	(revision 14521)
> +++ libavdevice/Makefile	(working copy)
> @@ -5,7 +5,8 @@
>  
> -OBJS    = alldevices.o
> +OBJS    = alldevices.o \
> +          utils.o

ditto

> --- cmdutils.c	(revision 14521)
> +++ cmdutils.c	(working copy)
> @@ -223,7 +223,7 @@
>             "libavcodec  %d\n"
>             "libavformat %d\n"
>             "libavdevice %d\n",
> -           LIBAVUTIL_BUILD, avcodec_version(), LIBAVFORMAT_BUILD, LIBAVDEVICE_BUILD);
> +           avutil_version(), avcodec_version(), avformat_version(), avdevice_version());

What about show_banner()?

Diego




More information about the ffmpeg-devel mailing list