[Patch] Bashism in startkde
Armin Berres
trigger at space-based.de
Mon Jun 1 20:25:08 BST 2009
On Mon, 01 Jun 09 11:46, Oswald Buddenhagen wrote:
> On Mon, Jun 01, 2009 at 08:13:01AM +0000, Sune Vuorela wrote:
> > On 2009-06-01, Thiago Macieira <thiago at kde.org> wrote:
> > > Isn't $() another extension? Shouldn't we use `` instead?
> >
> > It depends on what you call "extension".
> > $() is posix shell, but afaik not supported by the original Bourne
> > Shell. No one uses Bourne Shell today though.
> >
> > I just tested:
> >
> > FOO=$(id -u)
> > echo $FOO
> >
> > and I get the expected output in
> > bash
> > dash
> > posh
> > pdksh
> > ksh93
> > mksh
> > zsh
> >
> > (And then I ran out of shells)
> >
> now log in into a default install of some closed-source unix and try
> again. :-P
>
> fwiw, kde assumes a posix shell in several places, but kprocess goes
> through some hoops to make sure it gets one.
So, this one should be ok to commit then, right:
|Index: workspace/startkde.cmake
|===================================================================
|--- workspace/startkde.cmake (revision 975766)
|+++ workspace/startkde.cmake (working copy)
|@@ -367,7 +367,7 @@
| export KDE_SESSION_VERSION
| xprop -root -f KDE_SESSION_VERSION 32c -set KDE_SESSION_VERSION 4
|
|-KDE_SESSION_UID=$UID
|+KDE_SESSION_UID=`id -ru`
| export KDE_SESSION_UID
|
| # We set LD_BIND_NOW to increase the efficiency of kdeinit.
Greetings,
Armmin
More information about the kde-core-devel
mailing list