Bug 145848 - kdebase/workspace/startkde runs dbus-launch --auto-syntax, which gives errors is SHELL is /bin/csh
Summary: kdebase/workspace/startkde runs dbus-launch --auto-syntax, which gives errors...
Status: RESOLVED FIXED
Alias: None
Product: kstart
Classification: Plasma
Component: general (show other bugs)
Version: unspecified
Platform: Compiled Sources Solaris
: NOR normal
Target Milestone: ---
Assignee: Lubos Lunak
URL:
Keywords:
Depends on:
Blocks:
 
Reported: 2007-05-23 14:44 UTC by Steve Evans
Modified: 2007-05-24 13:51 UTC (History)
0 users

See Also:
Latest Commit:
Version Fixed In:


Attachments

Note You need to log in before you can comment on or make changes to this bug.
Description Steve Evans 2007-05-23 14:44:32 UTC
Version:            (using KDE Devel)
Installed from:    Compiled sources
Compiler:          gcc 3.4.6 
OS:                Solaris

(sorry, this bug is in startkde, but bugs.kde.org does not provide that in its list of applications)

startkde calls dbus-launch like this:

eval ` dbus-launch --auto-syntax --exit-with-session < /dev/null`

Unfortunately this gives errors if $SHELL is set to /bin/csh because dbus-launch outputs stuff in CSH syntax.

Changing --auto-syntax into --sh-syntax fixes it. This is in line with the dbus-launch man page that says:

Instead  of  passing  --auto-syntax,  you  may
explicity  specify a particular one by using --sh-syntax for
Bourne syntax, or --csh-syntax for csh syntax.  In  scripts,
it's  more  robust  to avoid --auto-syntax and you hopefully
know which shell your script is written in.
Comment 1 Lubos Lunak 2007-05-24 13:51:37 UTC
SVN commit 667903 by lunakl:

startkde uses /bin/sh, so ask explicitly for sh syntax
when launching dbus (for people not using sh-compatible shell)
BUG: 145848



 M  +1 -1      startkde  


--- trunk/KDE/kdebase/workspace/startkde #667902:667903
@@ -278,7 +278,7 @@
 if ! qdbus >/dev/null 2>/dev/null; then
   # Remove the next two lines after D-Bus 1.0 is released.
   # It comes with auto-starting, so running dbus-launch won't be necessary
-  eval `dbus-launch --auto-syntax --exit-with-session < /dev/null`
+  eval `dbus-launch --sh-syntax --exit-with-session < /dev/null`
   if ! qdbus >/dev/null 2>/dev/null; then
     # Startup error
     echo 'startkde: Could not start D-Bus. Check your installation.'  1>&2