[EGIT] [apps/terminology] master 01/01: options: use ecore_exe_pipe_run() instead of fork+exec

[Boris Faure]

billiob pushed a commit to branch master.

http://git.enlightenment.org/apps/terminology.git/commit/?id=72eaf53e26c8a7774a80cbb34a7740db17cd5a7b

commit 72eaf53e26c8a7774a80cbb34a7740db17cd5a7b
Author: Boris Faure <bill...@gmail.com>
Date:   Sun May 28 23:30:05 2017 +0200

    options: use ecore_exe_pipe_run() instead of fork+exec
    
    Patch provided by Vincent Torri
---
 src/bin/options_elm.c | 16 ++++------------
 1 file changed, 4 insertions(+), 12 deletions(-)

diff --git a/src/bin/options_elm.c b/src/bin/options_elm.c
index 01dd554..f48b6f3 100644
--- a/src/bin/options_elm.c
+++ b/src/bin/options_elm.c
@@ -9,18 +9,10 @@ launch_elm_config(void *_data EINA_UNUSED,
                 Evas_Object *_obj EINA_UNUSED,
                 void *_event_info EINA_UNUSED)
 {
-   int pid;
-
-   pid = fork();
-   if (pid < 0)
-     {
-        ERR("unable to fork: %s", strerror(errno));
-        return;
-     }
-   if (pid == 0)
-     {
-        execl("/bin/sh", "sh", "-c", "elementary_config", NULL);
-     }
+   Ecore_Exe *exe;
+
+   exe = ecore_exe_pipe_run("elementary_config", ECORE_EXE_NONE, NULL);
+   ecore_exe_free(exe);
 }
 
 void

--