Bug 125284 - /etc/X11/xdm/Xsession fails to dectect the correct switchdesk path
/etc/X11/xdm/Xsession fails to dectect the correct switchdesk path
Status: CLOSED RAWHIDE
Product: Fedora
Classification: Fedora
Component: xinitrc (Show other bugs)
2
noarch Linux
medium Severity medium
: ---
: ---
Assigned To: Mike A. Harris
:
Depends On:
Blocks:
  Show dependency treegraph
 
Reported: 2004-06-04 10:00 EDT by Stefan Hoelldampf
Modified: 2007-11-30 17:10 EST (History)
2 users (show)

See Also:
Fixed In Version:
Doc Type: Bug Fix
Doc Text:
Story Points: ---
Clone Of:
Environment:
Last Closed: 2004-06-24 12:44:47 EDT
Type: ---
Regression: ---
Mount Type: ---
Documentation: ---
CRM:
Verified Versions:
Category: ---
oVirt Team: ---
RHEL 7.3 requirements from Atomic Host:
Cloudforms Team: ---


Attachments (Terms of Use)

  None (edit)
Description Stefan Hoelldampf 2004-06-04 10:00:16 EDT
In /etc/X11/xdm/Xsession, the switchdesk path is to be detected by

------------------------------------------------------------------
# backward compatibility with old switchdesk directory
if [ -f /usr/share/switchdesk/switchdesk-gui.py ] ; then
    SWITCHDESKPATH=/usr/share/switchdesk
else
    SWITCHDESKPATH=/usr/share/apps/switchdesk
fi
------------------------------------------------------------------

However, the file /usr/share/switchdesk/switchdesk-gui.py does not
exist and SWITCHDESKPATH is set to "/usr/share/apps/switchdesk",
although the Xclients.* files are located in /usr/share/switchdesk.

Version-Release number of selected component (if applicable):
xinitrc-3.41-1 and switchdesk-4.0.3-1

How reproducible:
Always
Comment 1 Rex Dieter 2004-06-04 12:00:39 EDT
Appears to be related to bug #122941
Comment 2 Mike A. Harris 2004-06-06 20:10:34 EDT
Than, what are your thoughts about this one?

I haven't yet looked at the switchdesk package or investigated the
code at hand here.  Should I just remove all of the above, and hard
code the path that is in our current switchdesk instead?

TIA
Comment 3 Ngo Than 2004-06-17 11:24:21 EDT
Mike, hier is the patch, which should fix this problem

--- /etc/X11/xdm/Xsession       2004-03-31 09:10:04.000000000 +0200
+++ Xsession    2004-06-17 17:24:03.845427726 +0200
@@ -29,7 +29,7 @@
 fi

 # backward compatibility with old switchdesk directory
-if [ -f /usr/share/switchdesk/switchdesk-gui.py ] ; then
+if [ -d /usr/share/switchdesk ] ; then
     SWITCHDESKPATH=/usr/share/switchdesk
 else
     SWITCHDESKPATH=/usr/share/apps/switchdesk
Comment 4 Mike A. Harris 2004-06-24 12:44:47 EDT
Fixed in rawhide xinitrc-4.0.0-1
Comment 5 Rex Dieter 2004-07-15 09:31:00 EDT
My $0.02:

Is the switchdesk stuff relevent anymore?  I mean, at least for
gnome/kde, all switchdesk does is run gnome-session,startkde
respectively, which is already (or should be) in xinitrc anyway.

I would argue gutting switchdesk, and removing the (extra) dependancy
on it altogether.

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