Hello community,
here is the log from the commit of package kdebase4-workspace for openSUSE:Factory checked in at 2011-11-14 11:55:18
++++++++++++++++++++++++++++++++++++++++++++++++++++++++++++++++++++++++
Comparing /work/SRC/openSUSE:Factory/kdebase4-workspace (Old)
and /work/SRC/openSUSE:Factory/.kdebase4-workspace.new (New)
++++++++++++++++++++++++++++++++++++++++++++++++++++++++++++++++++++++++
Package is "kdebase4-workspace", Maintainer is "kde-maintainers@suse.de"
Changes:
--------
--- /work/SRC/openSUSE:Factory/kdebase4-workspace/kdebase4-workspace.changes 2011-11-05 08:35:17.000000000 +0100
+++ /work/SRC/openSUSE:Factory/.kdebase4-workspace.new/kdebase4-workspace.changes 2011-11-14 11:55:31.000000000 +0100
@@ -1,0 +2,18 @@
+Wed Nov 9 19:38:02 UTC 2011 - javier@opensuse.org
+
+- Add plasma-crash-on-logout.diff to fix bko#275286
+
+-------------------------------------------------------------------
+Thu Nov 3 15:46:31 UTC 2011 - mlin@suse.com
+
+- Add plasma-detect-gpsd-correctly.diff
+ * used return value of gps_open() as condition instead of checking
+ m_gpsdata, that fix plasmoid crash when gpsd not running. (bnc#722133)
+
+-------------------------------------------------------------------
+Wed Nov 2 20:25:11 CET 2011 - dmueller@suse.de
+
+- update to 4.7.3
+ * see http://kde.org/announcements/changelogs/changelog4_7_2to4_7_3.php for details
+
+-------------------------------------------------------------------
@@ -4,0 +23,5 @@
+
+-------------------------------------------------------------------
+Wed Nov 2 19:59:05 CET 2011 - dmueller@suse.de
+
+- rediff 4.7 branch diff to include fix for regression kde#285347
python-kdebase4.changes: same change
Old:
----
kde-workspace-4.7.2.tar.bz2
kdebase4-workspace-powerdevil-kconfig-cbce41f0.diff
kwin-performance.diff
New:
----
kde-workspace-4.7.3.tar.bz2
plasma-crash-on-logout.diff
plasma-detect-gpsd-correctly.diff
++++++++++++++++++++++++++++++++++++++++++++++++++++++++++++++++++++++++
Other differences:
------------------
++++++ kdebase4-workspace.spec ++++++
--- /var/tmp/diff_new_pack.OfPSY5/_old 2011-11-14 11:55:35.000000000 +0100
+++ /var/tmp/diff_new_pack.OfPSY5/_new 2011-11-14 11:55:35.000000000 +0100
@@ -23,7 +23,7 @@
Group: System/GUI/KDE
Summary: The KDE Workspace Components
Url: http://www.kde.org/
-Version: 4.7.2
+Version: 4.7.3
Release: 1
BuildRequires: ConsoleKit-devel
%if %suse_version > 1140
@@ -98,13 +98,14 @@
Patch106: plasma-dashboard-leave.diff
Patch107: plasma-kickoff-newly-collapsing.diff
Patch108: plasma-panel-resize-hint.diff
+Patch109: plasma-crash-on-logout.diff
Patch123: kdm-fix-generic-greeter.diff
Patch124: kdm-fix-labelcolors.diff
Patch125: pam-translate.diff
Patch201: plasma-notifications-kill-timer.diff
Patch202: plasma-disable-networkmanager.diff
Patch211: disable-python-support.diff
-Patch212: kdebase4-workspace-powerdevil-kconfig-cbce41f0.diff
+Patch212: plasma-detect-gpsd-correctly.diff
Requires: kde4-kgreeter-plugins = %version
Requires: kdebase4-workspace-liboxygenstyle = %version
%kde4_runtime_requires
@@ -173,6 +174,7 @@
%patch106
%patch107 -p1
%patch108
+%patch109 -p1
%patch123 -p1
%patch124 -p1
###KDE47: rediff
++++++ python-kdebase4.spec ++++++
--- /var/tmp/diff_new_pack.OfPSY5/_old 2011-11-14 11:55:35.000000000 +0100
+++ /var/tmp/diff_new_pack.OfPSY5/_new 2011-11-14 11:55:35.000000000 +0100
@@ -18,7 +18,7 @@
Name: python-kdebase4
-Version: 4.7.2
+Version: 4.7.3
Release: 1
License: GPLv2+
Summary: Python bindings for KDE 4 desktop shell
++++++ 4_7_BRANCH.diff ++++++
++++ 1873 lines (skipped)
++++ between /work/SRC/openSUSE:Factory/kdebase4-workspace/4_7_BRANCH.diff
++++ and /work/SRC/openSUSE:Factory/.kdebase4-workspace.new/4_7_BRANCH.diff
++++++ kde-workspace-4.7.2.tar.bz2 -> kde-workspace-4.7.3.tar.bz2 ++++++
/work/SRC/openSUSE:Factory/kdebase4-workspace/kde-workspace-4.7.2.tar.bz2 /work/SRC/openSUSE:Factory/.kdebase4-workspace.new/kde-workspace-4.7.3.tar.bz2 differ: char 11, line 1
++++++ plasma-crash-on-logout.diff ++++++
commit 77b1374b99ee91d23d8802dd1dec9a5f630e919a
Author: Aaron Seigo
Date: Wed Nov 9 20:15:50 2011 +0100
only delete the layout when it isn't assigned to another layout
based on the patch by Raul Fernandes
BUG:275286
diff --git a/plasma/desktop/applets/tasks/taskgroupitem.cpp b/plasma/desktop/applets/tasks/taskgroupitem.cpp
index 6b23b71..d7816d3 100644
--- a/plasma/desktop/applets/tasks/taskgroupitem.cpp
+++ b/plasma/desktop/applets/tasks/taskgroupitem.cpp
@@ -79,8 +79,11 @@ TaskGroupItem::TaskGroupItem(QGraphicsWidget *parent, Tasks *applet)
TaskGroupItem::~TaskGroupItem()
{
- delete m_tasksLayout;
- m_tasksLayout = 0;
+ if (!m_offscreenLayout && !m_mainLayout) {
+ // only delete this if we have neither an offscreen layout or a mainlayout
+ // if we do, then they will delete the layout for us.
+ delete m_tasksLayout;
+ }
}
bool TaskGroupItem::isSplit()
++++++ plasma-detect-gpsd-correctly.diff ++++++
From 60d13c58234086816b8c241cb5a0b29539e23303 Mon Sep 17 00:00:00 2001
From: Max Lin
Date: Thu, 3 Nov 2011 16:00:08 +0800
Subject: [PATCH] Used the gps_open() return value as condition instead of checking m_gpsdata
Checking the return value of gps_open() as condition instead of checking
m_gpsdata for GPSD_API_MAJOR_VERSION >=5, there should be prevent crash
occurred when no gpsd running.
BUG: 277036
REVIEW: 103035
Index: kde-workspace-4.7.2/plasma/generic/dataengines/geolocation/location_gps.cpp
===================================================================
--- kde-workspace-4.7.2.orig/plasma/generic/dataengines/geolocation/location_gps.cpp
+++ kde-workspace-4.7.2/plasma/generic/dataengines/geolocation/location_gps.cpp
@@ -83,11 +83,11 @@ Gps::Gps(QObject* parent, const QVariant
{
#if GPSD_API_MAJOR_VERSION >= 5
m_gpsdata = new gps_data_t;
- gps_open("localhost", DEFAULT_GPSD_PORT, m_gpsdata);
+ if (gps_open("localhost", DEFAULT_GPSD_PORT, m_gpsdata) != -1) {
#else
gps_data_t* m_gpsdata = gps_open("localhost", DEFAULT_GPSD_PORT);
-#endif
if (m_gpsdata) {
+#endif
kDebug() << "gpsd found.";
m_gpsd = new Gpsd(m_gpsdata);
connect(m_gpsd, SIGNAL(dataReady(const Plasma::DataEngine::Data&)),
++++++ plasma-kickoff-newly-collapsing.diff ++++++
--- /var/tmp/diff_new_pack.OfPSY5/_old 2011-11-14 11:55:35.000000000 +0100
+++ /var/tmp/diff_new_pack.OfPSY5/_new 2011-11-14 11:55:35.000000000 +0100
@@ -1,7 +1,8 @@
-diff -urN kde-workspace-4.7.0.orig/plasma/desktop/applets/kickoff/applet/applet.cpp kde-workspace-4.7.0/plasma/desktop/applets/kickoff/applet/applet.cpp
---- kde-workspace-4.7.0.orig/plasma/desktop/applets/kickoff/applet/applet.cpp 2011-08-02 15:20:44.912157708 +0200
-+++ kde-workspace-4.7.0/plasma/desktop/applets/kickoff/applet/applet.cpp 2011-08-02 15:20:55.501158687 +0200
-@@ -165,6 +165,8 @@
+Index: kde-workspace-4.7.3/plasma/desktop/applets/kickoff/applet/applet.cpp
+===================================================================
+--- kde-workspace-4.7.3.orig/plasma/desktop/applets/kickoff/applet/applet.cpp
++++ kde-workspace-4.7.3/plasma/desktop/applets/kickoff/applet/applet.cpp
+@@ -165,6 +165,8 @@ void LauncherApplet::createConfiguration
d->ui.iconButton->setIcon(popupIcon());
d->ui.switchOnHoverCheckBox->setChecked(d->launcher->switchTabsOnHover());
d->ui.appsByNameCheckBox->setChecked(d->launcher->showAppsByName());
@@ -10,7 +11,7 @@
connect(d->ui.iconButton, SIGNAL(iconChanged(QString)), parent, SLOT(settingsModified()));
connect(d->ui.switchOnHoverCheckBox, SIGNAL(toggled(bool)), parent, SLOT(settingsModified()));
connect(d->ui.appsByNameCheckBox, SIGNAL(toggled(bool)), parent, SLOT(settingsModified()));
-@@ -204,6 +206,9 @@
+@@ -204,6 +206,9 @@ void LauncherApplet::configAccepted()
bool switchTabsOnHover = d->ui.switchOnHoverCheckBox->isChecked();
bool showAppsByName = d->ui.appsByNameCheckBox->isChecked();
@@ -20,7 +21,7 @@
const QString iconname = d->ui.iconButton->icon();
// TODO: should this be moved into Launcher as well? perhaps even the config itself?
-@@ -223,6 +228,9 @@
+@@ -223,6 +228,9 @@ void LauncherApplet::configAccepted()
d->launcher->setSwitchTabsOnHover(switchTabsOnHover);
d->launcher->setShowAppsByName(showAppsByName);
@@ -30,9 +31,10 @@
}
QList LauncherApplet::contextualActions()
-diff -urN kde-workspace-4.7.0.orig/plasma/desktop/applets/kickoff/applet/kickoffConfig.ui kde-workspace-4.7.0/plasma/desktop/applets/kickoff/applet/kickoffConfig.ui
---- kde-workspace-4.7.0.orig/plasma/desktop/applets/kickoff/applet/kickoffConfig.ui 2011-08-02 15:20:44.912157708 +0200
-+++ kde-workspace-4.7.0/plasma/desktop/applets/kickoff/applet/kickoffConfig.ui 2011-08-02 15:20:55.502158687 +0200
+Index: kde-workspace-4.7.3/plasma/desktop/applets/kickoff/applet/kickoffConfig.ui
+===================================================================
+--- kde-workspace-4.7.3.orig/plasma/desktop/applets/kickoff/applet/kickoffConfig.ui
++++ kde-workspace-4.7.3/plasma/desktop/applets/kickoff/applet/kickoffConfig.ui
@@ -54,7 +54,7 @@
</property>
</spacer>
@@ -89,10 +91,11 @@
</layout>
</widget>
<customwidgets>
-diff -urN kde-workspace-4.7.0.orig/plasma/desktop/applets/kickoff/core/applicationmodel.cpp kde-workspace-4.7.0/plasma/desktop/applets/kickoff/core/applicationmodel.cpp
---- kde-workspace-4.7.0.orig/plasma/desktop/applets/kickoff/core/applicationmodel.cpp 2011-08-02 15:20:44.913157708 +0200
-+++ kde-workspace-4.7.0/plasma/desktop/applets/kickoff/core/applicationmodel.cpp 2011-08-02 15:20:55.501158687 +0200
-@@ -109,7 +109,10 @@
+Index: kde-workspace-4.7.3/plasma/desktop/applets/kickoff/core/applicationmodel.cpp
+===================================================================
+--- kde-workspace-4.7.3.orig/plasma/desktop/applets/kickoff/core/applicationmodel.cpp
++++ kde-workspace-4.7.3/plasma/desktop/applets/kickoff/core/applicationmodel.cpp
+@@ -109,7 +109,10 @@ public:
systemApplicationPolicy(ApplicationModel::ShowSystemOnlyPolicy),
primaryNamePolicy(ApplicationModel::GenericNamePrimary),
displayOrder(NameAfterDescription),
@@ -104,7 +107,7 @@
{
systemApplications = Kickoff::systemApplicationList();
reloadTimer = new QTimer(qq);
-@@ -123,6 +126,8 @@
+@@ -123,6 +126,8 @@ public:
}
void fillNode(const QString &relPath, AppNode *node);
@@ -113,7 +116,7 @@
static QHash iconNameMap();
ApplicationModel *q;
-@@ -133,78 +138,117 @@
+@@ -133,78 +138,117 @@ public:
QStringList systemApplications;
DisplayOrder displayOrder;
bool allowSeparators;
@@ -277,7 +280,7 @@
}
if (systemApplicationPolicy == ApplicationModel::ShowSystemOnlyPolicy &&
-@@ -214,9 +258,40 @@
+@@ -214,9 +258,40 @@ void ApplicationModelPrivate::fillNode(c
continue;
}
@@ -320,7 +323,7 @@
if (serviceGroup->noDisplay() || serviceGroup->childCount() == 0) {
continue;
-@@ -225,41 +300,42 @@
+@@ -225,41 +300,42 @@ void ApplicationModelPrivate::fillNode(c
kDebug(250) << "Service group" << serviceGroup->entryPath() << serviceGroup->icon()
<< serviceGroup->relPath() << serviceGroup->directoryEntryPath();
@@ -378,7 +381,7 @@
}
// set the subTitleMandatory field for nodes that do not provide a unique generic
-@@ -274,6 +350,20 @@
+@@ -274,6 +350,20 @@ void ApplicationModelPrivate::fillNode(c
}
}
@@ -399,7 +402,7 @@
ApplicationModel::ApplicationModel(QObject *parent, bool allowSeparators)
: KickoffAbstractModel(parent),
d(new ApplicationModelPrivate(this, allowSeparators))
-@@ -283,6 +373,7 @@
+@@ -283,6 +373,7 @@ ApplicationModel::ApplicationModel(QObje
QDBusConnection::sessionBus().registerObject("/kickoff", this);
dbus.connect(QString(), "/kickoff", "org.kde.plasma", "reloadMenu", this, SLOT(reloadMenu()));
connect(KSycoca::self(), SIGNAL(databaseChanged(QStringList)), this, SLOT(checkSycocaChange(QStringList)));
@@ -407,7 +410,7 @@
}
ApplicationModel::~ApplicationModel()
-@@ -309,6 +400,32 @@
+@@ -309,6 +400,32 @@ DisplayOrder ApplicationModel::nameDispl
return d->displayOrder;
}
@@ -440,7 +443,7 @@
int ApplicationModel::columnCount(const QModelIndex &parent) const
{
Q_UNUSED(parent)
-@@ -516,8 +633,9 @@
+@@ -516,8 +633,9 @@ void ApplicationModel::reloadMenu()
{
delete d->root;
d->root = new AppNode();
@@ -451,7 +454,7 @@
}
void ApplicationModel::checkSycocaChange(const QStringList &changes)
-@@ -537,6 +655,104 @@
+@@ -537,6 +655,104 @@ ApplicationModel::SystemApplicationPolic
return d->systemApplicationPolicy;
}
@@ -556,10 +559,11 @@
/**
* FIXME This is a temporary workaround to map the icon names found
* in the desktop directory files (from /usr/share/desktop-directories)
-diff -urN kde-workspace-4.7.0.orig/plasma/desktop/applets/kickoff/core/applicationmodel.h kde-workspace-4.7.0/plasma/desktop/applets/kickoff/core/applicationmodel.h
---- kde-workspace-4.7.0.orig/plasma/desktop/applets/kickoff/core/applicationmodel.h 2011-08-02 15:20:44.913157708 +0200
-+++ kde-workspace-4.7.0/plasma/desktop/applets/kickoff/core/applicationmodel.h 2011-08-02 15:20:55.501158687 +0200
-@@ -114,10 +114,19 @@
+Index: kde-workspace-4.7.3/plasma/desktop/applets/kickoff/core/applicationmodel.h
+===================================================================
+--- kde-workspace-4.7.3.orig/plasma/desktop/applets/kickoff/core/applicationmodel.h
++++ kde-workspace-4.7.3/plasma/desktop/applets/kickoff/core/applicationmodel.h
+@@ -114,10 +114,19 @@ public:
virtual QModelIndex parent(const QModelIndex &index) const;
virtual int rowCount(const QModelIndex &parent = QModelIndex()) const;
@@ -579,7 +583,7 @@
private:
bool nameAfterDescription(const QModelIndex &index) const;
-@@ -125,6 +134,8 @@
+@@ -125,6 +134,8 @@ private:
friend class ApplicationModelPrivate;
ApplicationModelPrivate *const d;
@@ -588,10 +592,11 @@
Q_DISABLE_COPY(ApplicationModel)
};
-diff -urN kde-workspace-4.7.0.orig/plasma/desktop/applets/kickoff/simpleapplet/simpleapplet.cpp kde-workspace-4.7.0/plasma/desktop/applets/kickoff/simpleapplet/simpleapplet.cpp
---- kde-workspace-4.7.0.orig/plasma/desktop/applets/kickoff/simpleapplet/simpleapplet.cpp 2011-08-02 15:20:44.913157708 +0200
-+++ kde-workspace-4.7.0/plasma/desktop/applets/kickoff/simpleapplet/simpleapplet.cpp 2011-08-02 15:20:55.502158687 +0200
-@@ -114,12 +114,17 @@
+Index: kde-workspace-4.7.3/plasma/desktop/applets/kickoff/simpleapplet/simpleapplet.cpp
+===================================================================
+--- kde-workspace-4.7.3.orig/plasma/desktop/applets/kickoff/simpleapplet/simpleapplet.cpp
++++ kde-workspace-4.7.3/plasma/desktop/applets/kickoff/simpleapplet/simpleapplet.cpp
+@@ -114,12 +114,17 @@ public:
MenuLauncherApplet::FormatType formattype;
int maxRecentApps;
bool showMenuTitles;
@@ -609,7 +614,7 @@
QList actions;
QAction* switcher;
-@@ -137,6 +142,8 @@
+@@ -137,6 +142,8 @@ public:
iconButton(0),
formatComboBox(0),
showMenuTitlesCheckBox(0),
@@ -618,7 +623,7 @@
switcher(0),
contextMenuFactory(0)
{}
-@@ -485,7 +492,19 @@
+@@ -485,7 +492,19 @@ void MenuLauncherApplet::createConfigura
d->showMenuTitlesCheckBox->setChecked(d->showMenuTitles);
grid->addWidget(d->showMenuTitlesCheckBox, 3, 1);
@@ -639,7 +644,7 @@
parent->addPage(p, i18n("Options"), "configure");
connect(parent, SIGNAL(applyClicked()), this, SLOT(configAccepted()));
-@@ -551,6 +570,19 @@
+@@ -551,6 +570,19 @@ void MenuLauncherApplet::configAccepted(
d->showMenuTitles = showMenuTitles;
cg.writeEntry("showMenuTitles", showMenuTitles);
}
@@ -659,8 +664,8 @@
if (needssaving) {
d->updateTooltip();
-@@ -618,6 +650,9 @@
- appModel->setPrimaryNamePolicy(Kickoff::ApplicationModel::AppNamePrimary);
+@@ -618,6 +650,9 @@ void MenuLauncherApplet::showMenu(bool p
+ }
appModel->setSystemApplicationPolicy(Kickoff::ApplicationModel::ShowApplicationAndSystemPolicy);
+ appModel->setShowRecentlyInstalled(d->showRecentlyInstalled);
@@ -669,7 +674,7 @@
menuview->addModel(appModel, Kickoff::MenuView::None, d->relativePath);
if (d->relativePath.isEmpty()) {
-@@ -862,6 +897,8 @@
+@@ -873,6 +908,8 @@ void MenuLauncherApplet::configChanged()
d->setMaxRecentApps(cg.readEntry("maxRecentApps", qMin(5, Kickoff::RecentApplications::self()->maximum())));
d->showMenuTitles = cg.readEntry("showMenuTitles", false);
@@ -678,10 +683,11 @@
d->icon->setIcon(KIcon(cg.readEntry("icon", d->iconname)));
-diff -urN kde-workspace-4.7.0.orig/plasma/desktop/applets/kickoff/ui/launcher.cpp kde-workspace-4.7.0/plasma/desktop/applets/kickoff/ui/launcher.cpp
---- kde-workspace-4.7.0.orig/plasma/desktop/applets/kickoff/ui/launcher.cpp 2011-08-02 15:20:44.912157708 +0200
-+++ kde-workspace-4.7.0/plasma/desktop/applets/kickoff/ui/launcher.cpp 2011-08-02 15:20:55.502158687 +0200
-@@ -680,6 +680,30 @@
+Index: kde-workspace-4.7.3/plasma/desktop/applets/kickoff/ui/launcher.cpp
+===================================================================
+--- kde-workspace-4.7.3.orig/plasma/desktop/applets/kickoff/ui/launcher.cpp
++++ kde-workspace-4.7.3/plasma/desktop/applets/kickoff/ui/launcher.cpp
+@@ -680,6 +680,30 @@ void Launcher::setShowAppsByName(bool sh
}
}
@@ -712,7 +718,7 @@
bool Launcher::switchTabsOnHover() const
{
return d->contentSwitcher->switchTabsOnHover();
-@@ -690,6 +714,16 @@
+@@ -690,6 +714,16 @@ bool Launcher::showAppsByName() const
return d->applicationModel->nameDisplayOrder() == Kickoff::NameBeforeDescription;
}
@@ -729,7 +735,7 @@
void Launcher::setVisibleItemCount(int count)
{
d->visibleItemCount = count;
-@@ -708,6 +742,9 @@
+@@ -708,6 +742,9 @@ void Launcher::setApplet(Plasma::Applet
cg = applet->config();
setShowAppsByName(cg.readEntry("ShowAppsByName", showAppsByName()));
setVisibleItemCount(cg.readEntry("VisibleItemsCount", visibleItemCount()));
@@ -739,10 +745,11 @@
d->applet = applet;
d->contextMenuFactory->setApplet(applet);
-diff -urN kde-workspace-4.7.0.orig/plasma/desktop/applets/kickoff/ui/launcher.h kde-workspace-4.7.0/plasma/desktop/applets/kickoff/ui/launcher.h
---- kde-workspace-4.7.0.orig/plasma/desktop/applets/kickoff/ui/launcher.h 2011-08-02 15:20:44.912157708 +0200
-+++ kde-workspace-4.7.0/plasma/desktop/applets/kickoff/ui/launcher.h 2011-08-02 15:20:55.502158687 +0200
-@@ -78,6 +78,14 @@
+Index: kde-workspace-4.7.3/plasma/desktop/applets/kickoff/ui/launcher.h
+===================================================================
+--- kde-workspace-4.7.3.orig/plasma/desktop/applets/kickoff/ui/launcher.h
++++ kde-workspace-4.7.3/plasma/desktop/applets/kickoff/ui/launcher.h
+@@ -78,6 +78,14 @@ public:
we don't display old searches and switch back to the favorite-view. */
void reset();
--
To unsubscribe, e-mail: opensuse-commit+unsubscribe@opensuse.org
For additional commands, e-mail: opensuse-commit+help@opensuse.org