[gnome-builder] subprocess: prefer g_unix_open_pipe() over pipe2()
- From: Christian Hergert <chergert src gnome org>
- To: commits-list gnome org
- Cc: 
- Subject: [gnome-builder] subprocess: prefer g_unix_open_pipe() over pipe2()
- Date: Wed,  5 Oct 2016 21:51:16 +0000 (UTC)
commit b980add939bbdd9e7fbe5610e209d102846dacc0
Author: Christian Hergert <chergert redhat com>
Date:   Wed Oct 5 14:49:32 2016 -0700
    subprocess: prefer g_unix_open_pipe() over pipe2()
    
    pipe2() is apparently not available for us on Mac OS X. While
    that is not a major target for us, if it's easy to support we
    might as well do it.
    
    https://bugzilla.gnome.org/show_bug.cgi?id=772279
 libide/subprocess/ide-breakout-subprocess.c |   21 ++++++---------------
 1 files changed, 6 insertions(+), 15 deletions(-)
---
diff --git a/libide/subprocess/ide-breakout-subprocess.c b/libide/subprocess/ide-breakout-subprocess.c
index 5a1a0ea..2056959 100644
--- a/libide/subprocess/ide-breakout-subprocess.c
+++ b/libide/subprocess/ide-breakout-subprocess.c
@@ -1203,11 +1203,8 @@ ide_breakout_subprocess_initable_init (GInitable     *initable,
     }
   else if (self->flags & G_SUBPROCESS_FLAGS_STDIN_PIPE)
     {
-      if (pipe2 (stdin_pair, O_CLOEXEC) != 0)
-        {
-          set_error_from_errno (error);
-          IDE_GOTO (cleanup_fds);
-        }
+      if (!g_unix_open_pipe (stdin_pair, O_CLOEXEC, error))
+        IDE_GOTO (cleanup_fds);
     }
   else
     {
@@ -1248,11 +1245,8 @@ ide_breakout_subprocess_initable_init (GInitable     *initable,
     }
   else if (self->flags & G_SUBPROCESS_FLAGS_STDOUT_PIPE)
     {
-      if (pipe2 (stdout_pair, O_CLOEXEC) != 0)
-        {
-          set_error_from_errno (error);
-          IDE_GOTO (cleanup_fds);
-        }
+      if (!g_unix_open_pipe (stdout_pair, O_CLOEXEC, error))
+        IDE_GOTO (cleanup_fds);
     }
   else
     {
@@ -1291,11 +1285,8 @@ ide_breakout_subprocess_initable_init (GInitable     *initable,
     }
   else if (self->flags & G_SUBPROCESS_FLAGS_STDERR_PIPE)
     {
-      if (pipe2 (stderr_pair, O_CLOEXEC) != 0)
-        {
-          set_error_from_errno (error);
-          IDE_GOTO (cleanup_fds);
-        }
+      if (!g_unix_open_pipe (stderr_pair, O_CLOEXEC, error))
+        IDE_GOTO (cleanup_fds);
     }
   else
     {
[
Date Prev][
Date Next]   [
Thread Prev][
Thread Next]   
[
Thread Index]
[
Date Index]
[
Author Index]