Build failed in Jenkins: plasma-framework_master_qt5 #696

2013-09-06 Thread KDE CI System
See Changes: [scripty] SVN_SILENT made messages (.desktop file) -- Started by remote host 2a01:4f8:160:9363::9 with note: Triggered by commit Building remotely on LinuxSlave - 2 in workspac

Review Request 112574: Reorganizing knewstuff framework

2013-09-06 Thread Jeremy Paul Whiting
--- This is an automatically generated e-mail. To reply, visit: http://git.reviewboard.kde.org/r/112574/ --- Review request for KDE Frameworks. Description --- I renamed knewstuf

Review Request 112560: Remove KNotification dependency in KCompletion

2013-09-06 Thread Aleix Pol Gonzalez
--- This is an automatically generated e-mail. To reply, visit: http://git.reviewboard.kde.org/r/112560/ --- Review request for KDE Frameworks. Description --- Drop KNotification

Re: Review Request 111938: Allow installing both qca2 and qca3

2013-09-06 Thread Ivan Romanov
> On Sept. 1, 2013, 5:16 p.m., David Faure wrote: > > Looks fine to me. > > Ivan Romanov wrote: > I don't agree with this patch. I didn't get any notification about this. > So I very ask you before do any changes in cmake rules talk with me. It is > important for me. > > David Faure wrote

Re: Review Request 112519: trim KStyle

2013-09-06 Thread Wojciech Kapuscinski
> On Sept. 4, 2013, 10:20 p.m., Christoph Feck wrote: > > staging/frameworkintegration/src/kstyle/kstyle.h, line 98 > > > > > > The "runtime element extension" should probably stay. They are useful, > > and Qt/QSt

Re: Review Request 112519: trim KStyle

2013-09-06 Thread Wojciech Kapuscinski
--- This is an automatically generated e-mail. To reply, visit: http://git.reviewboard.kde.org/r/112519/ --- (Updated Sept. 6, 2013, 7:48 p.m.) Review request for KDE Frameworks, Chri

Review Request 112570: Port some kde4_add_executable usages to add_executable

2013-09-06 Thread Aleix Pol Gonzalez
--- This is an automatically generated e-mail. To reply, visit: http://git.reviewboard.kde.org/r/112570/ --- Review request for KDE Frameworks. Description --- kde4_add_executabl

Re: Review Request 112519: trim KStyle

2013-09-06 Thread Wojciech Kapuscinski
--- This is an automatically generated e-mail. To reply, visit: http://git.reviewboard.kde.org/r/112519/ --- (Updated Sept. 6, 2013, 3:11 p.m.) Review request for KDE Frameworks. Ch

Re: Review Request 112527: Clean up KEmoticons framework (prior to splitting)

2013-09-06 Thread David Gil Oliva
--- This is an automatically generated e-mail. To reply, visit: http://git.reviewboard.kde.org/r/112527/ --- (Updated Sept. 6, 2013, 8:59 p.m.) Review request for KDE Frameworks. Ch

Review Request 112571: enable clearButton in kpassworddialog

2013-09-06 Thread Wojciech Kapuscinski
--- This is an automatically generated e-mail. To reply, visit: http://git.reviewboard.kde.org/r/112571/ --- Review request for KDE Frameworks. Description --- enable clearButton

Re: Review Request 112281: Allow setting a custom highligher on the spell check decorator

2013-09-06 Thread David Faure
--- This is an automatically generated e-mail. To reply, visit: http://git.reviewboard.kde.org/r/112281/#review39491 --- Looks ok to me, apart from the formatting of the documentation

Re: Review Request 112559: Port the remaining uses of KIO::NetAccess to KIO/Jobs

2013-09-06 Thread David Faure
--- This is an automatically generated e-mail. To reply, visit: http://git.reviewboard.kde.org/r/112559/#review39476 --- khtml/ecma/xmlhttprequest.cpp

Re: Review Request 112281: Allow setting a custom highligher on the spell check decorator

2013-09-06 Thread David Edmundson
--- This is an automatically generated e-mail. To reply, visit: http://git.reviewboard.kde.org/r/112281/ --- (Updated Sept. 6, 2013, 5:53 p.m.) Review request for KDE Frameworks. Ch

Re: Review Request 112281: Allow setting a custom highligher on the spell check decorator

2013-09-06 Thread David Faure
> On Sept. 1, 2013, 8:31 a.m., David Faure wrote: > > tier1/sonnet/src/ui/spellcheckdecorator.h, line 42 > > > > > > Missing documentation (in particular about the ownership being > > transferred to the decorator)

Re: Review Request 112281: Allow setting a custom highligher on the spell check decorator

2013-09-06 Thread David Edmundson
--- This is an automatically generated e-mail. To reply, visit: http://git.reviewboard.kde.org/r/112281/ --- (Updated Sept. 6, 2013, 6:42 p.m.) Review request for KDE Frameworks. De

Re: Review Request 112281: Allow setting a custom highligher on the spell check decorator

2013-09-06 Thread David Edmundson
> On Sept. 1, 2013, 8:31 a.m., David Faure wrote: > > tier1/sonnet/src/ui/spellcheckdecorator.h, line 42 > > > > > > Missing documentation (in particular about the ownership being > > transferred to the decorator)

Build failed in Jenkins: plasma-framework_master_qt5 #695

2013-09-06 Thread KDE CI System
See Changes: [notmart] fix size hints propagation -- Started by remote host 2a01:4f8:160:9363::9 with note: Triggered by commit Building remotely on LinuxSlave - 2 in workspace

Build failed in Jenkins: plasma-framework_master_qt5 #694

2013-09-06 Thread KDE CI System
See Changes: [notmart] experiment support for panel auto resize -- Started by remote host 2a01:4f8:160:9363::9 with note: Triggered by commit Building remotely on LinuxSlave - 2 in workspac

Re: Review Request 112559: Port the remaining uses of KIO::NetAccess to KIO/Jobs

2013-09-06 Thread Àlex Fiestas
--- This is an automatically generated e-mail. To reply, visit: http://git.reviewboard.kde.org/r/112559/ --- (Updated Sept. 6, 2013, 3:10 p.m.) Review request for KDE Frameworks and D

Review Request 112559: Port the remaining uses of KIO::NetAccess to KIO/Jobs

2013-09-06 Thread Àlex Fiestas
--- This is an automatically generated e-mail. To reply, visit: http://git.reviewboard.kde.org/r/112559/ --- Review request for KDE Frameworks and David Faure. Description --- Po

Re: Review Request 112559: Port the remaining uses of KIO::NetAccess to KIO/Jobs

2013-09-06 Thread Àlex Fiestas
--- This is an automatically generated e-mail. To reply, visit: http://git.reviewboard.kde.org/r/112559/ --- (Updated Sept. 6, 2013, 2:28 p.m.) Status -- This change has been dis

Re: Review Request 112463: Port SMB kioslave to KF5/Qt5

2013-09-06 Thread Mark Gaiser
> On Sept. 6, 2013, 1:10 p.m., Dawit Alemayehu wrote: > > I have several changes coming to this ioslave. Mainly I am going to > > implement the copyFrom and copyTo optimizations like I did for the sftp > > ioslave and add support for upload/download resumptions. The changes are > > going to co

Review Request 112557: Remove all links to libkdeqt5staging

2013-09-06 Thread Wojciech Kapuscinski
--- This is an automatically generated e-mail. To reply, visit: http://git.reviewboard.kde.org/r/112557/ --- Review request for KDE Frameworks. Description --- Remove all links t

Re: Plasma Frameworks 5 article

2013-09-06 Thread David Faure
On Friday 06 September 2013 14:39:07 Sebastian Kügler wrote: > Hey Howard, > > Thanks for the excellent article published yesterday! > > On Friday, September 06, 2013 20:20:24 Ho Wan Chan wrote: > > We at the KDE Promotion Team are working on an article to explain the > > benefits of KDE Framewor

Re: Review Request 112463: Port SMB kioslave to KF5/Qt5

2013-09-06 Thread Dawit Alemayehu
--- This is an automatically generated e-mail. To reply, visit: http://git.reviewboard.kde.org/r/112463/#review39463 --- I have several changes coming to this ioslave. Mainly I am goin

Re: Plasma Frameworks 5 article

2013-09-06 Thread Sebastian Kügler
Hey Howard, Thanks for the excellent article published yesterday! On Friday, September 06, 2013 20:20:24 Ho Wan Chan wrote: > We at the KDE Promotion Team are working on an article to explain the > benefits of KDE Frameworks 5 (which is different from the Dot article we > posted yesterday, that i

Plasma Frameworks 5 article

2013-09-06 Thread Ho Wan Chan
Hello David, Kevin and the KDE Frameworks 5 developers, We at the KDE Promotion Team are working on an article to explain the benefits of KDE Frameworks 5 (which is different from the Dot article we posted yesterday, that is about the release cycle). We would like to know who currently are workin

Build failed in Jenkins: plasma-framework_master_qt5 #693

2013-09-06 Thread KDE CI System
See Changes: [notmart] use plasma components [notmart] use native text rendering -- Started by remote host 2a01:4f8:160:9363::9 with note: Triggered by commit Building remotely on LinuxSla

Re: Review Request 111938: Allow installing both qca2 and qca3

2013-09-06 Thread David Faure
> On Sept. 1, 2013, 5:16 p.m., David Faure wrote: > > Looks fine to me. > > Ivan Romanov wrote: > I don't agree with this patch. I didn't get any notification about this. > So I very ask you before do any changes in cmake rules talk with me. It is > important for me. Holy... and here I wa

Re: Review Request 112527: Clean up KEmoticons framework (prior to splitting)

2013-09-06 Thread David Faure
--- This is an automatically generated e-mail. To reply, visit: http://git.reviewboard.kde.org/r/112527/#review39452 --- staging/kemoticons/src/core/kemoticonsprovider.h

Re: Review Request 112527: Clean up KEmoticons framework (prior to splitting)

2013-09-06 Thread Aurélien Gâteau
--- This is an automatically generated e-mail. To reply, visit: http://git.reviewboard.kde.org/r/112527/#review39450 --- Ship it! Looks good to me. - Aurélien Gâteau On Sept. 5, 20