Re: plasma-framework in kdereview

2014-04-25 Thread David Edmundson
In Plasma::Service there's a method called dummyGroup() which will always return an empty KConfigGroup in an overly complicated way. It seems to be completely unused. Any idea what it might be for? ___ Plasma-devel mailing list Plasma-devel@kde.org https

Re: plasma-framework in kdereview

2014-04-25 Thread Guillaume DE BURE
Hi Everyone, Indeed, there was an intention to use some plasma components in Skrooge, but it never materialized in any official release. Today, this is mostly abandoned code, to my disappointment, but I never committed myself enough on finishing this part... As far as Skrooge is concerned, you

Re: plasma-framework in kdereview

2014-04-25 Thread Marco Martin
about that, would be me to blame. I did the integration into KConfig. > No idea how it could happen that you weren't aware of it. And I also don't > remember why I didn't adjust plasma-framework, but I assume that it was too > much in flux at that time to do such a change. S

Re: Re: plasma-framework in kdereview

2014-04-25 Thread David Edmundson
KConfig. No > idea how it could happen that you weren't aware of it. And I also don't > remember why I didn't adjust plasma-framework, but I assume that it was too > much in flux at that time to do such a change. > FYI I'm on it now. _

Re: Re: plasma-framework in kdereview

2014-04-25 Thread Martin Gräßlin
x27;t notified/things weren't ported to it? sorry about that, would be me to blame. I did the integration into KConfig. No idea how it could happen that you weren't aware of it. And I also don't remember why I didn't adjust plasma-framework, but I assume that it was too muc

Re: plasma-framework in kdereview

2014-04-25 Thread Marco Martin
On Friday 25 April 2014 19:16:43 Kevin Ottens wrote: > I think these statements show you totally ignore the history behind > libplasma or how applications can use it... They (at least Amarok, not 100% > sure for Skrooge) benefit from the component model used in libplasma: > packages, dataengines,

Re: plasma-framework in kdereview

2014-04-25 Thread Marco Martin
On Friday 25 April 2014 19:28:59 David Edmundson wrote: > > parse kconfigskeletons at runtime for what i'm concerned), that's the call > > of the application developer, as any workspace. > > That KConfigLoader already moved to KConfigGui. > > (and I agree that class is really really useful) argh

Re: plasma-framework in kdereview

2014-04-25 Thread David Edmundson
On Fri, Apr 25, 2014 at 6:56 PM, Marco Martin wrote: > On Friday 25 April 2014 17:46:23 David Edmundson wrote: >> > Well... it's been planned this way for three years if not more. Before >> > that it was in kdelibs. >> > >> >> Also, right now there is only one user of this framework >> >> (plasma-

Re: plasma-framework in kdereview

2014-04-25 Thread Kevin Ottens
On Friday 25 April 2014 17:46:23 David Edmundson wrote: > > Well... it's been planned this way for three years if not more. Before > > that it was in kdelibs. > > > >> Also, right now there is only one user of this framework > >> (plasma-desktop), > > > > That's because the other users weren't po

Re: plasma-framework in kdereview

2014-04-25 Thread Marco Martin
On Friday 25 April 2014 17:46:23 David Edmundson wrote: > > Well... it's been planned this way for three years if not more. Before > > that it was in kdelibs. > > > >> Also, right now there is only one user of this framework > >> (plasma-desktop), > > > > That's because the other users weren't po

Re: plasma-framework in kdereview

2014-04-25 Thread David Edmundson
> Well... it's been planned this way for three years if not more. Before that it > was in kdelibs. > >> Also, right now there is only one user of this framework (plasma-desktop), > > That's because the other users weren't ported to KF5 yet. But there's > definitely more plasma users (amarok comes t

Re: plasma-framework in kdereview

2014-04-25 Thread Kevin Ottens
On Friday 25 April 2014 15:14:46 Àlex Fiestas wrote: > On Friday 25 April 2014 12:34:32 Marco Martin wrote: > > Hi all, > > since it was done earlier this week, better announce it formally, so > > everybody can actually do the -review part ;) > > > > the plasma-fr

Re: plasma-framework in kdereview

2014-04-25 Thread Marco Martin
On Friday 25 April 2014 15:24:50 Luigi Toscano wrote: > On Friday 25 of April 2014 15:14:46 Àlex Fiestas wrote: > > Moving plasma-framework to frameworks means that we will loose flexibility > > since we won't be able to break api/abi. > > > > So, do we really have

Re: plasma-framework in kdereview

2014-04-25 Thread Luigi Toscano
On Friday 25 of April 2014 15:14:46 Àlex Fiestas wrote: > Moving plasma-framework to frameworks means that we will loose flexibility > since we won't be able to break api/abi. > > So, do we really have to move it there? Imho would be prudent to keep it > somewhere else where

Re: plasma-framework in kdereview

2014-04-25 Thread Marco Martin
On Friday 25 April 2014 15:14:46 you wrote: > > * there was the plasma shell: has been removed and moved to > > plasma-workspace, decreasing dependencies > > Moving plasma-framework to frameworks means that we will loose flexibility > since we won't be able to break ap

Re: plasma-framework in kdereview

2014-04-25 Thread Àlex Fiestas
On Friday 25 April 2014 12:34:32 Marco Martin wrote: > Hi all, > since it was done earlier this week, better announce it formally, so > everybody can actually do the -review part ;) > > the plasma-framework repository has been moved in kdereview, headed > hopefully in fra

plasma-framework in kdereview

2014-04-25 Thread Marco Martin
Hi all, since it was done earlier this week, better announce it formally, so everybody can actually do the -review part ;) the plasma-framework repository has been moved in kdereview, headed hopefully in frameworks. what it contains: * libplasma: it's the old plasma library that used to

Re: Review Request 117723: Adapt the PanelView to the new item status in plasma-framework

2014-04-24 Thread Aleix Pol Gonzalez
--- This is an automatically generated e-mail. To reply, visit: https://git.reviewboard.kde.org/r/117723/ --- (Updated April 24, 2014, 2:58 p.m.) Status -- This change has been m

Re: Review Request 117723: Adapt the PanelView to the new item status in plasma-framework

2014-04-24 Thread Commit Hook
--- This is an automatically generated e-mail. To reply, visit: https://git.reviewboard.kde.org/r/117723/#review56392 --- This review has been submitted with commit 3df9ed42c0d9701471

Re: Review Request 117723: Adapt the PanelView to the new item status in plasma-framework

2014-04-23 Thread Marco Martin
--- This is an automatically generated e-mail. To reply, visit: https://git.reviewboard.kde.org/r/117723/#review56300 --- Ship it! Ship It! - Marco Martin On April 23, 2014, 4:28 p

Review Request 117723: Adapt the PanelView to the new item status in plasma-framework

2014-04-23 Thread Aleix Pol Gonzalez
--- This is an automatically generated e-mail. To reply, visit: https://git.reviewboard.kde.org/r/117723/ --- Review request for Plasma. Repository: plasma-workspace Description ---

Re: APIDox for Plasma Framework

2014-04-15 Thread Aleix Pol
On Tue, Apr 15, 2014 at 3:28 PM, Aurélien Gâteau wrote: > Alex Merry wrote: > > > On 12/04/14 12:48, Aleix Pol wrote: > >> Hi, > >> I just realized that we're not generating the API documentation for > >> Plasma Framework here [1]. > >> &g

APIDox for Plasma Framework

2014-04-12 Thread Aleix Pol
Hi, I just realized that we're not generating the API documentation for Plasma Framework here [1]. Maybe it would be worth adding? Aleix [1] http://api.kde.org/frameworks-api/frameworks5-apidocs/ ___ Plasma-devel mailing list Plasma-devel@kd

Re: Review Request 115475: Split KRunner out of plasma-framework

2014-02-28 Thread Aleix Pol Gonzalez
marked as submitted. Review request for Plasma. Repository: plasma-framework Description --- Splitted out KRunner so we can use it from any solution and eventually move out from it, if required. It can be found now in kde:scratch/apol/krunner. Diffs - src/plasma/includes/Plasma

Re: Review Request 115475: Split KRunner out of plasma-framework

2014-02-04 Thread David Edmundson
tests, history is less important. > > David Edmundson wrote: > It has history via git graft (tested here) > > Marco Martin wrote: > how is done? > just cloning the new repository it has exactly one commit path/to/qt5/qtrepotools/bin/git-qt-grafts /path/to/plasma-f

Re: Review Request 115475: Split KRunner out of plasma-framework

2014-02-04 Thread Marco Martin
atically generated e-mail. To reply, visit: > https://git.reviewboard.kde.org/r/115475/ > --- > > (Updated Feb. 4, 2014, 3:33 p.m.) > > > Review request for Plasma. > > > Repository: plasma-framework > > > Descrip

Re: Review Request 115475: Split KRunner out of plasma-framework

2014-02-04 Thread David Edmundson
--- > > (Updated Feb. 4, 2014, 3:33 p.m.) > > > Review request for Plasma. > > > Repository: plasma-framework > > > Description > --- > > Splitted out KRunner so we can use it from any solution and eventually move >

Re: Review Request 115475: Split KRunner out of plasma-framework

2014-02-04 Thread David Edmundson
ed Feb. 4, 2014, 3:33 p.m.) > > > Review request for Plasma. > > > Repository: plasma-framework > > > Description > --- > > Splitted out KRunner so we can use it from any solution and eventually move > out from it, if required. > &

Re: Review Request 115475: Split KRunner out of plasma-framework

2014-02-04 Thread Marco Martin
: > > --- > This is an automatically generated e-mail. To reply, visit: > https://git.reviewboard.kde.org/r/115475/ > --- > > (Updated Feb. 4, 2014, 3:33 p.m.) > > > Review request for Plasma. > > > Repositor

Review Request 115475: Split KRunner out of plasma-framework

2014-02-04 Thread Aleix Pol Gonzalez
--- This is an automatically generated e-mail. To reply, visit: https://git.reviewboard.kde.org/r/115475/ --- Review request for Plasma. Repository: plasma-framework Description

[plasma-framework] src/declarativeimports: units.longDuration and units.shortDuration properties

2014-02-03 Thread Sebastian Kügler
ations.js http://commits.kde.org/plasma-framework/30e98ba8f7646fcceaf85cc808e05fcc88b66442 diff --git a/src/declarativeimports/core/units.cpp b/src/declarativeimports/core/units.cpp index bb40901..ae4c134 100644 --- a/src/declarativeimports/core/units.cpp +++ b/src/declarativeimports/core/uni

Re: Compiling plasma-framework and KDE5

2014-02-01 Thread Sebastian Kügler
Hi Rishav, On Friday, January 31, 2014 12:59:06 Rishav Thakker wrote: > Hi > > I'm trying to build KDE5, but I get stuck while compiling plasma-framework, > kdewebkit, and a few other packages. I'm following instructions given here: > > http://community.kde.org/Fra

Re: Compiling plasma-framework and KDE5

2014-02-01 Thread Bhushan Shah
Hello, On Fri, Jan 31, 2014 at 12:59 PM, Rishav Thakker wrote: >> rishav@rishav-laptop:~/plasma-framework/build$ cmake >> -DCMAKE_INSTALL_PREFIX=$KF5 -DCMAKE_PREFIX_PATH=$KF5 .. >> >> CMake Error at CMakeLists.txt:52 (find_package): >> >> Could not find a pa

Re: Compiling plasma-framework and KDE5

2014-02-01 Thread Heena Mahour
have you updated Qt5? On Fri, Jan 31, 2014 at 12:59 PM, Rishav Thakker wrote: > Hi > > I'm trying to build KDE5, but I get stuck while compiling > plasma-framework, kdewebkit, and a few other packages. I'm following > instructions given here: > > http://commun

Compiling plasma-framework and KDE5

2014-02-01 Thread Rishav Thakker
Hi I'm trying to build KDE5, but I get stuck while compiling plasma-framework, kdewebkit, and a few other packages. I'm following instructions given here: http://community.kde.org/Frameworks/Building I even tried manually building the individual packages, and that's when i found

find_package(KActivities) in Plasma Framework

2014-01-22 Thread David Narvaez
Hi, I see that CMakeLists.cmake in plasma-framework has find_package(KActivities 5.0.0 CONFIG REQUIRED) and I assume 5.0.0 there means "find the framework version", but the KDE4 version of KActivites is actually 6.2.0 which is OK for that find_package so when I tried building plasma

[plasma-framework] src/shell: [plasma-shell] Add an option to suppress any output (--shut-up)

2013-12-17 Thread Àlex Fiestas
roject.org/#change,73943 CCMAIL: plasma-devel@kde.org M +12 -0src/shell/main.cpp http://commits.kde.org/plasma-framework/c047dd5f686c5850fdb2638854885bd76151ef93 diff --git a/src/shell/main.cpp b/src/shell/main.cpp index f6c797d..e073d00 100644 --- a/src/shell/main.cpp +++ b/src/shell/main.cp

[plasma-framework] src/shell: [plasma-shell] Mute all debug output when started from autostart

2013-12-17 Thread Àlex Fiestas
t drown on warnings. CCMAIL: plasma-devel@kde.org M +1-1src/shell/plasma-shell.desktop http://commits.kde.org/plasma-framework/dfcfad1182e51d5165bd8d289a33253aba7b8776 diff --git a/src/shell/plasma-shell.desktop b/src/shell/plasma-shell.desktop index f6c4229..0d6d231 100644 --- a/src/sh

Re: Review Request 113831: Make plasma-framework build without tests

2013-11-13 Thread Bhushan Shah
marked as submitted. Review request for Plasma and Sebastian Kügler. Repository: plasma-framework Description --- Without this patch, when we try to compile plasma-framework without tests using -DBUILD_TESTING it causes error. Diffs - src/plasma/autotests/CMakeLists.txt 7863cf3

Re: Review Request 113831: Make plasma-framework build without tests

2013-11-13 Thread Commit Hook
org/r/113831/ > --- > > (Updated Nov. 13, 2013, 6:44 a.m.) > > > Review request for Plasma and Sebastian Kügler. > > > Repository: plasma-framework > > > Description > --- > > Without this pat

Re: Review Request 113831: Make plasma-framework build without tests

2013-11-13 Thread Sebastian Kügler
v. 13, 2013, 6:44 a.m.) > > > Review request for Plasma and Sebastian Kügler. > > > Repository: plasma-framework > > > Description > --- > > Without this patch, when we try to compile plasma-framework without tests > using -DBUILD_TESTING it cau

Review Request 113831: Make plasma-framework build without tests

2013-11-12 Thread Bhushan Shah
-framework Description --- Without this patch, when we try to compile plasma-framework without tests using -DBUILD_TESTING it causes error. Diffs - src/plasma/autotests/CMakeLists.txt 7863cf3 Diff: http://git.reviewboard.kde.org/r/113831/diff/ Testing --- cmake ... -DBUILD_TESTING

plasma-framework cmake issue?

2013-10-28 Thread Giorgos Tsiapaliokas
Hello, I am having a KDE4Support cmake issue in plasma-framework. I have attached the output of cmake. Do you know what is going wrong? thanks in advance -- Giorgos Tsiapaliokas (terietor) terietor.org cmake -DCMAKE_INSTALL_PREFIX=$KF5 .. -- The C compiler identification is GNU 4.8.1

Re: Plasma-framework cmake issue

2013-09-26 Thread Sebastian Kügler
On Thursday, September 26, 2013 15:24:46 Sebastian Kügler wrote: > Hi Antonis, > > On Thursday, September 26, 2013 16:10:21 Antonis Tsiapaliokas wrote: > > I was trying to build plasma-framework but cmake fails. > > The installation was with a clean install/build dir of ev

Re: Plasma-framework cmake issue

2013-09-26 Thread Sebastian Kügler
Hi Antonis, On Thursday, September 26, 2013 16:10:21 Antonis Tsiapaliokas wrote: > I was trying to build plasma-framework but cmake fails. > The installation was with a clean install/build dir of everything. > > If you need more info, please let me know... > > P.S. I have

Plasma-framework cmake issue

2013-09-26 Thread Antonis Tsiapaliokas
Hello, I was trying to build plasma-framework but cmake fails. The installation was with a clean install/build dir of everything. If you need more info, please let me know... P.S. I have attached the cmake error. Regards, Antonis error-cmake Description: Binary data

Re: [plasma-framework] src/plasma: Look up dataengines in kf5/plasma/dataengines

2013-09-25 Thread Aaron J. Seigo
On Thursday, September 26, 2013 04:21:21 Sebastian Kügler wrote: > You can find numbers of the SSD runs in my email "Plugin locator performance > ballpark" to kde-frameworks-devel from September, 5. I haven't cleaned up > my measurements on rotating metal. oh wow, that is not encouraging. :/ i’m s

Re: [plasma-framework] src/plasma: Look up dataengines in kf5/plasma/dataengines

2013-09-25 Thread Sebastian Kügler
On Thursday, September 26, 2013 01:50:54 Aaron J. Seigo wrote: > On Wednesday, September 25, 2013 23:43:43 Sebastian Kügler wrote: > > This changes the path where to find dataengines to the > > subdirectory-per-servicetype setup. > > for shit’s ‘n giggles: has anyone done any performance profiling

Re: [plasma-framework] src/plasma: Look up dataengines in kf5/plasma/dataengines

2013-09-25 Thread Aaron J. Seigo
On Wednesday, September 25, 2013 23:43:43 Sebastian Kügler wrote: > This changes the path where to find dataengines to the > subdirectory-per-servicetype setup. for shit’s ‘n giggles: has anyone done any performance profiling of this new system? -- Aaron J. Seigo ___

[plasma-framework] src/plasma: Look up dataengines in kf5/plasma/dataengines

2013-09-25 Thread Sebastian Kügler
te: You'll need to update and reinstall kde-workspace/plasma, to install the dataengines into the new path. CCMAIL:plasma-devel@kde.org M +1-1src/plasma/pluginloader.cpp http://commits.kde.org/plasma-framework/5b33844a0b9e49759289dc839a31c0bc8f417d37 diff --git a/src/plasma/pluginl

Re: Review Request 111908: Add KConfigLoader from Plasma Framework to KConfigGui

2013-09-16 Thread Commit Hook
marked as submitted. Review request for KDE Frameworks, Plasma and Aaron J. Seigo. Description --- Add KConfigLoader from Plasma Framework to KConfigGui The ConfigLoader is way to awesome to not be directly in KConfig. Diffs - tier1/kconfig/autotests/CMakeLists.txt c913da3 tier1

Re: Review Request 111908: Add KConfigLoader from Plasma Framework to KConfigGui

2013-09-16 Thread Commit Hook
org/r/111908/ > --- > > (Updated Aug. 9, 2013, 12:38 p.m.) > > > Review request for KDE Frameworks, Plasma and Aaron J. Seigo. > > > Description > --- > > Add KConfigLoader from Plasma Framewo

Re: plasma framework compile errors

2013-09-14 Thread Heena Mahour
builds now . On Sat, Sep 14, 2013 at 2:41 PM, Heena Mahour wrote: > Hi > I have build kde libs though I got a error regarding kdetoolstarget but it > did build out too . http://pastebin.com/raw.php?i=PfgBiJft > Here is my plasma framework error http://pastebin.com/raw.php?i=qth

plasma framework compile errors

2013-09-14 Thread Heena Mahour
Hi I have build kde libs though I got a error regarding kdetoolstarget but it did build out too . http://pastebin.com/raw.php?i=PfgBiJft Here is my plasma framework error http://pastebin.com/raw.php?i=qthyDrue Do let me know where I am going wrong .. Regards -- -Heena Season of kde&#x

Re: Review Request 111908: Add KConfigLoader from Plasma Framework to KConfigGui

2013-09-12 Thread David Faure
g. 9, 2013, 12:38 p.m.) > > > Review request for KDE Frameworks, Plasma and Aaron J. Seigo. > > > Description > --- > > Add KConfigLoader from Plasma Framework to KConfigGui > > The ConfigLoader is way to awesome to not be directly in KConfig. > > &

Re: Review Request 111908: Add KConfigLoader from Plasma Framework to KConfigGui

2013-09-11 Thread Martin Gräßlin
-------- > > (Updated Aug. 9, 2013, 2:38 p.m.) > > > Review request for KDE Frameworks, Plasma and Aaron J. Seigo. > > > Description > --- > > Add KConfigLoader from Plasma Framework to KConfigGui > > The ConfigLoader is way to aw

Examples for plasma-framework [was: documentation on QMLSceneGraph internals]

2013-09-04 Thread Marco Martin
Hi all, I just want to give an heads up on a little reorganization happened in repos: All the examples of plasma that were relevant to plasma2 have been moved in the plasma-framework repository, under the example/ folder * the move has been done to conform the guidelines in https

Re: Review Request 111908: Add KConfigLoader from Plasma Framework to KConfigGui

2013-09-03 Thread David Faure
---- > > (Updated Aug. 9, 2013, 12:38 p.m.) > > > Review request for KDE Frameworks, Plasma and Aaron J. Seigo. > > > Description > --- > > Add KConfigLoader from Plasma Framework to KConfigGui > > The ConfigLoader

Re: Review Request 111908: Add KConfigLoader from Plasma Framework to KConfigGui

2013-09-02 Thread Martin Gräßlin
--- > This is an automatically generated e-mail. To reply, visit: > http://git.reviewboard.kde.org/r/111908/ > --- > > (Updated Aug. 9, 2013, 2:38 p.m.) > > > Review request for KDE Frameworks, Pl

Force push to plasma-framework

2013-08-31 Thread Ben Cooksley
Hi everyone, Due to an unfortunate accident involving git merge, several branches of the plasma-framework repository were rendered unusable. To correct this, two branches have been forcibly rewound. They are: - master: now at ea1b637 - ivan/shell-switching: now at f8c2ff5. The following commits

Re: plasma-framework cmake error after git pull --rebase

2013-08-14 Thread Heena Mahour
;> On Tuesday 13 August 2013 16:26:08 Heena Mahour wrote: >>> > Hi , >>> > In order to make use of data engine tasks in plasma 2 .I did git pull >>> > --rebase in extra-cmake-module and kdelibs and rebuild it using runtime >>> > setup .Then I di

Re: plasma-framework cmake error after git pull --rebase

2013-08-14 Thread Heena Mahour
ld it using runtime >> > setup .Then I did the same with plasma-framework .But getting cmake >> error >> > on plasma-framework http://pastebin.com/raw.php?i=d7apCJkn >> > Please help me out :p (plasma framework takes time ;) ) >> >> try to add -DCMAKE_PR

Re: plasma-framework cmake error after git pull --rebase

2013-08-14 Thread Heena Mahour
odule and kdelibs and rebuild it using runtime > > setup .Then I did the same with plasma-framework .But getting cmake error > > on plasma-framework http://pastebin.com/raw.php?i=d7apCJkn > > Please help me out :p (plasma framework takes time ;) ) > > try to add -DCMAKE_PREFIX_PAT

Re: plasma-framework cmake error after git pull --rebase

2013-08-13 Thread Marco Martin
On Tuesday 13 August 2013 16:26:08 Heena Mahour wrote: > Hi , > In order to make use of data engine tasks in plasma 2 .I did git pull > --rebase in extra-cmake-module and kdelibs and rebuild it using runtime > setup .Then I did the same with plasma-framework .But getting cmake error

plasma-framework cmake error after git pull --rebase

2013-08-13 Thread Heena Mahour
Hi , In order to make use of data engine tasks in plasma 2 .I did git pull --rebase in extra-cmake-module and kdelibs and rebuild it using runtime setup .Then I did the same with plasma-framework .But getting cmake error on plasma-framework http://pastebin.com/raw.php?i=d7apCJkn Please help me out

Re: Review Request 111908: Add KConfigLoader from Plasma Framework to KConfigGui

2013-08-09 Thread David Faure
s an automatically generated e-mail. To reply, visit: > http://git.reviewboard.kde.org/r/111908/ > --- > > (Updated Aug. 9, 2013, 12:38 p.m.) > > > Review request for KDE Frameworks, Plasma and Aaron J. Seigo. > >

Re: Review Request 111908: Add KConfigLoader from Plasma Framework to KConfigGui

2013-08-09 Thread David Edmundson
an automatically generated e-mail. To reply, visit: > http://git.reviewboard.kde.org/r/111908/ > --- > > (Updated Aug. 9, 2013, 12:38 p.m.) > > > Review request for KDE Frameworks, Plasma and Aaron J. Seigo. > > > Descri

Re: Review Request 111908: Add KConfigLoader from Plasma Framework to KConfigGui

2013-08-09 Thread David Edmundson
Updated Aug. 9, 2013, 12:38 p.m.) > > > Review request for KDE Frameworks, Plasma and Aaron J. Seigo. > > > Description > --- > > Add KConfigLoader from Plasma Framework to KConfigGui > > The ConfigLoader is way to awesome to not be directly in KConf

Re: Review Request 111908: Add KConfigLoader from Plasma Framework to KConfigGui

2013-08-09 Thread Kai Uwe Broulik
t; > (Updated Aug. 9, 2013, 12:38 p.m.) > > > Review request for KDE Frameworks, Plasma and Aaron J. Seigo. > > > Description > --- > > Add KConfigLoader from Plasma Framework to KConfigGui > > The ConfigLoader is way to awesome to not be directly in KCon

Re: Review Request 111908: Add KConfigLoader from Plasma Framework to KConfigGui

2013-08-09 Thread David Edmundson
enerated e-mail. To reply, visit: > http://git.reviewboard.kde.org/r/111908/ > --- > > (Updated Aug. 9, 2013, 12:38 p.m.) > > > Review request for KDE Frameworks, Plasma and Aaron J. Seigo. > > > Description

Re: Review Request 111908: Add KConfigLoader from Plasma Framework to KConfigGui

2013-08-09 Thread Martin Gräßlin
, Plasma and Aaron J. Seigo. Changes --- const ref instead of pointer Description --- Add KConfigLoader from Plasma Framework to KConfigGui The ConfigLoader is way to awesome to not be directly in KConfig. Diffs (updated) - tier1/kconfig/autotests/CMakeLists.txt c913da3 tier1

Re: Review Request 111908: Add KConfigLoader from Plasma Framework to KConfigGui

2013-08-09 Thread Kevin Ottens
request for KDE Frameworks, Plasma and Aaron J. Seigo. > > > Description > --- > > Add KConfigLoader from Plasma Framework to KConfigGui > > The ConfigLoader is way to awesome to not be directly in KConfig. > > > Diffs > - > > tier1/kconf

Re: Review Request 111908: Add KConfigLoader from Plasma Framework to KConfigGui

2013-08-08 Thread Martin Gräßlin
, Plasma and Aaron J. Seigo. Changes --- fixed the issues outlined by David E. and Kevin Description --- Add KConfigLoader from Plasma Framework to KConfigGui The ConfigLoader is way to awesome to not be directly in KConfig. Diffs (updated) - tier1/kconfig/autotests/CMakeLists.txt

Re: Review Request 111908: Add KConfigLoader from Plasma Framework to KConfigGui

2013-08-08 Thread Kevin Ottens
st for KDE Frameworks, Plasma and Aaron J. Seigo. > > > Description > --- > > Add KConfigLoader from Plasma Framework to KConfigGui > > The ConfigLoader is way to awesome to not be directly in KConfig. > > > Diffs > - > > tier1/kconfig/autotests/CMakeLists.tx

Re: Review Request 111908: Add KConfigLoader from Plasma Framework to KConfigGui

2013-08-08 Thread Martin Gräßlin
, Plasma and Aaron J. Seigo. Changes --- Extended the documentation to show how one can use this class. Description --- Add KConfigLoader from Plasma Framework to KConfigGui The ConfigLoader is way to awesome to not be directly in KConfig. Diffs (updated) - tier1/kconfig/autotests

Re: Review Request 111908: Add KConfigLoader from Plasma Framework to KConfigGui

2013-08-08 Thread David Faure
-- > This is an automatically generated e-mail. To reply, visit: > http://git.reviewboard.kde.org/r/111908/ > --- > > (Updated Aug. 8, 2013, 4:58 a.m.) > > > Review request for KDE Frameworks, Plasma

Re: Review Request 111908: Add KConfigLoader from Plasma Framework to KConfigGui

2013-08-07 Thread Martin Gräßlin
, Plasma and Aaron J. Seigo. Changes --- Added Aaron to get some feedback Description --- Add KConfigLoader from Plasma Framework to KConfigGui The ConfigLoader is way to awesome to not be directly in KConfig. Diffs - tier1/kconfig/autotests/CMakeLists.txt PRE-CREATION tier1

Re: Review Request 111908: Add KConfigLoader from Plasma Framework to KConfigGui

2013-08-07 Thread Martin Gräßlin
:25 p.m., Martin Gräßlin wrote: > > --- > This is an automatically generated e-mail. To reply, visit: > http://git.reviewboard.kde.org/r/111908/ > ------- > > (Updated Aug. 6, 2013, 2:25 p.m.) >

Re: Review Request 111908: Add KConfigLoader from Plasma Framework to KConfigGui

2013-08-07 Thread David Faure
enerated e-mail. To reply, visit: > http://git.reviewboard.kde.org/r/111908/ > --- > > (Updated Aug. 6, 2013, 12:25 p.m.) > > > Review request for KDE Frameworks and Plasma. > > > Description > --- > > Add KConfigLoader

Re: Review Request 111908: Add KConfigLoader from Plasma Framework to KConfigGui

2013-08-06 Thread Martin Gräßlin
m., Martin Gräßlin wrote: > > --- > This is an automatically generated e-mail. To reply, visit: > http://git.reviewboard.kde.org/r/111908/ > --- > > (Updated Aug. 6,

Re: Review Request 111908: Add KConfigLoader from Plasma Framework to KConfigGui

2013-08-06 Thread Martin Gräßlin
Aug. 6, 2013, 2:25 p.m.) > > > Review request for KDE Frameworks and Plasma. > > > Description > --- > > Add KConfigLoader from Plasma Framework to KConfigGui > > The ConfigLoader is way to awesome to not be directly in KConfig. > > > Diffs >

Re: Review Request 111908: Add KConfigLoader from Plasma Framework to KConfigGui

2013-08-06 Thread Albert Astals Cid
gt; This is an automatically generated e-mail. To reply, visit: > http://git.reviewboard.kde.org/r/111908/ > --- > > (Updated Aug. 6, 2013, 12:25 p.m.) > > > Review request for KDE Frameworks and Plasma. > > > Description > --- > > Add KConfigLoader from

Re: Review Request 111908: Add KConfigLoader from Plasma Framework to KConfigGui

2013-08-06 Thread Aleix Pol Gonzalez
- > This is an automatically generated e-mail. To reply, visit: > http://git.reviewboard.kde.org/r/111908/ > --- > > (Updated Aug. 6, 2013, 12:25 p.m.) > > > Review request for KDE Fram

Re: Review Request 111908: Add KConfigLoader from Plasma Framework to KConfigGui

2013-08-06 Thread Martin Gräßlin
-mail. To reply, visit: > http://git.reviewboard.kde.org/r/111908/ > --- > > (Updated Aug. 6, 2013, 2:25 p.m.) > > > Review request for KDE Frameworks and Plasma. > > > Description > --- > > Add KConfigLoader from Plasma Fram

Re: Review Request 111908: Add KConfigLoader from Plasma Framework to KConfigGui

2013-08-06 Thread David Edmundson
gt; > > Review request for KDE Frameworks and Plasma. > > > Description > --- > > Add KConfigLoader from Plasma Framework to KConfigGui > > The ConfigLoader is way to awesome to not be directly in KConfig. > > > Diffs > - > > tier1/kco

Review Request 111908: Add KConfigLoader from Plasma Framework to KConfigGui

2013-08-06 Thread Martin Gräßlin
KConfigLoader from Plasma Framework to KConfigGui The ConfigLoader is way to awesome to not be directly in KConfig. Diffs - tier1/kconfig/autotests/CMakeLists.txt PRE-CREATION tier1/kconfig/autotests/kconfigloadertest.h PRE-CREATION tier1/kconfig/autotests/kconfigloadertest.cpp PRE

Re: dirmodel in plasma-framework

2013-08-03 Thread Heena Mahour
I have pushed it in branch heenamahour/gsoc (plasma-framework) http://quickgit.kde.org/?p=plasma-framework.git&a=commit&h=18266f3ab78108a3246bac3540cb3aef6f97bbe6 >the reason this doesn't work yet, is probably that most kioslaves aren't ported yet, so it will need a bit of b

Re: dirmodel in plasma-framework

2013-08-03 Thread Marco Martin
that url trash:/ can not be listed .However it worked > fine with plasma1 . > Further I want to know how shall I be able to use data engines in plasma2 > as window-list requires tasks data engine . > Regards yes, dirmodel exists plasma-framework/src/declarativeimports/dirmodel/ (and

dirmodel in plasma-framework

2013-08-03 Thread Heena Mahour
Hi , Is dirModel ported to plasma2 ? As when in dirModelplugin.cpp I changed qmlRegisterType(uri, 0, 1, "DirModel") to qmlRegisterType(uri, 2,0, "DirModel") I am getting an output in my trashcan plasmoid that url trash:/ can not be listed .However it worked fine with plasma1 . Further I want to k

Re: plasma-framework compile error

2013-07-29 Thread Sebastian Kügler
On Monday, July 29, 2013 11:42:40 Heena Mahour wrote: > Now, plasma-shell is giving out error > > :~/plasma-framework$ plasma-shell > > plasma-shell: symbol lookup error: > /home/heena/kf5/lib/i386-linux-gnu/libKDE4Support.so.5: undefined symbol: > kde_kiosk_

Re: plasma-framework compile error

2013-07-28 Thread Heena Mahour
wow ,it just build right now ;) hugs to plasma people :D ^_^ On Sun, Jul 28, 2013 at 10:54 PM, Sebastian Kügler wrote: > On Sunday, July 28, 2013 18:03:19 Heena Mahour wrote: > > > heena@heena-Aspire-5750:~/extra-cmake-modules/build$ cmake > > > -DCMAKE_INSTALL_PREFIX=$KF5 -DCMAKE_PREFIX_PATH=

Re: plasma-framework compile error

2013-07-28 Thread Sebastian Kügler
On Sunday, July 28, 2013 18:03:19 Heena Mahour wrote: > > heena@heena-Aspire-5750:~/extra-cmake-modules/build$ cmake > > -DCMAKE_INSTALL_PREFIX=$KF5 -DCMAKE_PREFIX_PATH=$KF5 > http://pastebin.com/raw.php?i=iZ6TtNpT The KF5 environmental variable is not set. You can only use the cmake command l

Re: plasma-framework compile error

2013-07-28 Thread Heena Mahour
http://pastebin.com/raw.php?i=iZ6TtNpT On Sun, Jul 28, 2013 at 5:03 PM, Sebastian Kügler wrote: > On Sunday, July 28, 2013 12:04:17 Heena Mahour wrote: > > I tried cmake in build directory > > but then also > > heena@heena-Aspire-5750:~/extra-cmake-modules/build$ cmake > > -DCMAKE_INSTALL_PREF

Re: plasma-framework compile error

2013-07-28 Thread Sebastian Kügler
On Sunday, July 28, 2013 12:04:17 Heena Mahour wrote: > I tried cmake in build directory > but then also > heena@heena-Aspire-5750:~/extra-cmake-modules/build$ cmake > -DCMAKE_INSTALL_PREFIX=$KF5 -DCMAKE_PREFIX_PATH=$KF5 .. CMake Error at > /usr/share/cmake-2.8/Modules/CMakePackageConfigHelpers.

Re: plasma-framework compile error

2013-07-28 Thread Heena Mahour
to the file: Call Stack (most recent call first): CMakeLists.txt:47 (configure_package_config_file) -- Configuring incomplete, errors occurred! Plasma framework is a bit difficult to compile these days I guess On Sun, Jul 28, 2013 at 11:42 AM, Sebastian Kügler wrote: > On Sunday, July

Re: plasma-framework compile error

2013-07-28 Thread Sebastian Kügler
On Sunday, July 28, 2013 11:26:30 Heena Mahour wrote: > Hi Sebastian , > I did $git pull --rebase now getting build errors : > heena@heena-Aspire-5750:~/extra-cmake-modules$ git pull --rebase > Current branch master is up to date. > heena@heena-Aspire-5750:~/extra-cmake-modules$ cmake > -DCMAKE_INS

Re: plasma-framework compile error

2013-07-28 Thread Heena Mahour
Hi Sebastian , I did $git pull --rebase now getting build errors : heena@heena-Aspire-5750:~/extra-cmake-modules$ git pull --rebase Current branch master is up to date. heena@heena-Aspire-5750:~/extra-cmake-modules$ cmake -DCMAKE_INSTALL_PREFIX=$KF5 . CMake Error at /usr/share/cmake-2.8/Modules/CMa

Re: plasma-framework compile error

2013-07-27 Thread Sebastian Kügler
On Saturday, July 27, 2013 09:33:27 Heena Mahour wrote: > I rebuild kdelibs after disabling src/declarativeimports/dirmodel still > same error You'll also need to update extra-cmake-modules. -- sebas http://www.kde.org | http://vizZzion.org | GPG Key ID: 9119 0EF9 ___

Re: plasma-framework compile error

2013-07-27 Thread Heena Mahour
I rebuild kdelibs after disabling src/declarativeimports/dirmodel still same error On Sat, Jul 27, 2013 at 7:50 AM, Martin Graesslin wrote: > On Friday 26 July 2013 22:27:01 Heena Mahour wrote: > > Hi , > > I am getting cmake error when I build plasma-framework > > http://p

Re: plasma-framework compile error

2013-07-27 Thread Martin Graesslin
On Friday 26 July 2013 22:27:01 Heena Mahour wrote: > Hi , > I am getting cmake error when I build plasma-framework > http://pastebin.com/raw.php?i=aHbnWHbn .I have build framework branch of > kdelibs .KIndly suggest something . This should be fixed already, but you need to reb

<    1   2   3   4   5   6   7   >