revert kde#108510, kde#183534

This commit is contained in:
Than Ngo 2012-10-04 14:52:20 +02:00
parent 83097942e5
commit ff6b24590c
2 changed files with 65 additions and 1 deletions

View File

@ -0,0 +1,57 @@
diff -up kdelibs-4.9.2/kinit/kinit.cpp.me kdelibs-4.9.2/kinit/kinit.cpp
--- kdelibs-4.9.2/kinit/kinit.cpp.me 2012-10-04 14:14:07.992428593 +0200
+++ kdelibs-4.9.2/kinit/kinit.cpp 2012-10-04 14:19:28.783484339 +0200
@@ -57,7 +57,6 @@
#include <klibrary.h>
#include <kdemacros.h>
#include <kstandarddirs.h>
-#include <kglobalsettings.h>
#include <kglobal.h>
#include <kconfig.h>
#include <kapplication.h>
@@ -538,7 +537,6 @@ static pid_t launch(int argc, const char
#endif
// find out these paths before forking, doing it afterwards
// crashes on some platforms, notably OSX
- const QByteArray docPath = QFile::encodeName(KGlobalSettings::documentPath());
#ifdef Q_WS_MAC
const QString bundlepath = s_instance->dirs()->findExe(QFile::decodeName(execpath));
#endif
@@ -567,12 +565,6 @@ static pid_t launch(int argc, const char
// we still want to execute `foo` even if the chdir() failed.
if (cwd && *cwd) {
(void)chdir(cwd);
- } else {
- // on Maemo5, documentPath() is on the SD card, setting it as working directory would block
- // USB mass storage access
-#ifndef Q_WS_MAEMO_5
- (void)chdir(docPath.constData());
-#endif
}
if( reset_env ) // KWRAPPER/SHELL
diff -up kdelibs-4.9.2/kio/kio/krun.cpp.me kdelibs-4.9.2/kio/kio/krun.cpp
--- kdelibs-4.9.2/kio/kio/krun.cpp.me 2012-10-04 14:24:07.520105922 +0200
+++ kdelibs-4.9.2/kio/kio/krun.cpp 2012-10-04 14:45:34.687377085 +0200
@@ -55,7 +55,6 @@
#include <kmessageboxwrapper.h>
#include <kurl.h>
#include <kglobal.h>
-#include <kglobalsettings.h>
#include <ktoolinvocation.h>
#include <kdebug.h>
#include <klocale.h>
@@ -1073,13 +1072,6 @@ bool KRun::runCommand(const QString& cmd
kDebug(7010) << "runCommand " << cmd << "," << execName;
KProcess * proc = new KProcess;
proc->setShellCommand(cmd);
- if (workingDirectory.isEmpty()) {
- // see bug 108510, and we need "alt+f2 editor" (which starts a desktop file via klauncher)
- // and "alt+f2 editor -someoption" (which calls runCommand) to be consistent.
- proc->setWorkingDirectory(KGlobalSettings::documentPath());
- } else {
- proc->setWorkingDirectory(workingDirectory);
- }
QString bin = binaryName(execName, true);
KService::Ptr service = KService::serviceByDesktopName(bin);
return runCommandInternal(proc, service.data(),

View File

@ -25,7 +25,7 @@
Summary: KDE Libraries
Version: 4.9.2
Release: 3%{?dist}
Release: 4%{?dist}
Name: kdelibs
Epoch: 6
@ -148,6 +148,9 @@ Patch55: kdelibs-4.9.2-FindSamba_samba4.patch
# backport FindKipi.cmake from Digikam SC 3.0.0-beta1 for libkipi 2 (kde#307213)
Patch56: kdelibs-4.9.1-FindKipi-libkipi2.patch
# krunner using ~/Documents as working directory, revert kde#108510, kde#183534
Patch57: kdelibs-4.9.2-revert-kde#108510-kde#183534.patch
## upstream
## security fix
@ -318,6 +321,7 @@ sed -i -e "s|@@VERSION_RELEASE@@|%{version}-%{release}|" kio/kio/kprotocolmanage
%patch54 -p1 -b .kjs-locale
%patch55 -p1 -b .FindSamba-samba4
%patch56 -p1 -b .FindKipi-libkipi2
%patch57 -p1 -b .revert-kde#108510-kde#183534
# upstream patches
@ -575,6 +579,9 @@ rm -rf %{buildroot}
%changelog
* Thu Oct 04 2012 Than Ngo <than@redhat.com> - 6:4.9.2-4
- revert kde#108510, kde#183534
* Tue Oct 02 2012 Rex Dieter <rdieter@fedoraproject.org> 6:4.9.2-3
- patch FindSamba.cmake to use pkg-config hints (#862169)