gEDA-cvs: branch: master updated (1.3.1-20080110-5-g082c6d6)

[git@xxxxxxxxxx]

The branch, master has been updated
       via  082c6d6778d218aabd6df20a955d211335c2f43c (commit)
      from  5d9f43178c3846100c149c0baef38c26126ec233 (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
=========

 libgeda/src/s_clib.c |    2 +-
 1 files changed, 1 insertions(+), 1 deletions(-)


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

commit 082c6d6778d218aabd6df20a955d211335c2f43c
Author: Werner Hoch <werner.ho@xxxxxx>
Date:   Sat Jan 12 18:08:53 2008 +0100

    fixed error handling bug in run_source_command() [#1869941]
    
    WIFEXITED returns 1 if the command has exited normaly. Well
    an error is also a normal exit. A kill would be not be normal.
    Thus we need to check WIFEXITED and WEXITSTATUS.

:100644 100644 fc4d437... b72b746... M	libgeda/src/s_clib.c

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

commit 082c6d6778d218aabd6df20a955d211335c2f43c
Author: Werner Hoch <werner.ho@xxxxxx>
Date:   Sat Jan 12 18:08:53 2008 +0100

    fixed error handling bug in run_source_command() [#1869941]
    
    WIFEXITED returns 1 if the command has exited normaly. Well
    an error is also a normal exit. A kill would be not be normal.
    Thus we need to check WIFEXITED and WEXITSTATUS.

diff --git a/libgeda/src/s_clib.c b/libgeda/src/s_clib.c
index fc4d437..b72b746 100644
--- a/libgeda/src/s_clib.c
+++ b/libgeda/src/s_clib.c
@@ -459,7 +459,7 @@ static gchar *run_source_command (const gchar *command)
     s_log_message (_("Library command failed [%s]: Uncaught signal %i.\n"),
                    command, WTERMSIG(exit_status));
     
-  } else if (!WIFEXITED(exit_status)) {
+  } else if (WIFEXITED(exit_status) && WEXITSTATUS(exit_status)) {
     s_log_message (_("Library command failed [%s]\n"), command);
     s_log_message(_("Error output was:\n%s\n"), standard_error);
 




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