[prev in list] [next in list] [prev in thread] [next in thread]
List: kde-release-team
Subject: =?utf-8?Q?Re:=20KDE=20Frameworks=C2=B75.11.0:=20trashcan=20applet=20fails?=
From: Sebastian Kügler <sebas () kde ! org>
Date: 2015-06-18 0:33:59
Message-ID: 1610302.YQW5P9ZUTZ () monet
[Download RAW message or body]
On Tuesday, June 16, 2015 18:06:06 Bhushan Shah wrote:
> On Tue, Jun 16, 2015 at 8:33 AM, Rex Dieter <rdieter@math.unl.edu> wrote:
> > Yes, per the aforementioned bug reports, trying to place the trashcan
> > applet yields:
> >
> > Error loading Applet: package inexistent
>
> I debugged this little bit and found that trash applet have
> X-KDE-Library set in its metadata.desktop which is unused. I removed
> it now however I see that if condition added in
> 462ad8a6cae22bc2855c8e4461ca503d6fd15884,
>
> + if (!offer->property("X-Plasma-API").toString().isEmpty() &&
> + offer->property("Library").toString().isEmpty()) {
>
> is removed in ea924b14691da3819ca17d876f63ffc686fcf840 as I think
> there is no replacement for this in KPluginMetadata or something..
>
> If we want to make it backwards compatible this case should be handled
> somehow or otherwise any other desktop files which are ported to QML
> and have stray X-KDE-Library needs fixing.
There's not reason why this can't be done (in a similar way) with
KPluginMetaData, you can go through KPluginMetaData::rawData() to look up
these keys.
--
sebas
Sebastian Kügler | http://vizZzion.org | http://kde.org
_______________________________________________
release-team mailing list
release-team@kde.org
https://mail.kde.org/mailman/listinfo/release-team
[prev in list] [next in list] [prev in thread] [next in thread]
Configure |
About |
News |
Add a list |
Sponsored by KoreLogic