[FFmpeg-devel] [PATCH v2] Beautify build output making it easier to see compiler warnings

[Stefano Sabatini]

On date Saturday 2010-03-06 17:18:44 +0000, M?ns Rullg?rd encoded:
> Georgi Chorbadzhiyski <gf at unixsol.org> writes:
> 
> > Around 03/04/10 16:37, Luca Barbato scribbled:
> >> On 03/03/2010 05:57 PM, Georgi Chorbadzhiyski wrote:
> >>> With this patch the build output will look similar to Linux Kernel kbuild and git.
> >>> Something like this:
> >>>
> >>>   CC         libavdevice/alldevices.c
> >>> libavdevice/alldevices.c:32: warning: no previous prototype for 'avdevice_register_all'
> >>>   CC         libavdevice/avdevice.c
> >>>   CC         libavdevice/alsa-audio-common.c
> >>>
> >>> This makes it very easy to see compiler warnings for example. If you want
> >>> to see the whole command that make executes you may use
> >>>
> >>> make Q=
> >> 
> >> Usually you call it with V=1, beside that I did something like that long
> >> ago, yours seems less invasive since doesn't cover the install target.
> >
> > Of course it should be with V=1, my bad. Attached version 2 of the patch now
> > with added support for install and uninstall targets, also make V=1 should be
> > used to see the executed command or exported BUILD_OUPUT=1.
> >
> > make clean and distclean are not covered in the moment. I can easily add them
> > but I don't think their output will improve.
> >
> > diffstat pretty-build-output_v2.diff
> >  Makefile   |   67 ++++++++++++++++++++++++++++++++-------------------
> >  common.mak |   37 +++++++++++++++++++++++-----
> >  subdir.mak |   79 ++++++++++++++++++++++++++++++++++++-------------------------
> >  3 files changed, 119 insertions(+), 64 deletions(-)
> 
> Done my way.

Which in term of git --stat is:
 
 Makefile   |    6 +++---
 common.mak |   14 +++++++++++++-
 subdir.mak |   35 +++++++++++++++++++----------------

Impressive!

Let's hope that this will help to push down the warning count...

Regards.
-- 
FFmpeg = Fostering & Fanciful Magnificient Puristic Eager Game