[FFmpeg-devel] [RFC] Modify the ff* tools -version output

[Stefano Sabatini]

On date Tuesday 2008-08-12 16:00:45 +0200, Michael Niedermayer encoded:
> On Tue, Aug 12, 2008 at 03:42:48PM +0200, Stefano Sabatini wrote:
> [...]
> > > [...]
> > > > Index: cmdutils.c
> > > > ===================================================================
> > > > --- cmdutils.c	(revision 14695)
> > > > +++ cmdutils.c	(working copy)
> > > > @@ -201,13 +201,6 @@
> > > >      fprintf(stderr, "%s version " FFMPEG_VERSION ", Copyright (c) %d-2008 Fabrice Bellard, et al.\n",
> > > >              program_name, program_birth_year);
> > > >      fprintf(stderr, "  configuration: " FFMPEG_CONFIGURATION "\n");
> > > > -    fprintf(stderr, "  libavutil version: " AV_STRINGIFY(LIBAVUTIL_VERSION) "\n");
> > > > -    fprintf(stderr, "  libavcodec version: " AV_STRINGIFY(LIBAVCODEC_VERSION) "\n");
> > > > -    fprintf(stderr, "  libavformat version: " AV_STRINGIFY(LIBAVFORMAT_VERSION) "\n");
> > > > -    fprintf(stderr, "  libavdevice version: " AV_STRINGIFY(LIBAVDEVICE_VERSION) "\n");
> > > > -#if ENABLE_AVFILTER
> > > > -    fprintf(stderr, "  libavfilter version: " AV_STRINGIFY(LIBAVFILTER_VERSION) "\n");
> > > > -#endif
> > > >      fprintf(stderr, "  built on " __DATE__ " " __TIME__);
> > > >  #ifdef __GNUC__
> > > >      fprintf(stderr, ", gcc: " __VERSION__ "\n");
> > > > @@ -216,14 +209,22 @@
> > > >  #endif
> > > >  }
> > > 
> > > rejected, we need to have the version in the output in bugreports
> > 
> > See that, but just to make my point: that information may be deduced
> > by the SVN revision number, so it's redundant and leads to code
> > duplication.
> 
> with which version of libavcodec.so has a user if SVN r14710 linked
> his ffmpeg?

Is this sentence supposed to mean something?

Version numbers currently shown in show_banner() are determined
*compile-time*, so I think there is an unambiguous correlation between
SVN revision number and these libraries version numbers, then an user
can link run-time whatever libraries he likes and show_banner() will
always show the same.

But I really don't care so much, if you prefer to show them in
show_banner() I can stand with it (my previous patch already leave
them).

Regards.
-- 
FFmpeg = Fantastic & Fiendish Meaningful Portable Easy Game