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

gEDA-cvs: gaf.git: branch: master updated (1.7.1-20110619-179-g400f396)



The branch, master has been updated
       via  400f396b8aa598017fda2ead72c2fbc72b1fc321 (commit)
       via  f70e0ec9cb2ea1b9518642f7e300c1fc230b1d5a (commit)
      from  1bf087eac8287e5d835be9961e82bb6a523a2c75 (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
=========

 utils/src/gsch2pcb.c |   28 ++++++++++++++--------------
 1 files changed, 14 insertions(+), 14 deletions(-)


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

commit 400f396b8aa598017fda2ead72c2fbc72b1fc321
Author: Krzysztof KoÅ?ciuszkiewicz <k.kosciuszkiewicz@xxxxxxxxx>
Commit: Krzysztof KoÅ?ciuszkiewicz <k.kosciuszkiewicz@xxxxxxxxx>

    gsch2pcb: do not use sys/wait.h macros
    
    These are not available on windows - exit status of subprocess is
    directly returned by g_spawn_sync.
    
    It seems same trick should work for Unix/Linux processes that called
    exit(0), so we should be safe here.
    
    If not, we can include conditional compilation based on G_OS_WIN32.

:100644 100644 2a61e42... 76eed3e... M	utils/src/gsch2pcb.c

commit f70e0ec9cb2ea1b9518642f7e300c1fc230b1d5a
Author: Krzysztof KoÅ?ciuszkiewicz <k.kosciuszkiewicz@xxxxxxxxx>
Commit: Krzysztof KoÅ?ciuszkiewicz <k.kosciuszkiewicz@xxxxxxxxx>

    gsch2pcb: whitespace fix

:100644 100644 74e09f0... 2a61e42... M	utils/src/gsch2pcb.c

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

commit 400f396b8aa598017fda2ead72c2fbc72b1fc321
Author: Krzysztof KoÅ?ciuszkiewicz <k.kosciuszkiewicz@xxxxxxxxx>
Commit: Krzysztof KoÅ?ciuszkiewicz <k.kosciuszkiewicz@xxxxxxxxx>

    gsch2pcb: do not use sys/wait.h macros
    
    These are not available on windows - exit status of subprocess is
    directly returned by g_spawn_sync.
    
    It seems same trick should work for Unix/Linux processes that called
    exit(0), so we should be safe here.
    
    If not, we can include conditional compilation based on G_OS_WIN32.

diff --git a/utils/src/gsch2pcb.c b/utils/src/gsch2pcb.c
index 2a61e42..76eed3e 100644
--- a/utils/src/gsch2pcb.c
+++ b/utils/src/gsch2pcb.c
@@ -193,7 +193,7 @@ build_and_run_command (const gchar *format, ...)
                       &standard_error,      /* standard error */
                       &status,              /* exit status return */
                       &error)) {            /* GError return */
-      if (WIFEXITED(status) && WEXITSTATUS(status) == 0)
+      if (status == 0)
         result = TRUE;
       else {
         if (standard_error)

commit f70e0ec9cb2ea1b9518642f7e300c1fc230b1d5a
Author: Krzysztof KoÅ?ciuszkiewicz <k.kosciuszkiewicz@xxxxxxxxx>
Commit: Krzysztof KoÅ?ciuszkiewicz <k.kosciuszkiewicz@xxxxxxxxx>

    gsch2pcb: whitespace fix

diff --git a/utils/src/gsch2pcb.c b/utils/src/gsch2pcb.c
index 74e09f0..2a61e42 100644
--- a/utils/src/gsch2pcb.c
+++ b/utils/src/gsch2pcb.c
@@ -183,21 +183,21 @@ build_and_run_command (const gchar *format, ...)
       printf ("\n%s", SEP_STRING);
     
     if (g_spawn_sync (".",                  /* Working directory */
-		      args,                 /* argv */
-		      NULL,                 /* envp */
-		      G_SPAWN_SEARCH_PATH | /* flags */
-		      G_SPAWN_STDOUT_TO_DEV_NULL,
-		      NULL,                 /* child_setup */
-		      NULL,                 /* user data */
-		      NULL,                 /* standard output */
-		      &standard_error,       /* standard error */
-		      &status,              /* exit status return */
-              &error)) {              /* GError return */
+                      args,                 /* argv */
+                      NULL,                 /* envp */
+                      G_SPAWN_SEARCH_PATH | /* flags */
+                      G_SPAWN_STDOUT_TO_DEV_NULL,
+                      NULL,                 /* child_setup */
+                      NULL,                 /* user data */
+                      NULL,                 /* standard output */
+                      &standard_error,      /* standard error */
+                      &status,              /* exit status return */
+                      &error)) {            /* GError return */
       if (WIFEXITED(status) && WEXITSTATUS(status) == 0)
-	result = TRUE;
+        result = TRUE;
       else {
-	if (standard_error)
-      fputs(standard_error, stderr);
+        if (standard_error)
+          fputs(standard_error, stderr);
       }
     }
     else {



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