[master 2/2] Use exec instead of system to run metacity

David Shea dshea at redhat.com
Wed Jul 30 21:00:34 UTC 2014


This way we don't up with a second process named "anaconda" that's
actually a middleman for the window manager.
---
 anaconda | 11 +++++------
 1 file changed, 5 insertions(+), 6 deletions(-)

diff --git a/anaconda b/anaconda
index 4d4676e..6ff4a89 100755
--- a/anaconda
+++ b/anaconda
@@ -269,14 +269,13 @@ def startMetacityWM():
         # after this point the method should never return (or throw an exception
         # outside)
         try:
-            returncode = iutil.execWithRedirect('metacity', ["--display", ":1", "--sm-disable"])
-        except BaseException as e:
+            os.execlp("metacity", "metacity", "--display", ":1", "--sm-disable")
+        except BaseException:
             # catch all possible exceptions
-            log.error("Problems running the window manager: %s", e)
-            os._exit(1)
+            pass
 
-        log.info("The window manager has terminated.")
-        os._exit(returncode)
+        log.error("Problems running the window manager")
+        os._exit(1)
 
     return childpid
 
-- 
2.0.0



More information about the anaconda-patches mailing list