Re: Review Request 110529: more error handling in KIdleTime

2013-05-19 Thread David Faure
--- This is an automatically generated e-mail. To reply, visit: http://git.reviewboard.kde.org/r/110529/#review32795 --- Ship it! Looks good, just whitespace issues to fix before comm

Re: Review Request 110529: more error handling in KIdleTime

2013-05-19 Thread Ian Monroe
--- This is an automatically generated e-mail. To reply, visit: http://git.reviewboard.kde.org/r/110529/ --- (Updated May 20, 2013, 3:25 a.m.) Review request for KDE Frameworks and kd

Re: Review Request 110042: Find Qt5 version of DBusMenuQt

2013-05-19 Thread Ian Monroe
--- This is an automatically generated e-mail. To reply, visit: http://git.reviewboard.kde.org/r/110042/#review32794 --- Ship it! Ship It! - Ian Monroe On April 16, 2013, 1:26 p.m.

Review Request 110529: more error handling in KIdleTime

2013-05-19 Thread Ian Monroe
--- This is an automatically generated e-mail. To reply, visit: http://git.reviewboard.kde.org/r/110529/ --- Review request for kdelibs. Description --- more error handling in KI

Re: Review Request 109611: Add option to show "Recently Installed" apps in kickoff plasmoid

2013-05-19 Thread Kai Uwe Broulik
--- This is an automatically generated e-mail. To reply, visit: http://git.reviewboard.kde.org/r/109611/#review32792 --- I'm not sure about adding a submenu for that. Imho it should hi

Re: Review Request 109611: Add option to show "Recently Installed" apps in kickoff plasmoid

2013-05-19 Thread Wolfgang Bauer
--- This is an automatically generated e-mail. To reply, visit: http://git.reviewboard.kde.org/r/109611/ --- (Updated May 19, 2013, 11:39 p.m.) Review request for kde-workspace. Cha

Re: Review Request 109611: Add option to show "Recently Installed" apps in kickoff plasmoid

2013-05-19 Thread Wolfgang Bauer
--- This is an automatically generated e-mail. To reply, visit: http://git.reviewboard.kde.org/r/109611/ --- (Updated May 19, 2013, 11:41 p.m.) Review request for kde-workspace. Des

Re: Review Request 109611: Add option to show "Recently Installed" apps in kickoff plasmoid

2013-05-19 Thread Wolfgang Bauer
> On April 16, 2013, 4:45 p.m., David Faure wrote: > > plasma/desktop/applets/kickoff/core/applicationmodel.cpp, line 640 > > > > > > Can it really be null? Did you see this elsewhere, or did you add it > > "just

Re: Review Request 103555: Fix for stale permissions information in properties dialog

2013-05-19 Thread David Faure
--- This is an automatically generated e-mail. To reply, visit: http://git.reviewboard.kde.org/r/103555/#review32785 --- Ship it! Ship It! - David Faure On May 19, 2013, 2:53 p.m.,

Re: Review Request 103555: Fix for stale permissions information in properties dialog

2013-05-19 Thread Dawit Alemayehu
--- This is an automatically generated e-mail. To reply, visit: http://git.reviewboard.kde.org/r/103555/ --- (Updated May 19, 2013, 2:53 p.m.) Review request for kdelibs and David Fau

Re: Review Request 103555: Fix for stale permissions information in properties dialog

2013-05-19 Thread David Faure
--- This is an automatically generated e-mail. To reply, visit: http://git.reviewboard.kde.org/r/103555/#review32773 --- Looks good, just a few minor things to fix. kio/kio/kdirliste

Re: KF5 Update Meeting 2013-w20

2013-05-19 Thread Stephen Kelly
Alexander Neundorf wrote: > On Thursday 16 May 2013, Stephen Kelly wrote: >> Kevin Ottens wrote: >> >> Beside that, I would like if we could do a release of >> >> extra-cmake-modules as soon as possible, so other projects, KDE and >> >> non-KDE can start to make use of it and people can start to >