Accepting request 244426 from KDE:Frameworks5

Update to 5.1.0

OBS-URL: https://build.opensuse.org/request/show/244426
OBS-URL: https://build.opensuse.org/package/show/openSUSE:Factory/kio?expand=0&rev=7
This commit is contained in:
Ludwig Nussel 2014-08-16 13:42:07 +00:00 committed by Git OBS Bridge
commit cee5c4b276
8 changed files with 17 additions and 221 deletions

View File

@ -1,48 +0,0 @@
From c0217c6e5e1e7592fcd0d808f1766e8be6289db0 Mon Sep 17 00:00:00 2001
From: David Faure <faure@kde.org>
Date: Fri, 4 Jul 2014 22:59:48 +0200
Subject: [PATCH 04/10] When executing commands set the working directory to be
the current directory
if not specified by the service.
Forward port commit 89d4585 by Dawit which fixes #142597.
+ fix path() to toLocalFile().
REVIEW: 119020
---
src/widgets/krun.cpp | 10 ++++++++--
1 file changed, 8 insertions(+), 2 deletions(-)
diff --git a/src/widgets/krun.cpp b/src/widgets/krun.cpp
index 77708a0..bc39d83 100644
--- a/src/widgets/krun.cpp
+++ b/src/widgets/krun.cpp
@@ -176,7 +176,8 @@ bool KRun::runUrl(const QUrl &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, u.adjusted(QUrl::RemoveFilename).path())); // just execute the url as a command
+ return (KRun::runCommand(KShell::quoteArg(u.toLocalFile()), QString(), QString(),
+ window, asn, u.adjusted(QUrl::RemoveFilename).toLocalFile())); // just execute the url as a command
// ## TODO implement deleting the file if tempFile==true
} else {
noAuth = true;
@@ -426,8 +427,13 @@ static bool runTempService(const KService &_service, const QList<QUrl> &_urls, Q
}
//qDebug() << "runTempService: KProcess args=" << args;
+ QString path(_service.path());
+ if (path.isEmpty() && !_urls.isEmpty() && _urls.first().isLocalFile()) {
+ path = _urls.first().adjusted(QUrl::RemoveFilename).toLocalFile();
+ }
+
return runCommandInternal(args.join(" "), &_service, KIO::DesktopExecParser::executablePath(_service.exec()),
- _service.name(), _service.icon(), window, asn, _service.path());
+ _service.name(), _service.icon(), window, asn, path);
}
// WARNING: don't call this from DesktopExecParser, since klauncher uses that too...
--
2.0.0

View File

@ -1,54 +0,0 @@
From c42e52d366ffcc016a487627b935b34183af759a Mon Sep 17 00:00:00 2001
From: Eike Hein <hein@kde.org>
Date: Sat, 5 Jul 2014 20:52:52 +0200
Subject: [PATCH 05/10] Fix template discovery in KNewFileMenu.
Incorrect port to QStandardPaths was collecting dirs instead of
files within them.
Reviewed-By: David Faure
REVIEW:119130
---
src/filewidgets/knewfilemenu.cpp | 16 ++++++++++++++--
1 file changed, 14 insertions(+), 2 deletions(-)
diff --git a/src/filewidgets/knewfilemenu.cpp b/src/filewidgets/knewfilemenu.cpp
index 4f1ca10..a24ac9c 100644
--- a/src/filewidgets/knewfilemenu.cpp
+++ b/src/filewidgets/knewfilemenu.cpp
@@ -841,10 +841,12 @@ void KNewFileMenuPrivate::_k_slotFillTemplates()
{
KNewFileMenuSingleton *s = kNewMenuGlobals();
//qDebug();
+
+ const QStringList dirs = QStandardPaths::locateAll(QStandardPaths::GenericDataLocation, "templates", QStandardPaths::LocateDirectory);
+
// Ensure any changes in the templates dir will call this
if (! s->dirWatch) {
s->dirWatch = new KDirWatch;
- const QStringList dirs = QStandardPaths::locateAll(QStandardPaths::GenericDataLocation, "templates", QStandardPaths::LocateDirectory);
for (QStringList::const_iterator it = dirs.constBegin(); it != dirs.constEnd(); ++it) {
//qDebug() << "Templates resource dir:" << *it;
s->dirWatch->addDir(*it);
@@ -863,7 +865,17 @@ void KNewFileMenuPrivate::_k_slotFillTemplates()
s->templatesList->clear();
// Look into "templates" dirs.
- const QStringList files = QStandardPaths::locateAll(QStandardPaths::GenericDataLocation, "templates", QStandardPaths::LocateDirectory);
+ QStringList files;
+ QDir dir;
+
+ Q_FOREACH (const QString &path, dirs) {
+ dir.setPath(path);
+ const QStringList &entryList(dir.entryList(QStringList() << "*.desktop", QDir::Files));
+ Q_FOREACH (const QString &entry, entryList) {
+ files.append(dir.path() + dir.separator() + entry);
+ }
+ }
+
QMap<QString, KNewFileMenuSingleton::Entry> slist; // used for sorting
Q_FOREACH (const QString &file, files) {
//qDebug() << file;
--
2.0.0

View File

@ -1,77 +0,0 @@
From bffdd7fa0c03f255bded5071e39f50c768891eca Mon Sep 17 00:00:00 2001
From: Harald Sitter <sitter@kde.org>
Date: Tue, 8 Jul 2014 11:45:04 -0400
Subject: [PATCH 07/10] fix kurlrequester dialog accept handling
switch to qdialog::accepted signal, rename slot (on private class) and
remove accepted check in slot
BUG: 337226
REVIEW: 119174
---
src/widgets/kurlrequester.cpp | 16 +++++++---------
src/widgets/kurlrequester.h | 2 +-
2 files changed, 8 insertions(+), 10 deletions(-)
diff --git a/src/widgets/kurlrequester.cpp b/src/widgets/kurlrequester.cpp
index bc40284..cf0b0c7 100644
--- a/src/widgets/kurlrequester.cpp
+++ b/src/widgets/kurlrequester.cpp
@@ -213,7 +213,7 @@ public:
// slots
void _k_slotUpdateUrl();
void _k_slotOpenDialog();
- void _k_slotFileDialogFinished();
+ void _k_slotFileDialogAccepted();
QUrl m_startDir;
KUrlRequester *m_parent; // TODO: rename to 'q'
@@ -408,18 +408,16 @@ void KUrlRequester::KUrlRequesterPrivate::_k_slotOpenDialog()
}
}
-void KUrlRequester::KUrlRequesterPrivate::_k_slotFileDialogFinished()
+void KUrlRequester::KUrlRequesterPrivate::_k_slotFileDialogAccepted()
{
if (!myFileDialog) {
return;
}
- if (myFileDialog->result() == QDialog::Accepted) {
- QUrl newUrl = myFileDialog->selectedUrls().first();
- if (newUrl.isValid()) {
- m_parent->setUrl(newUrl);
- emit m_parent->urlSelected(url());
- }
+ QUrl newUrl = myFileDialog->selectedUrls().first();
+ if (newUrl.isValid()) {
+ m_parent->setUrl(newUrl);
+ emit m_parent->urlSelected(url());
}
}
@@ -466,7 +464,7 @@ QFileDialog *KUrlRequester::fileDialog() const
d->applyFileMode(d->myFileDialog, d->fileDialogMode);
d->myFileDialog->setWindowModality(d->fileDialogModality);
- connect(d->myFileDialog, SIGNAL(finished()), SLOT(_k_slotFileDialogFinished()));
+ connect(d->myFileDialog, SIGNAL(accepted()), SLOT(_k_slotFileDialogAccepted()));
}
return d->myFileDialog;
diff --git a/src/widgets/kurlrequester.h b/src/widgets/kurlrequester.h
index 528cffc..163c435 100644
--- a/src/widgets/kurlrequester.h
+++ b/src/widgets/kurlrequester.h
@@ -328,7 +328,7 @@ private:
Q_PRIVATE_SLOT(d, void _k_slotUpdateUrl())
Q_PRIVATE_SLOT(d, void _k_slotOpenDialog())
- Q_PRIVATE_SLOT(d, void _k_slotFileDialogFinished())
+ Q_PRIVATE_SLOT(d, void _k_slotFileDialogAccepted())
};
--
2.0.0

View File

@ -1,28 +0,0 @@
From cf430d82c09978d4d9e43e90f6bc60e2a2900942 Mon Sep 17 00:00:00 2001
From: David Faure <faure@kde.org>
Date: Tue, 8 Jul 2014 17:48:23 +0200
Subject: [PATCH 08/10] warn about asserting when people can't write proper
desktop files
Tested by Eike Hein with mpv's desktop file.
---
src/core/desktopexecparser.cpp | 3 ++-
1 file changed, 2 insertions(+), 1 deletion(-)
diff --git a/src/core/desktopexecparser.cpp b/src/core/desktopexecparser.cpp
index 085643d..be62791 100644
--- a/src/core/desktopexecparser.cpp
+++ b/src/core/desktopexecparser.cpp
@@ -176,7 +176,8 @@ QStringList KIO::DesktopExecParser::supportedProtocols(const KService &service)
KRunMX1 mx1(service);
QString exec = service.exec();
if (mx1.expandMacrosShellQuote(exec) && !mx1.hasUrls) {
- Q_ASSERT(supportedProtocols.isEmpty()); // huh? If you support protocols you need %u or %U...
+ qWarning() << service.entryPath() << "contains a X-KDE-Protocols line but doesn't use %u or %U in its Exec line! This is inconsistent.";
+ return QStringList();
} else {
if (supportedProtocols.isEmpty()) {
// compat mode: assume KIO if not set and it's a KDE app (or a KDE service)
--
2.0.0

View File

@ -1,3 +0,0 @@
version https://git-lfs.github.com/spec/v1
oid sha256:652c258fcb96d0b7bfecaa1dfef8b502f43b0dd01c3fe110cdb312eafb12f62f
size 2538568

3
kio-5.1.0.tar.xz Normal file
View File

@ -0,0 +1,3 @@
version https://git-lfs.github.com/spec/v1
oid sha256:73ceb6be282b9ceb0bc8ca12ddbc2c10cf215ec8807455136cdcb91fbf5c3dab
size 2251736

View File

@ -1,3 +1,15 @@
-------------------------------------------------------------------
Sat Aug 2 10:20:18 UTC 2014 - hrvoje.senjan@gmail.com
- Update to 5.1.0
* For more details please see:
http://www.kde.org/announcements/kde-frameworks-5.1.php
- Drop 0004-When-executing-commands-set-the-working-directory-to.patch,
0005-Fix-template-discovery-in-KNewFileMenu.patch,
0007-fix-kurlrequester-dialog-accept-handling.patch and
0008-warn-about-asserting-when-people-can-t-write-proper-.patch,
merged upstream
-------------------------------------------------------------------
Fri Jul 11 09:48:45 UTC 2014 - hrvoje.senjan@gmail.com

View File

@ -17,11 +17,11 @@
Name: kio
Version: 5.0.0
Version: 5.1.0
Release: 0
%define kf5_version %{version}
BuildRequires: cmake >= 2.8.12
BuildRequires: extra-cmake-modules >= 1.0.0
BuildRequires: extra-cmake-modules >= 1.1.0
BuildRequires: fdupes
BuildRequires: karchive-devel >= %{kf5_version}
BuildRequires: kbookmarks-devel >= %{kf5_version}
@ -66,11 +66,6 @@ Group: System/GUI/KDE
Url: http://www.kde.org
Source: http://download.kde.org/stable/frameworks/%{version}/%{name}-%{version}.tar.xz
Source1: baselibs.conf
# PATCH-FIX-UPSTREAMS
Patch1: 0004-When-executing-commands-set-the-working-directory-to.patch
Patch2: 0005-Fix-template-discovery-in-KNewFileMenu.patch
Patch3: 0007-fix-kurlrequester-dialog-accept-handling.patch
Patch4: 0008-warn-about-asserting-when-people-can-t-write-proper-.patch
BuildRoot: %{_tmppath}/%{name}-%{version}-build
%description
@ -100,10 +95,6 @@ Development files.
%lang_package
%prep
%setup -q
%patch1 -p1
%patch2 -p1
%patch3 -p1
%patch4 -p1
%build
%cmake_kf5 -d build