[gdm] slave: don't set PATH in two places
- From: Ray Strode <halfline src gnome org>
 
- To: commits-list gnome org
 
- Cc: 
 
- Subject: [gdm] slave: don't set PATH in two places
 
- Date: Wed, 29 Aug 2012 20:29:06 +0000 (UTC)
 
commit d3b749e909e3aa65e8b053e41b434de3f1d0ab71
Author: Ray Strode <rstrode redhat com>
Date:   Wed Aug 29 16:26:59 2012 -0400
    slave: don't set PATH in two places
    
    It's redundant and causing confusion with pam_env.
    
    https://bugzilla.gnome.org/show_bug.cgi?id=682907
 daemon/gdm-session.c |   15 ---------------
 1 files changed, 0 insertions(+), 15 deletions(-)
---
diff --git a/daemon/gdm-session.c b/daemon/gdm-session.c
index c14bffc..04bc366 100644
--- a/daemon/gdm-session.c
+++ b/daemon/gdm-session.c
@@ -63,10 +63,6 @@
 #define GDM_SESSION_DBUS_ERROR_CANCEL "org.gnome.DisplayManager.Session.Error.Cancel"
 #define GDM_SESSION_DBUS_OBJECT_PATH "/org/gnome/DisplayManager/Session"
 
-#ifndef GDM_SESSION_DEFAULT_PATH
-#define GDM_SESSION_DEFAULT_PATH "/usr/local/bin:/usr/bin:/bin"
-#endif
-
 #define GDM_WORKER_DBUS_PATH "/org/gnome/DisplayManager/Worker"
 
 typedef struct
@@ -2301,17 +2297,6 @@ setup_session_environment (GdmSession *self)
                                                       "WINDOWPATH",
                                                       g_getenv ("WINDOWPATH"));
         }
-
-
-        /* FIXME: We do this here and in the session worker.  We should consolidate
-         * somehow.
-         */
-        gdm_session_set_environment_variable (self,
-                                              "PATH",
-                                              strcmp (BINDIR, "/usr/bin") == 0?
-                                              GDM_SESSION_DEFAULT_PATH :
-                                              BINDIR ":" GDM_SESSION_DEFAULT_PATH);
-
 }
 
 void
[
Date Prev][
Date Next]   [
Thread Prev][
Thread Next]   
[
Thread Index]
[
Date Index]
[
Author Index]