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

2013-09-07 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 111938: Allow installing both qca2 and qca3

2013-09-07 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. David Faure wrote:

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

2013-09-07 Thread Christophe Giboudeaux
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 112559: Port the remaining uses of KIO::NetAccess to KIO/Jobs

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

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

2013-09-07 Thread Àlex Fiestas
--- This is an automatically generated e-mail. To reply, visit: http://git.reviewboard.kde.org/r/112559/#review39533 --- khtml/khtml_part.cpp

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

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

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

2013-09-07 Thread Ivan Romanov
On Sept. 1, 2013, 11: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 111938: Allow installing both qca2 and qca3

2013-09-07 Thread Ivan Romanov
On Sept. 1, 2013, 11: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 111938: Allow installing both qca2 and qca3

2013-09-07 Thread Ivan Romanov
On Sept. 1, 2013, 11: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:

Review Request 112583: Steps to make KConfigWidgets compile Standalone

2013-09-07 Thread David Edmundson
--- This is an automatically generated e-mail. To reply, visit: http://git.reviewboard.kde.org/r/112583/ --- Review request for KDE Frameworks. Description --- KConfigWidgets

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

2013-09-07 Thread Christophe Giboudeaux
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 111938: Allow installing both qca2 and qca3

2013-09-07 Thread Ivan Romanov
On Sept. 1, 2013, 11: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 111938: Allow installing both qca2 and qca3

2013-09-07 Thread Ivan Romanov
On Sept. 1, 2013, 11: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 111938: Allow installing both qca2 and qca3

2013-09-07 Thread Ivan Romanov
On Sept. 1, 2013, 11: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 112583: Steps to make KConfigWidgets compile Standalone

2013-09-07 Thread Aleix Pol Gonzalez
--- This is an automatically generated e-mail. To reply, visit: http://git.reviewboard.kde.org/r/112583/#review39551 --- staging/kconfigwidgets/src/CMakeLists.txt

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

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

Build failed in Jenkins: plasma-framework_master_qt5 #697

2013-09-07 Thread KDE CI System
See http://build.kde.org/job/plasma-framework_master_qt5/697/changes 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 - 3 in workspace