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

[Stefano Sabatini]

Hi all,

with the patches applied the output of the ff* tools -version option becomes:

stefano at geppetto ~/s/ffmpeg> ffmpeg -version
FFmpeg version SVN-r14667, Copyright (c) 2000-2008 Fabrice Bellard, et al.
  configuration: --prefix=/home/stefano --disable-shared --enable-libschroedinger --enable-libx264 --enable-pthreads --enable-gpl --enable-debug=3 --enable-libtheora --enable-libvorbis --enable-avfilter --enable-libamr-nb --enable-libamr-wb --enable-nonfree --enable-libfaad --enable-libfaac --enable-x11grab --enable-libmp3lame --disable-optimizations --disable-mmx
  built on Aug  8 2008 21:21:18, gcc: 4.2.3 20071014 (prerelease) (Debian 4.2.2-3)
FFmpeg SVN-r14667
  libavutil version(C): 49.9.0
  libavutil version(L): 49.9.0
  libavcodec version(C): 51.63.0
  libavcodec version:(L) 51.63.0
  libavformat version(C): 52.20.0
  libavformat version(L): 52.20.0
  libavdevice version(C): 52.1.0
  libavdevice version(L): 52.1.0
  libavfilter version(C): 0.1.0
  libavfilter version(L): 0.1.0

I think showing the libav* versions in the banner is useless, since
they may be deduced anyway from the SVN revision number *or* from the
-version output.

Also I see no point to print them on stderr rather on stdout.

The (C) and (L) in the -version output stand for "Linked" and
"Compiled", I'm not sure this is the best way to show it, anyway I
find this (to show both the compiled and linked libav* versions) a
pretty useful debugging feature.

As a final note I think that the comment:
     /* TODO: add function interface to avutil and avformat avdevice*/

(which I think refers to the missing implementation of av*_version())
can be removed now.

Let me know what you think about these changes (yes I know the
cmdutils.c needs to be splitted).

Regards.
-- 
FFmpeg = Friendly and Frightening Mastering Purposeless Elastic Goblin
-------------- next part --------------
A non-text attachment was scrubbed...
Name: implement-av-stringify-version-00.patch
Type: text/x-diff
Size: 1305 bytes
Desc: not available
URL: <http://lists.mplayerhq.hu/pipermail/ffmpeg-devel/attachments/20080808/624c2207/attachment.patch>
-------------- next part --------------
A non-text attachment was scrubbed...
Name: change-version-00.patch
Type: text/x-diff
Size: 2379 bytes
Desc: not available
URL: <http://lists.mplayerhq.hu/pipermail/ffmpeg-devel/attachments/20080808/624c2207/attachment-0001.patch>