[Author Prev][Author Next][Thread Prev][Thread Next][Author Index][Thread Index]

Re: gEDA-user: Patch to PCB build system needs testing/feedback

To: gEDA user mailing list <geda-user@xxxxxxxxxxxxxx>
Subject: Re: gEDA-user: Patch to PCB build system needs testing/feedback
From: Jared Casper <jaredcasper@xxxxxxxxx>
Date: Mon, 19 Apr 2010 09:36:11 -0700
Delivered-to: archiver@xxxxxxxx
Delivered-to: geda-user@xxxxxxxxxxxxxx
Delivery-date: Mon, 19 Apr 2010 12:36:29 -0400
Dkim-signature: v=1; a=rsa-sha256; c=relaxed/relaxed; d=gmail.com; s=gamma; h=domainkey-signature:mime-version:received:in-reply-to:references :date:received:message-id:subject:from:to:content-type :content-transfer-encoding; bh=8sL81Ud1frjgi+R3dGl48LOOz4Ys1kDhCr1r826G4y8=; b=U/tPCaCDhBp297FU6k4Z/W8s6B35XP1VPvpT2wVjGviqDyxnHQ1GNKy+pqQghJp732 Fx1cFzJdBFuU7X6s+UzJi3HqjGZmeMUDyevDUVWDUYray8b13iMAPcJjQHeXJwT2zdb2 DXuKCZOUoUVIBwEOWi/hm7U2CV1EZ3zHeOmEc=
Domainkey-signature: a=rsa-sha1; c=nofws; d=gmail.com; s=gamma; h=mime-version:in-reply-to:references:date:message-id:subject:from:to :content-type:content-transfer-encoding; b=sVlh/BJelX+27tRcW+iyXL9D6Qcp4MqCteF4zmvtgVmlkX3bTf2x05EPNeomqDja6G lcftYRdTSAzsrPwVxIFAUeK+p6J4idT8N9MZNrPm23ntKjDGqtU8WiHELdm/3Rr06yCM S25F6Prldz6qsNIOvm7iG9KkfmwCLuBeEncOM=
In-reply-to: <4BCC6F40.2090405@xxxxxxxxxxxxxx>
List-archive: <http://www.seul.org/pipermail/geda-user>
List-help: <mailto:geda-user-request@moria.seul.org?subject=help>
List-id: gEDA user mailing list <geda-user.moria.seul.org>
List-post: <mailto:geda-user@moria.seul.org>
List-subscribe: <http://www.seul.org/cgi-bin/mailman/listinfo/geda-user>, <mailto:geda-user-request@moria.seul.org?subject=subscribe>
List-unsubscribe: <http://www.seul.org/cgi-bin/mailman/options/geda-user>, <mailto:geda-user-request@moria.seul.org?subject=unsubscribe>
References: <h2lc91559131004181605g25b86fd4z469503413eb530ff@xxxxxxxxxxxxxx> <4BCC6F40.2090405@xxxxxxxxxxxxxx>
Reply-to: gEDA user mailing list <geda-user@xxxxxxxxxxxxxx>
Sender: geda-user-bounces@xxxxxxxxxxxxxx

On Mon, Apr 19, 2010 at 7:57 AM, John Griessen <john@xxxxxxxxxxxxxx> 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@xxxxxxxxxxxxxx
http://www.seul.org/cgi-bin/mailman/listinfo/geda-user

References:
- gEDA-user: Patch to PCB build system needs testing/feedback
  - From: Jared Casper
- Re: gEDA-user: Patch to PCB build system needs testing/feedback
  - From: John Griessen

Prev by Author: Re: gEDA-user: Patch to PCB build system needs testing/feedback
Next by Author: Re: gEDA-user: Patch to PCB build system needs testing/feedback
Previous by thread: Re: gEDA-user: Patch to PCB build system needs testing/feedback
Next by thread: Re: gEDA-user: Patch to PCB build system needs testing/feedback
Index(es):
- Author
- Thread