[prev in list] [next in list] [prev in thread] [next in thread] 

List:       kde-commits
Subject:    Re: quanta [POSSIBLY UNSAFE]
From:       Mantia Andras <amantia () freemail ! hu>
Date:       2002-12-08 16:55:46
[Download RAW message or body]

On 2002. December 08., Sunday 17:35, Dirk Mueller wrote:
> On Son, 08 Dez 2002, Andras Mantia wrote:
> > -  m_process = new KShellProcess;
> > +  m_process = new KProcess;
> > +  m_process->setUseShell(true);
>
> are you sure that you *want* to have a shell at all?

As shell script should be able to be accepted as Quanta plugins, yes I think I 
need the shell... Until now I haven't thought much about it, but now I've 
done a little test, and without setUseShell(true) the shell scripts are not 
executed (just as I thought). 
 BTW, Ossi's idea about an "expandFilename(const QString &name);" method would 
be helpful. As I saw from the commits log, it would be useful also fo 
Cervisia...

Andras
[prev in list] [next in list] [prev in thread] [next in thread]