[PATCH UPDATE] tools/wineshelllink *METHOD 4*

Speeddymon speeddymon at yahoo.com
Thu Apr 25 16:34:03 CDT 2002


--- Paul Millar <paulm at astro.gla.ac.uk> wrote:
> On Thu, 25 Apr 2002, Speeddymon wrote:
> > +  if [ -d "/usr/lib/menu" -a "$mode" = "menu" -a
> ! `whoami` = "root" ]
> > +  then
> [...]
> > +  elif [ `whoami` = "root" ]
> > +  then
> > +    mdk_entry >> "/usr/lib/menu/wine"
> > +  fi
> 
> Sorry, I haven't been following this but isn't the
> elif condition
> redundant?
> 

not really, because what it says is:

if /usr/lib/menu is a directory and $mode is menu and
i am NOT root then su to root and run this command
else if i AM root then just run this command

if that elif were an else then the command would be
run for example if /usr/lib/menu isn't a directory or
if the mode isnt menu

> Also, AFAIK although its unusual, there's no reason
> an account name cannot
> have a space in it, so
>  `whoami`  should be  "`whoami`"
> 

only reason it is `whoami` instead of "`whoami`" is
because i copied it from the wineinstall script, so
just for the sake of argument and not having to have
someone change it in all of the scripts, i left it
like that...

> [spent too much time debugging strings in scripts
> ;^]
> 

same here, thats why im contributing...

__________________________________________________
Do You Yahoo!?
Yahoo! Games - play chess, backgammon, pool and more
http://games.yahoo.com/



More information about the wine-devel mailing list