On Mon, Apr 19, 2010 at 7:57 AM, John Griessen <j...@ecosensory.com> wrote: > Anything to help understandability of compile output is good by me. > Your change probably won't decrease error output, just the ones that > go smoothly are shortened, right? >
The errors/warnings themselves aren't shortened, it just shortens the command line used to compile. As an example this is the output of the build with this patch applied, versus without. (sorry for the spam of compile output, but since people without automake 1.11 won't be able to see the change, I think this will help demonstrate what this patch would do. In fact the mere fact I feel like I need to apologize for pasting compile output could be part of the argument :) With patch (warnings obvious) CC action.o CC autoplace.o CC autoroute.o ../../../pcb/pcb/src/autoroute.c:3431: warning: ‘add_clearance’ defined but not used CC buffer.o CC change.o ../../../pcb/pcb/src/change.c: In function ‘ChangeElementName’: ../../../pcb/pcb/src/change.c:1016: warning: unused variable ‘old’ CC clip.o CC command.o Without patch (very difficult to quickly see what exactly is going on, let alone catch the warnings): gcc -std=gnu99 -DLOCALEDIR=\"/usr/local/share/locale\" -DHAVE_CONFIG_H -I. -I../../../pcb/pcb/src -I.. -I../../../pcb/pcb -I../../../pcb/pcb/src/icons -I../../../pcb/pcb/src/gts -I/usr/include -I/usr/include -I/usr/include -DPREFIXDIR=\"/usr/local\" -DBINDIR=\"/usr/local/bin\" -DHOST=\"x86_64-unknown-linux-gnu\" -DPCBLIBDIR=\"/usr/local/share/pcb\" -DPCBTREEDIR=\"/usr/local/share/pcb/newlib\" -DPCBTREEPATH=\"/usr/local/share/pcb/newlib:/usr/local/share/pcb/pcblib-newlib\" -DNDEBUG -g -O2 -I/usr/include -I/usr/include -I/usr/include -I/usr/include/glib-2.0 -I/usr/lib/glib-2.0/include -pthread -I/usr/include/gtk-2.0 -I/usr/lib/gtk-2.0/include -I/usr/include/atk-1.0 -I/usr/include/cairo -I/usr/include/pango-1.0 -I/usr/include/glib-2.0 -I/usr/lib/glib-2.0/include -I/usr/include/pixman-1 -I/usr/include/freetype2 -I/usr/include/libpng14 -Wall -Wdeclaration-after-statement -MT action.o -MD -MP -MF $depbase.Tpo -c -o action.o ../../../pcb/pcb/src/action.c &&\ mv -f $depbase.Tpo $depbase.Po depbase=`echo autoplace.o | sed 's|[^/]*$|.deps/&|;s|\.o$||'`;\ gcc -std=gnu99 -DLOCALEDIR=\"/usr/local/share/locale\" -DHAVE_CONFIG_H -I. -I../../../pcb/pcb/src -I.. -I../../../pcb/pcb -I../../../pcb/pcb/src/icons -I../../../pcb/pcb/src/gts -I/usr/include -I/usr/include -I/usr/include -DPREFIXDIR=\"/usr/local\" -DBINDIR=\"/usr/local/bin\" -DHOST=\"x86_64-unknown-linux-gnu\" -DPCBLIBDIR=\"/usr/local/share/pcb\" -DPCBTREEDIR=\"/usr/local/share/pcb/newlib\" -DPCBTREEPATH=\"/usr/local/share/pcb/newlib:/usr/local/share/pcb/pcblib-newlib\" -DNDEBUG -g -O2 -I/usr/include -I/usr/include -I/usr/include -I/usr/include/glib-2.0 -I/usr/lib/glib-2.0/include -pthread -I/usr/include/gtk-2.0 -I/usr/lib/gtk-2.0/include -I/usr/include/atk-1.0 -I/usr/include/cairo -I/usr/include/pango-1.0 -I/usr/include/glib-2.0 -I/usr/lib/glib-2.0/include -I/usr/include/pixman-1 -I/usr/include/freetype2 -I/usr/include/libpng14 -Wall -Wdeclaration-after-statement -MT autoplace.o -MD -MP -MF $depbase.Tpo -c -o autoplace.o ../../../pcb/pcb/src/autoplace.c &&\ mv -f $depbase.Tpo $depbase.Po depbase=`echo autoroute.o | sed 's|[^/]*$|.deps/&|;s|\.o$||'`;\ gcc -std=gnu99 -DLOCALEDIR=\"/usr/local/share/locale\" -DHAVE_CONFIG_H -I. -I../../../pcb/pcb/src -I.. -I../../../pcb/pcb -I../../../pcb/pcb/src/icons -I../../../pcb/pcb/src/gts -I/usr/include -I/usr/include -I/usr/include -DPREFIXDIR=\"/usr/local\" -DBINDIR=\"/usr/local/bin\" -DHOST=\"x86_64-unknown-linux-gnu\" -DPCBLIBDIR=\"/usr/local/share/pcb\" -DPCBTREEDIR=\"/usr/local/share/pcb/newlib\" -DPCBTREEPATH=\"/usr/local/share/pcb/newlib:/usr/local/share/pcb/pcblib-newlib\" -DNDEBUG -g -O2 -I/usr/include -I/usr/include -I/usr/include -I/usr/include/glib-2.0 -I/usr/lib/glib-2.0/include -pthread -I/usr/include/gtk-2.0 -I/usr/lib/gtk-2.0/include -I/usr/include/atk-1.0 -I/usr/include/cairo -I/usr/include/pango-1.0 -I/usr/include/glib-2.0 -I/usr/lib/glib-2.0/include -I/usr/include/pixman-1 -I/usr/include/freetype2 -I/usr/include/libpng14 -Wall -Wdeclaration-after-statement -MT autoroute.o -MD -MP -MF $depbase.Tpo -c -o autoroute.o ../../../pcb/pcb/src/autoroute.c &&\ mv -f $depbase.Tpo $depbase.Po ../../../pcb/pcb/src/autoroute.c:3431: warning: ‘add_clearance’ defined but not used depbase=`echo buffer.o | sed 's|[^/]*$|.deps/&|;s|\.o$||'`;\ gcc -std=gnu99 -DLOCALEDIR=\"/usr/local/share/locale\" -DHAVE_CONFIG_H -I. -I../../../pcb/pcb/src -I.. -I../../../pcb/pcb -I../../../pcb/pcb/src/icons -I../../../pcb/pcb/src/gts -I/usr/include -I/usr/include -I/usr/include -DPREFIXDIR=\"/usr/local\" -DBINDIR=\"/usr/local/bin\" -DHOST=\"x86_64-unknown-linux-gnu\" -DPCBLIBDIR=\"/usr/local/share/pcb\" -DPCBTREEDIR=\"/usr/local/share/pcb/newlib\" -DPCBTREEPATH=\"/usr/local/share/pcb/newlib:/usr/local/share/pcb/pcblib-newlib\" -DNDEBUG -g -O2 -I/usr/include -I/usr/include -I/usr/include -I/usr/include/glib-2.0 -I/usr/lib/glib-2.0/include -pthread -I/usr/include/gtk-2.0 -I/usr/lib/gtk-2.0/include -I/usr/include/atk-1.0 -I/usr/include/cairo -I/usr/include/pango-1.0 -I/usr/include/glib-2.0 -I/usr/lib/glib-2.0/include -I/usr/include/pixman-1 -I/usr/include/freetype2 -I/usr/include/libpng14 -Wall -Wdeclaration-after-statement -MT buffer.o -MD -MP -MF $depbase.Tpo -c -o buffer.o ../../../pcb/pcb/src/buffer.c &&\ mv -f $depbase.Tpo $depbase.Po depbase=`echo change.o | sed 's|[^/]*$|.deps/&|;s|\.o$||'`;\ gcc -std=gnu99 -DLOCALEDIR=\"/usr/local/share/locale\" -DHAVE_CONFIG_H -I. -I../../../pcb/pcb/src -I.. -I../../../pcb/pcb -I../../../pcb/pcb/src/icons -I../../../pcb/pcb/src/gts -I/usr/include -I/usr/include -I/usr/include -DPREFIXDIR=\"/usr/local\" -DBINDIR=\"/usr/local/bin\" -DHOST=\"x86_64-unknown-linux-gnu\" -DPCBLIBDIR=\"/usr/local/share/pcb\" -DPCBTREEDIR=\"/usr/local/share/pcb/newlib\" -DPCBTREEPATH=\"/usr/local/share/pcb/newlib:/usr/local/share/pcb/pcblib-newlib\" -DNDEBUG -g -O2 -I/usr/include -I/usr/include -I/usr/include -I/usr/include/glib-2.0 -I/usr/lib/glib-2.0/include -pthread -I/usr/include/gtk-2.0 -I/usr/lib/gtk-2.0/include -I/usr/include/atk-1.0 -I/usr/include/cairo -I/usr/include/pango-1.0 -I/usr/include/glib-2.0 -I/usr/lib/glib-2.0/include -I/usr/include/pixman-1 -I/usr/include/freetype2 -I/usr/include/libpng14 -Wall -Wdeclaration-after-statement -MT change.o -MD -MP -MF $depbase.Tpo -c -o change.o ../../../pcb/pcb/src/change.c &&\ mv -f $depbase.Tpo $depbase.Po ../../../pcb/pcb/src/change.c: In function ‘ChangeElementName’: ../../../pcb/pcb/src/change.c:1016: warning: unused variable ‘old’ depbase=`echo clip.o | sed 's|[^/]*$|.deps/&|;s|\.o$||'`;\ gcc -std=gnu99 -DLOCALEDIR=\"/usr/local/share/locale\" -DHAVE_CONFIG_H -I. -I../../../pcb/pcb/src -I.. -I../../../pcb/pcb -I../../../pcb/pcb/src/icons -I../../../pcb/pcb/src/gts -I/usr/include -I/usr/include -I/usr/include -DPREFIXDIR=\"/usr/local\" -DBINDIR=\"/usr/local/bin\" -DHOST=\"x86_64-unknown-linux-gnu\" -DPCBLIBDIR=\"/usr/local/share/pcb\" -DPCBTREEDIR=\"/usr/local/share/pcb/newlib\" -DPCBTREEPATH=\"/usr/local/share/pcb/newlib:/usr/local/share/pcb/pcblib-newlib\" -DNDEBUG -g -O2 -I/usr/include -I/usr/include -I/usr/include -I/usr/include/glib-2.0 -I/usr/lib/glib-2.0/include -pthread -I/usr/include/gtk-2.0 -I/usr/lib/gtk-2.0/include -I/usr/include/atk-1.0 -I/usr/include/cairo -I/usr/include/pango-1.0 -I/usr/include/glib-2.0 -I/usr/lib/glib-2.0/include -I/usr/include/pixman-1 -I/usr/include/freetype2 -I/usr/include/libpng14 -Wall -Wdeclaration-after-statement -MT clip.o -MD -MP -MF $depbase.Tpo -c -o clip.o ../../../pcb/pcb/src/clip.c &&\ mv -f $depbase.Tpo $depbase.Po depbase=`echo command.o | sed 's|[^/]*$|.deps/&|;s|\.o$||'`;\ gcc -std=gnu99 -DLOCALEDIR=\"/usr/local/share/locale\" -DHAVE_CONFIG_H -I. -I../../../pcb/pcb/src -I.. -I../../../pcb/pcb -I../../../pcb/pcb/src/icons -I../../../pcb/pcb/src/gts -I/usr/include -I/usr/include -I/usr/include -DPREFIXDIR=\"/usr/local\" -DBINDIR=\"/usr/local/bin\" -DHOST=\"x86_64-unknown-linux-gnu\" -DPCBLIBDIR=\"/usr/local/share/pcb\" -DPCBTREEDIR=\"/usr/local/share/pcb/newlib\" -DPCBTREEPATH=\"/usr/local/share/pcb/newlib:/usr/local/share/pcb/pcblib-newlib\" -DNDEBUG -g -O2 -I/usr/include -I/usr/include -I/usr/include -I/usr/include/glib-2.0 -I/usr/lib/glib-2.0/include -pthread -I/usr/include/gtk-2.0 -I/usr/lib/gtk-2.0/include -I/usr/include/atk-1.0 -I/usr/include/cairo -I/usr/include/pango-1.0 -I/usr/include/glib-2.0 -I/usr/lib/glib-2.0/include -I/usr/include/pixman-1 -I/usr/include/freetype2 -I/usr/include/libpng14 -Wall -Wdeclaration-after-statement -MT command.o -MD -MP -MF $depbase.Tpo -c -o command.o ../../../pcb/pcb/src/command.c &&\ mv -f $depbase.Tpo $depbase.Po Jared _______________________________________________ geda-user mailing list geda-user@moria.seul.org http://www.seul.org/cgi-bin/mailman/listinfo/geda-user