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

List:       kde-bugs-dist
Subject:    [Bug 271824] Dolphin runs scripts in a wrong working directory
From:       Peter Penz <peter.penz19 () gmail ! com>
Date:       2011-10-10 20:28:36
Message-ID: E1RDMSi-0002Qz-84 () bugs ! kde ! org
[Download RAW message or body]

https://bugs.kde.org/show_bug.cgi?id=271824





--- Comment #20 from Peter Penz <peter penz19 gmail com>  2011-10-10 20:28:35 ---
To me doing the patch in KRun instead of KFileItem::run() looks fine. @David:
Is it OK if I commit the following change to KDE/4.7?

------------------------------- kio/kio/krun.cpp
-------------------------------
index 0141a7c..c978576 100644
@@ -135,7 +135,7 @@ bool KRun::runUrl(const KUrl& u, const QString& _mimetype,
QWidget* window, bool
     else if (isExecutableFile(u, _mimetype)) {
         if (u.isLocalFile() && runExecutables) {
             if (KAuthorized::authorize("shell_access")) {
-                return (KRun::runCommand(KShell::quoteArg(u.toLocalFile()),
QString(), QString(), window, asn)); // just execute the url as a command
+                return (KRun::runCommand(KShell::quoteArg(u.toLocalFile()),
QString(), QString(), window, asn, u.directory())); // just execute the url as
a command

-- 
Configure bugmail: https://bugs.kde.org/userprefs.cgi?tab=email
------- You are receiving this mail because: -------
You are watching all bug changes.
[prev in list] [next in list] [prev in thread] [next in thread]