Bug 211406 - /etc/X11/xinit/Xsession relies on switchdesk
Summary: /etc/X11/xinit/Xsession relies on switchdesk
Keywords:
Status: CLOSED RAWHIDE
Alias: None
Product: Fedora
Classification: Fedora
Component: xorg-x11-xinit
Version: 6
Hardware: All
OS: Linux
medium
medium
Target Milestone: ---
Assignee: X/OpenGL Maintenance List
QA Contact:
URL:
Whiteboard:
Depends On:
Blocks:
TreeView+ depends on / blocked
 
Reported: 2006-10-18 22:02 UTC by Patrice Dumas
Modified: 2007-11-30 22:11 UTC (History)
0 users

Fixed In Version: 1.0.2-13.fc7
Doc Type: Bug Fix
Doc Text:
Clone Of:
Environment:
Last Closed: 2006-11-21 23:32:19 UTC
Type: ---
Embargoed:


Attachments (Terms of Use)

Description Patrice Dumas 2006-10-18 22:02:16 UTC
Description of problem:

the kde and the twm cases rely on switchdesk to be present 
to be launched correctly. To be more robust in the kde case
(and in the twm case switchdesk has allready been tested)
I think the following could be used:

    kde|kde1|kde2)
       if [ -x "$SWITCHDESKPATH/Xclients.kde" ]; then
           exec -l $SHELL -c "$SSH_AGENT $DBUS_LAUNCH $SWITCHDESKPATH/Xclients.kde";
       fi;
       exec -l $SHELL -c "$SSH_AGENT $DBUS_LAUNCH startkde"
       ;;

    twm)
        # fall back to twm
       exec -l $SHELL -c "$SSH_AGENT $DBUS_LAUNCH twm"
       ;;

Version-Release number of selected component (if applicable):


How reproducible:


Steps to Reproduce:
1.
2.
3.
  
Actual results:


Expected results:


Additional info:

Comment 1 Kristian Høgsberg 2006-10-23 18:36:18 UTC
We already check the existance of a switchdesk file earlier in the script, so we
should just take out the switchdesk code and launch startkde or twm directly. 
Just built xorg-x11-xinit-1.0.2-13.fc7 in rawhide with this patch:

@@ -62,11 +57,11 @@ case $# in
        exec -l $SHELL -c "$SSH_AGENT $DBUS_LAUNCH gnome-session"
        ;;
     kde|kde1|kde2)
-       exec -l $SHELL -c "$SSH_AGENT $DBUS_LAUNCH $SWITCHDESKPATH/Xclients.kde"
+       exec -l $SHELL -c "$SSH_AGENT $DBUS_LAUNCH startkde"
        ;;
     twm)
         # fall back to twm
-       exec -l $SHELL -c "$SSH_AGENT $DBUS_LAUNCH $SWITCHDESKPATH/Xclients.twm"
+       exec -l $SHELL -c "$SSH_AGENT $DBUS_LAUNCH twm"
        ;;
     *)
        # GDM provies either a command line as the first argument or

As for kde1 and kde2, switchdesk should provide symlinks if those cases are
still relevant (probably not in rawhide).

Once rawhide starts pulling from fc7 this change should be available there.

Comment 2 Patrice Dumas 2006-11-21 23:32:19 UTC
This seems to be fixed, closing RAWHIDE. The need to have 
Xsession.d/Xsession.$wm script for any desktop should 
certainly be added to the guidelines.



Note You need to log in before you can comment on or make changes to this bug.