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

List:       kde-commits
Subject:    [kdevplatform] plugins/executescript: don't start php with -e option
From:       Niko Sams <niko.sams () gmail ! com>
Date:       2014-02-02 10:13:51
Message-ID: E1W9u3j-00009o-JH () scm ! kde ! org
[Download RAW message or body]

Git commit 9e662113c6753c7a1e83c5436ec3b2ecbfe95342 by Niko Sams.
Committed on 02/02/2014 at 10:13.
Pushed by nsams into branch 'master'.

don't start php with -e option

Accoding to php docs it should 'Activate the extended information mode, to be used by \
a debugger/profiler.' However if using xdebug as debugger/profiler this option is not \
required.

I actually don't have any idea what this option does, it causes the xdebug plugin to \
fail so remove it.

M  +1    -1    plugins/executescript/scriptappconfig.cpp

http://commits.kde.org/kdevplatform/9e662113c6753c7a1e83c5436ec3b2ecbfe95342

diff --git a/plugins/executescript/scriptappconfig.cpp \
b/plugins/executescript/scriptappconfig.cpp index 8960294..5704143 100644
--- a/plugins/executescript/scriptappconfig.cpp
+++ b/plugins/executescript/scriptappconfig.cpp
@@ -53,7 +53,7 @@ static const QString interpreterForUrl(const KUrl& url) {
     static QHash<QString, QString> knownMimetypes;
     if ( knownMimetypes.isEmpty() ) {
         knownMimetypes["text/x-python"] = "python";
-        knownMimetypes["application/x-php"] = "php -e";
+        knownMimetypes["application/x-php"] = "php";
         knownMimetypes["application/x-ruby"] = "ruby";
         knownMimetypes["application/x-shellscript"] = "bash";
         knownMimetypes["application/x-perl"] = "perl -e";


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

Configure | About | News | Add a list | Sponsored by KoreLogic