[PATCH] Set --workdir when executing a service with konsole

Alex Merry kde at randomguy3.me.uk
Mon Aug 11 16:02:01 BST 2008


On Monday 11 August 2008 15:13:46 David Faure wrote:
> On Wednesday 06 August 2008, Alex Merry wrote:
> > The attached patch fixes this, but I'm not sure whether including another
> > macro (%w for working directory) is kosher.
>
> No, you shouldn't add an undocumented %w option which is not in the desktop
> entry standard. Isn't there a way to do this without adding a % option,
> something like terminal += " --workdir " + KShell::escape(service.path());
> ?

Ah, so they're defined in the desktop standard.  I didn't know where they came 
from, which is why I asked.

I've committed the non-macro version of the fix in r845368 (trunk) and r845369 
(4.1).

Alex


-- 
Proud KDE hacker: http://www.kde.org
Get KDE 4.1 - out now!
Arch Linux: perfect for geeks like me - http://www.archlinux.org
OpenSUSE: just works - http://www.opensuse.org

-------------- next part --------------
A non-text attachment was scrubbed...
Name: signature.asc
Type: application/pgp-signature
Size: 197 bytes
Desc: This is a digitally signed message part.
URL: <http://mail.kde.org/pipermail/kde-core-devel/attachments/20080811/7e5a2e20/attachment.sig>


More information about the kde-core-devel mailing list