cedric pushed a commit to branch master.

commit 5e5462a38171f3921ee84d8f5a32d80745b4416c
Author: Dieter Roelants <[email protected]>
Date:   Mon Mar 11 10:27:20 2013 +0900

    portability: don't rely on bash or zsh behavior.
    
    This force the command to be run with exec and prevent them
    for staying alive after startup or restart.
    
    Signed-off-by: Cedric BAIL <[email protected]>
---
 AUTHORS                              | 1 +
 ChangeLog                            | 4 ++++
 NEWS                                 | 1 +
 src/bin/e_init.c                     | 2 +-
 src/modules/temperature/e_mod_main.c | 2 +-
 5 files changed, 8 insertions(+), 2 deletions(-)

diff --git a/AUTHORS b/AUTHORS
index 9766f63..9578c27 100644
--- a/AUTHORS
+++ b/AUTHORS
@@ -51,3 +51,4 @@ David H. Bronke <[email protected]>
 José Roberto de Souza <[email protected]>
 Lucas Jóia <[email protected]>
 Deon Thomas
+Dieter Roelants <[email protected]>
diff --git a/ChangeLog b/ChangeLog
index afea217..770da18 100644
--- a/ChangeLog
+++ b/ChangeLog
@@ -1,3 +1,7 @@
+2013-03-11 Dieter Roelants
+
+       * portability: Don't rely on bash or zsh behavior when starting 
enlightenment_init and tempget.
+
 2013-03-01 Jérémy Zurcher
 
         * mixer shows more channels when using alsa subsystem and correctly 
disable controls
diff --git a/NEWS b/NEWS
index ffe444c..61805dd 100644
--- a/NEWS
+++ b/NEWS
@@ -121,6 +121,7 @@ Improvements:
     * EFM toolbar now has its own gadcon location
     * DND canvas merged to compositor
     * shelf gadcon can no longer resize smaller than 16x16, ensuring dnd 
success
+    * Don't rely on bash or zsh behavior when starting enlightenment_init and 
tempget.
 
 Fixes:
     * IBar menu didn't allow to configure different icon sources, show 
contents menu even on empty IBar.
diff --git a/src/bin/e_init.c b/src/bin/e_init.c
index 202f104..4db1431 100644
--- a/src/bin/e_init.c
+++ b/src/bin/e_init.c
@@ -68,7 +68,7 @@ e_init_show(void)
    else ver = strdup("XvX");
 
    snprintf(buf, sizeof(buf),
-            "%s/enlightenment/utils/enlightenment_init \'%s\' \'%i\' \'%s\' 
\'%s\'",
+            "exec %s/enlightenment/utils/enlightenment_init \'%s\' \'%i\' 
\'%s\' \'%s\'",
             e_prefix_lib_get(), theme,
             e_config->font_hinting, tit, ver);
    printf("RUN INIT: %s\n", buf);
diff --git a/src/modules/temperature/e_mod_main.c 
b/src/modules/temperature/e_mod_main.c
index 52c6998..a309b2c 100644
--- a/src/modules/temperature/e_mod_main.c
+++ b/src/modules/temperature/e_mod_main.c
@@ -321,7 +321,7 @@ temperature_face_update_config(Config_Face *inst)
        if (!inst->tempget_exe) 
          {
             snprintf(buf, sizeof(buf),
-                     "%s/%s/tempget %i \"%s\" %i", 
+                     "exec %s/%s/tempget %i \"%s\" %i", 
                      e_module_dir_get(temperature_config->module), 
MODULE_ARCH, 
                      inst->sensor_type,
                      (inst->sensor_name ? inst->sensor_name : "(null)"),

-- 

------------------------------------------------------------------------------
Symantec Endpoint Protection 12 positioned as A LEADER in The Forrester  
Wave(TM): Endpoint Security, Q1 2013 and "remains a good choice" in the  
endpoint security space. For insight on selecting the right partner to 
tackle endpoint security challenges, access the full report. 
http://p.sf.net/sfu/symantec-dev2dev

Reply via email to