gEDA-cvs: gaf.git: branch: master updated (1.5.0-20080706-278-g3a74831)

[git@xxxxxxxxxx]

The branch, master has been updated
       via  3a748314907a0167fc06fe08be498f40c6c7f318 (commit)
       via  2b1c2026611ef56fba3c07e88d1292b19047e818 (commit)
      from  6e10a7fbe26c858a6fa85f430d2bfe256a668c28 (commit)

Those revisions listed above that are new to this repository have
not appeared on any other notification email; so we list those
revisions in full, below.


=========
 Summary
=========

 gnetlist/src/gnetlist.c    |   10 ++++++++--
 gnetlist/utils/Makefile.am |    4 ++++
 utils/gschlas/Makefile.am  |    4 ++++
 3 files changed, 16 insertions(+), 2 deletions(-)


=================
 Commit Messages
=================

commit 3a748314907a0167fc06fe08be498f40c6c7f318
Author: Peter Clifton <pcjc2@xxxxxxxxx>
Date:   Sat Nov 1 17:46:01 2008 +0000

    More placeholders for fixing some warn-unused bugs
    
    Some were missed in the last (accidental) commit
    relating to these warnings.

:100644 100644 5192560... 0a812a7... M	gnetlist/src/gnetlist.c

commit 2b1c2026611ef56fba3c07e88d1292b19047e818
Author: Peter Clifton <pcjc2@xxxxxxxxx>
Date:   Sat Nov 1 17:45:57 2008 +0000

    Use -Werror when building with gcc compilers. (More Makefile.am)
    
    Hopefully this will help to ensure compiler warnings get noticed
    and fixed by developers.
    
    Some Makefile.am with compiled sources were missed in the last
    (accidental) commit relating to these CFLAGS.

:100644 100644 b45ed89... 07647c3... M	gnetlist/utils/Makefile.am
:100644 100644 b3269cf... 495b5f1... M	utils/gschlas/Makefile.am

=========
 Changes
=========

commit 3a748314907a0167fc06fe08be498f40c6c7f318
Author: Peter Clifton <pcjc2@xxxxxxxxx>
Date:   Sat Nov 1 17:46:01 2008 +0000

    More placeholders for fixing some warn-unused bugs
    
    Some were missed in the last (accidental) commit
    relating to these warnings.

diff --git a/gnetlist/src/gnetlist.c b/gnetlist/src/gnetlist.c
index 5192560..0a812a7 100644
--- a/gnetlist/src/gnetlist.c
+++ b/gnetlist/src/gnetlist.c
@@ -246,7 +246,10 @@ void main_prog(void *closure, int argc, char *argv[])
     /* since gnetlist is a command line utility and will deposit its output */
     /* in the current directory.  Having the output go to a different */
     /* directory will confuse the user (confused me, at first). */
-    chdir(cwd);
+    if (chdir (cwd)) {
+      /* Error occured with chdir */
+#warning FIME: What do we do?
+    }
     /* free(cwd); - Defered; see below */
 
     if (argv[argv_index] == NULL) {
@@ -309,7 +312,10 @@ void main_prog(void *closure, int argc, char *argv[])
 
     /* Change back to the directory where we started AGAIN.  This is done */
     /* because the s_traverse functions can change the Current Working Directory. */
-    chdir(cwd);
+    if (chdir (cwd)) {
+      /* Error occured with chdir */
+#warning FIXME: What do we do?
+    }
     g_free(cwd);
 
     /* temporarly reuse input_str */

commit 2b1c2026611ef56fba3c07e88d1292b19047e818
Author: Peter Clifton <pcjc2@xxxxxxxxx>
Date:   Sat Nov 1 17:45:57 2008 +0000

    Use -Werror when building with gcc compilers. (More Makefile.am)
    
    Hopefully this will help to ensure compiler warnings get noticed
    and fixed by developers.
    
    Some Makefile.am with compiled sources were missed in the last
    (accidental) commit relating to these CFLAGS.

diff --git a/gnetlist/utils/Makefile.am b/gnetlist/utils/Makefile.am
index b45ed89..07647c3 100644
--- a/gnetlist/utils/Makefile.am
+++ b/gnetlist/utils/Makefile.am
@@ -2,6 +2,10 @@
 
 bin_PROGRAMS = mk_verilog_syms
 
+if CCISGCC
+AM_CFLAGS = -Wall -Werror
+endif
+
 mk_verilog_syms_SOURCES = \
 		mk_verilog_syms.c
 
diff --git a/utils/gschlas/Makefile.am b/utils/gschlas/Makefile.am
index b3269cf..495b5f1 100644
--- a/utils/gschlas/Makefile.am
+++ b/utils/gschlas/Makefile.am
@@ -2,6 +2,10 @@
 
 bin_PROGRAMS = gschlas 
 
+if CCISGCC
+AM_CFLAGS = -Wall -Werror
+endif
+
 # don't forget all *.h files */
 gschlas_SOURCES = \
 		i_vars.c g_rc.c g_register.c globals.c \




_______________________________________________
geda-cvs mailing list
geda-cvs@xxxxxxxxxxxxxx
http://www.seul.org/cgi-bin/mailman/listinfo/geda-cvs