Re: Review Request 128492: [kio] Minor typo in file ioslave

2016-07-20 Thread David Faure
--- This is an automatically generated e-mail. To reply, visit: https://git.reviewboard.kde.org/r/128492/#review97688 --- Ship it! Porting bug indeed, the kdelibs4 code said KDE::l

Re: Review Request 128398: Fix KDescendantsProxyModel::setSourceModel(...) to reset internal data

2016-07-20 Thread Sune Vuorela
--- This is an automatically generated e-mail. To reply, visit: https://git.reviewboard.kde.org/r/128398/#review97685 --- I tried writing a small test for it, and it seemed like I cou

Jenkins-kde-ci: kdelibs4support master stable-kf5-qt5 » Linux,gcc - Build # 76 - Fixed!

2016-07-20 Thread no-reply
GENERAL INFO BUILD SUCCESS Build URL: https://build.kde.org/job/kdelibs4support%20master%20stable-kf5-qt5/PLATFORM=Linux,compiler=gcc/76/ Project: PLATFORM=Linux,compiler=gcc Date of build: Thu, 21 Jul 2016 04:17:30 + Build duration: 12 min CHANGE SET No changes JUNIT RESULTS Nam

Jenkins-kde-ci: kdelibs4support master stable-kf5-qt5 » Linux,gcc - Build # 76 - Fixed!

2016-07-20 Thread no-reply
GENERAL INFO BUILD SUCCESS Build URL: https://build.kde.org/job/kdelibs4support%20master%20stable-kf5-qt5/PLATFORM=Linux,compiler=gcc/76/ Project: PLATFORM=Linux,compiler=gcc Date of build: Thu, 21 Jul 2016 04:17:30 + Build duration: 12 min CHANGE SET No changes JUNIT RESULTS Nam

Review Request 128492: [kio] Minor typo in file ioslave

2016-07-20 Thread Chinmoy Ranjan Pradhan
--- This is an automatically generated e-mail. To reply, visit: https://git.reviewboard.kde.org/r/128492/ --- Review request for KDE Frameworks and David Faure. Repository: kio Desc

Jenkins-kde-ci: knotifyconfig master stable-kf5-qt5 » Linux,gcc - Build # 138 - Fixed!

2016-07-20 Thread no-reply
GENERAL INFO BUILD SUCCESS Build URL: https://build.kde.org/job/knotifyconfig%20master%20stable-kf5-qt5/PLATFORM=Linux,compiler=gcc/138/ Project: PLATFORM=Linux,compiler=gcc Date of build: Thu, 21 Jul 2016 02:46:54 + Build duration: 39 sec CHANGE SET No changes JUNIT RESULTS Name

Jenkins-kde-ci: knotifyconfig master stable-kf5-qt5 » Linux,gcc - Build # 138 - Fixed!

2016-07-20 Thread no-reply
GENERAL INFO BUILD SUCCESS Build URL: https://build.kde.org/job/knotifyconfig%20master%20stable-kf5-qt5/PLATFORM=Linux,compiler=gcc/138/ Project: PLATFORM=Linux,compiler=gcc Date of build: Thu, 21 Jul 2016 02:46:54 + Build duration: 39 sec CHANGE SET No changes JUNIT RESULTS Name

Jenkins-kde-ci: knotifyconfig master stable-kf5-qt5 » Linux,gcc - Build # 137 - Failure!

2016-07-20 Thread no-reply
GENERAL INFO BUILD FAILURE Build URL: https://build.kde.org/job/knotifyconfig%20master%20stable-kf5-qt5/PLATFORM=Linux,compiler=gcc/137/ Project: PLATFORM=Linux,compiler=gcc Date of build: Thu, 21 Jul 2016 02:19:00 + Build duration: 1 min 21 sec CHANGE SET No changes ___

Jenkins-kde-ci: kio master stable-kf5-qt5 » Linux,gcc - Build # 138 - Fixed!

2016-07-20 Thread no-reply
GENERAL INFO BUILD SUCCESS Build URL: https://build.kde.org/job/kio%20master%20stable-kf5-qt5/PLATFORM=Linux,compiler=gcc/138/ Project: PLATFORM=Linux,compiler=gcc Date of build: Thu, 21 Jul 2016 02:02:50 + Build duration: 5 min 39 sec CHANGE SET No changes JUNIT RESULTS Name: (r

Jenkins-kde-ci: kio master stable-kf5-qt5 » Linux,gcc - Build # 138 - Fixed!

2016-07-20 Thread no-reply
GENERAL INFO BUILD SUCCESS Build URL: https://build.kde.org/job/kio%20master%20stable-kf5-qt5/PLATFORM=Linux,compiler=gcc/138/ Project: PLATFORM=Linux,compiler=gcc Date of build: Thu, 21 Jul 2016 02:02:50 + Build duration: 5 min 39 sec CHANGE SET No changes JUNIT RESULTS Name: (r

Jenkins-kde-ci: plasma-framework master kf5-qt5 » Linux,All,gcc - Build # 107 - Fixed!

2016-07-20 Thread no-reply
GENERAL INFO BUILD SUCCESS Build URL: https://build.kde.org/job/plasma-framework%20master%20kf5-qt5/PLATFORM=Linux,Variation=All,compiler=gcc/107/ Project: PLATFORM=Linux,Variation=All,compiler=gcc Date of build: Thu, 21 Jul 2016 01:35:53 + Build duration: 9 min 58 sec CHANGE SET No changes

Jenkins-kde-ci: plasma-framework master kf5-qt5 » Linux,All,gcc - Build # 107 - Fixed!

2016-07-20 Thread no-reply
GENERAL INFO BUILD SUCCESS Build URL: https://build.kde.org/job/plasma-framework%20master%20kf5-qt5/PLATFORM=Linux,Variation=All,compiler=gcc/107/ Project: PLATFORM=Linux,Variation=All,compiler=gcc Date of build: Thu, 21 Jul 2016 01:35:53 + Build duration: 9 min 58 sec CHANGE SET No changes

Jenkins-kde-ci: oxygen master kf5-qt5 » Linux,gcc - Build # 19 - Fixed!

2016-07-20 Thread no-reply
GENERAL INFO BUILD SUCCESS Build URL: https://build.kde.org/job/oxygen%20master%20kf5-qt5/PLATFORM=Linux,compiler=gcc/19/ Project: PLATFORM=Linux,compiler=gcc Date of build: Thu, 21 Jul 2016 01:49:57 + Build duration: 3 min 10 sec CHANGE SET No changes JUNIT RESULTS Name: (root)

Jenkins-kde-ci: oxygen master kf5-qt5 » Linux,gcc - Build # 19 - Fixed!

2016-07-20 Thread no-reply
GENERAL INFO BUILD SUCCESS Build URL: https://build.kde.org/job/oxygen%20master%20kf5-qt5/PLATFORM=Linux,compiler=gcc/19/ Project: PLATFORM=Linux,compiler=gcc Date of build: Thu, 21 Jul 2016 01:49:57 + Build duration: 3 min 10 sec CHANGE SET No changes JUNIT RESULTS Name: (root)

Jenkins-kde-ci: plasma-framework master kf5-qt5 » Linux,NoX11,gcc - Build # 107 - Fixed!

2016-07-20 Thread no-reply
GENERAL INFO BUILD SUCCESS Build URL: https://build.kde.org/job/plasma-framework%20master%20kf5-qt5/PLATFORM=Linux,Variation=NoX11,compiler=gcc/107/ Project: PLATFORM=Linux,Variation=NoX11,compiler=gcc Date of build: Thu, 21 Jul 2016 01:35:53 + Build duration: 2 min 27 sec CHANGE SET No cha

Jenkins-kde-ci: plasma-framework master kf5-qt5 » Linux,NoX11,gcc - Build # 107 - Fixed!

2016-07-20 Thread no-reply
GENERAL INFO BUILD SUCCESS Build URL: https://build.kde.org/job/plasma-framework%20master%20kf5-qt5/PLATFORM=Linux,Variation=NoX11,compiler=gcc/107/ Project: PLATFORM=Linux,Variation=NoX11,compiler=gcc Date of build: Thu, 21 Jul 2016 01:35:53 + Build duration: 2 min 27 sec CHANGE SET No cha

Jenkins-kde-ci: breeze-icons master kf5-qt5 » Linux,gcc - Build # 176 - Fixed!

2016-07-20 Thread no-reply
GENERAL INFO BUILD SUCCESS Build URL: https://build.kde.org/job/breeze-icons%20master%20kf5-qt5/PLATFORM=Linux,compiler=gcc/176/ Project: PLATFORM=Linux,compiler=gcc Date of build: Thu, 21 Jul 2016 01:34:25 + Build duration: 3 min 45 sec CHANGE SET No changes JUNIT RESULTS Name:

Jenkins-kde-ci: breeze-icons master kf5-qt5 » Linux,gcc - Build # 176 - Fixed!

2016-07-20 Thread no-reply
GENERAL INFO BUILD SUCCESS Build URL: https://build.kde.org/job/breeze-icons%20master%20kf5-qt5/PLATFORM=Linux,compiler=gcc/176/ Project: PLATFORM=Linux,compiler=gcc Date of build: Thu, 21 Jul 2016 01:34:25 + Build duration: 3 min 45 sec CHANGE SET No changes JUNIT RESULTS Name:

Jenkins-kde-ci: kservice master stable-kf5-qt5 » Linux,gcc - Build # 85 - Unstable!

2016-07-20 Thread no-reply
GENERAL INFO BUILD UNSTABLE Build URL: https://build.kde.org/job/kservice%20master%20stable-kf5-qt5/PLATFORM=Linux,compiler=gcc/85/ Project: PLATFORM=Linux,compiler=gcc Date of build: Wed, 20 Jul 2016 23:37:44 + Build duration: 3 min 28 sec CHANGE SET No changes JUNIT RESULTS Nam

Jenkins-kde-ci: breeze-icons master kf5-qt5 » Linux,gcc - Build # 175 - Still Failing!

2016-07-20 Thread no-reply
GENERAL INFO BUILD FAILURE Build URL: https://build.kde.org/job/breeze-icons%20master%20kf5-qt5/PLATFORM=Linux,compiler=gcc/175/ Project: PLATFORM=Linux,compiler=gcc Date of build: Wed, 20 Jul 2016 22:25:33 + Build duration: 16 sec CHANGE SET Revision b55dd97a355cd08d9b0c02f69e8c9a35ec37885

Jenkins-kde-ci: breeze-icons master kf5-qt5 » Linux,gcc - Build # 174 - Failure!

2016-07-20 Thread no-reply
GENERAL INFO BUILD FAILURE Build URL: https://build.kde.org/job/breeze-icons%20master%20kf5-qt5/PLATFORM=Linux,compiler=gcc/174/ Project: PLATFORM=Linux,compiler=gcc Date of build: Wed, 20 Jul 2016 21:55:12 + Build duration: 27 sec CHANGE SET Revision 21fab2ca0885aebd46c61289311313cd4f60d10

Jenkins-kde-ci: plasma-framework master stable-kf5-qt5 » Linux,NoX11,gcc - Build # 108 - Unstable!

2016-07-20 Thread no-reply
GENERAL INFO BUILD UNSTABLE Build URL: https://build.kde.org/job/plasma-framework%20master%20stable-kf5-qt5/PLATFORM=Linux,Variation=NoX11,compiler=gcc/108/ Project: PLATFORM=Linux,Variation=NoX11,compiler=gcc Date of build: Wed, 20 Jul 2016 15:05:46 + Build duration: 5 min 29 sec CHANGE SE

Jenkins-kde-ci: plasma-framework master kf5-qt5 » Linux,NoX11,gcc - Build # 106 - Still Failing!

2016-07-20 Thread no-reply
GENERAL INFO BUILD FAILURE Build URL: https://build.kde.org/job/plasma-framework%20master%20kf5-qt5/PLATFORM=Linux,Variation=NoX11,compiler=gcc/106/ Project: PLATFORM=Linux,Variation=NoX11,compiler=gcc Date of build: Wed, 20 Jul 2016 15:05:46 + Build duration: 37 sec CHANGE SET Revision 3f5

Jenkins-kde-ci: plasma-framework master kf5-qt5 » Linux,All,gcc - Build # 106 - Still Failing!

2016-07-20 Thread no-reply
GENERAL INFO BUILD FAILURE Build URL: https://build.kde.org/job/plasma-framework%20master%20kf5-qt5/PLATFORM=Linux,Variation=All,compiler=gcc/106/ Project: PLATFORM=Linux,Variation=All,compiler=gcc Date of build: Wed, 20 Jul 2016 15:05:46 + Build duration: 47 sec CHANGE SET Revision 3f53070

Jenkins-kde-ci: oxygen master kf5-qt5 » Linux,gcc - Build # 18 - Failure!

2016-07-20 Thread no-reply
GENERAL INFO BUILD FAILURE Build URL: https://build.kde.org/job/oxygen%20master%20kf5-qt5/PLATFORM=Linux,compiler=gcc/18/ Project: PLATFORM=Linux,compiler=gcc Date of build: Wed, 20 Jul 2016 14:59:56 + Build duration: 1 min 21 sec CHANGE SET Revision 577ae658acc05c4474d01c91b0520dd096565219

Re: Review Request 128476: Add scrollbars to kcm started without systemsettings5

2016-07-20 Thread Olivier Churlaud
> On July 20, 2016, 4:20 p.m., David Edmundson wrote: > > This affects the initial size of the showed screen, that needs fixing. > > Previously it showed the entire KCM, now it won't. yes it's written in the description, I would need help for this actually. The good point is that now, I can val

Re: Review Request 128476: Add scrollbars to kcm started without systemsettings5

2016-07-20 Thread David Edmundson
--- This is an automatically generated e-mail. To reply, visit: https://git.reviewboard.kde.org/r/128476/#review97667 --- This affects the initial size of the showed screen, that need

Re: Review Request 128490: Checksums tab: replace groupbox with vertical separator

2016-07-20 Thread Thomas Pfeiffer
--- This is an automatically generated e-mail. To reply, visit: https://git.reviewboard.kde.org/r/128490/#review97665 --- +1 It may still scare people, but I think it's a good comprom

Re: Review Request 128490: Checksums tab: replace groupbox with vertical separator

2016-07-20 Thread Olivier Churlaud
> On July 20, 2016, 3:48 p.m., Thomas Pfeiffer wrote: > > +1 > > It may still scare people, but I think it's a good compromise. If we get bad feedback , it can be reverted. But I think we won't - Olivier --- This is an automatically gen

Re: Review Request 120904: [KCharSelect] Handle characters outside BMP

2016-07-20 Thread Christoph Feck
--- This is an automatically generated e-mail. To reply, visit: https://git.reviewboard.kde.org/r/120904/ --- (Updated July 20, 2016, 1:48 p.m.) Review request for KDE Frameworks and

Re: Review Request 128490: Checksums tab: replace groupbox with vertical separator

2016-07-20 Thread Olivier Churlaud
--- This is an automatically generated e-mail. To reply, visit: https://git.reviewboard.kde.org/r/128490/#review97664 --- Ship it! Ship It! - Olivier Churlaud On July 20, 2016,

Re: Review Request 128413: Add clipboard buttons in the Checksums tab

2016-07-20 Thread Olivier Churlaud
> On July 16, 2016, 12:30 p.m., Elvis Angelaccio wrote: > > Ping? > > > > (String freeze for frameworks is next Saturday, right?) I would say: replace the [copy] button by a select on click. and then ship it - Olivier --- This is an au

Re: Review Request 128466: Rename Checksums tab to Integrity

2016-07-20 Thread Elvis Angelaccio
--- This is an automatically generated e-mail. To reply, visit: https://git.reviewboard.kde.org/r/128466/ --- (Updated July 20, 2016, 1:02 p.m.) Status -- This change has been di

Review Request 128490: Checksums tab: replace groupbox with vertical separator

2016-07-20 Thread Elvis Angelaccio
--- This is an automatically generated e-mail. To reply, visit: https://git.reviewboard.kde.org/r/128490/ --- Review request for KDE Frameworks and KDE Usability. Repository: kio De

Jenkins-kde-ci: plasma-framework master kf5-qt5 » Linux,NoX11,gcc - Build # 105 - Failure!

2016-07-20 Thread no-reply
GENERAL INFO BUILD FAILURE Build URL: https://build.kde.org/job/plasma-framework%20master%20kf5-qt5/PLATFORM=Linux,Variation=NoX11,compiler=gcc/105/ Project: PLATFORM=Linux,Variation=NoX11,compiler=gcc Date of build: Wed, 20 Jul 2016 12:44:15 + Build duration: 6 min 42 sec CHANGE SET Revisi

Jenkins-kde-ci: plasma-framework master stable-kf5-qt5 » Linux,All,gcc - Build # 107 - Fixed!

2016-07-20 Thread no-reply
GENERAL INFO BUILD SUCCESS Build URL: https://build.kde.org/job/plasma-framework%20master%20stable-kf5-qt5/PLATFORM=Linux,Variation=All,compiler=gcc/107/ Project: PLATFORM=Linux,Variation=All,compiler=gcc Date of build: Wed, 20 Jul 2016 12:44:15 + Build duration: 5 min 0 sec CHANGE SET Revi

Jenkins-kde-ci: plasma-framework master stable-kf5-qt5 » Linux,All,gcc - Build # 107 - Fixed!

2016-07-20 Thread no-reply
GENERAL INFO BUILD SUCCESS Build URL: https://build.kde.org/job/plasma-framework%20master%20stable-kf5-qt5/PLATFORM=Linux,Variation=All,compiler=gcc/107/ Project: PLATFORM=Linux,Variation=All,compiler=gcc Date of build: Wed, 20 Jul 2016 12:44:15 + Build duration: 5 min 0 sec CHANGE SET Revi

Jenkins-kde-ci: plasma-framework master stable-kf5-qt5 » Linux,NoX11,gcc - Build # 107 - Fixed!

2016-07-20 Thread no-reply
GENERAL INFO BUILD SUCCESS Build URL: https://build.kde.org/job/plasma-framework%20master%20stable-kf5-qt5/PLATFORM=Linux,Variation=NoX11,compiler=gcc/107/ Project: PLATFORM=Linux,Variation=NoX11,compiler=gcc Date of build: Wed, 20 Jul 2016 12:44:15 + Build duration: 2 min 18 sec CHANGE SET

Jenkins-kde-ci: plasma-framework master stable-kf5-qt5 » Linux,NoX11,gcc - Build # 107 - Fixed!

2016-07-20 Thread no-reply
GENERAL INFO BUILD SUCCESS Build URL: https://build.kde.org/job/plasma-framework%20master%20stable-kf5-qt5/PLATFORM=Linux,Variation=NoX11,compiler=gcc/107/ Project: PLATFORM=Linux,Variation=NoX11,compiler=gcc Date of build: Wed, 20 Jul 2016 12:44:15 + Build duration: 2 min 18 sec CHANGE SET

Jenkins-kde-ci: plasma-framework master kf5-qt5 » Linux,All,gcc - Build # 105 - Failure!

2016-07-20 Thread no-reply
GENERAL INFO BUILD FAILURE Build URL: https://build.kde.org/job/plasma-framework%20master%20kf5-qt5/PLATFORM=Linux,Variation=All,compiler=gcc/105/ Project: PLATFORM=Linux,Variation=All,compiler=gcc Date of build: Wed, 20 Jul 2016 12:44:15 + Build duration: 32 sec CHANGE SET Revision a7068cb

Re: Review Request 128466: Rename Checksums tab to Integrity

2016-07-20 Thread Sebastian Kügler
> On July 18, 2016, 12:05 p.m., Sebastian Kügler wrote: > > Please don't ship it, yet. > > > > > > I find the UI illogical. There's a groupbox grouping the checksum buttons, > > but then you can input the checksum above, so essentially, the groupbox is > > unnecessary and confusing. > > > >

Re: Review Request 128466: Rename Checksums tab to Integrity

2016-07-20 Thread Elvis Angelaccio
> On July 18, 2016, 12:05 p.m., Sebastian Kügler wrote: > > Please don't ship it, yet. > > > > > > I find the UI illogical. There's a groupbox grouping the checksum buttons, > > but then you can input the checksum above, so essentially, the groupbox is > > unnecessary and confusing. > > > >

Re: Review Request 126161: OS X housekeeping

2016-07-20 Thread René J . V . Bertin
--- This is an automatically generated e-mail. To reply, visit: https://git.reviewboard.kde.org/r/126161/ --- (Updated July 20, 2016, 1:06 p.m.) Status -- This change has been ma

Re: Review Request 128466: Rename Checksums tab to Integrity

2016-07-20 Thread Thomas Pfeiffer
> On July 18, 2016, 12:05 p.m., Sebastian Kügler wrote: > > Please don't ship it, yet. > > > > > > I find the UI illogical. There's a groupbox grouping the checksum buttons, > > but then you can input the checksum above, so essentially, the groupbox is > > unnecessary and confusing. > > > >

Review Request 128488: Add a fallback method for query_qmake() when there's no Qt5 installation

2016-07-20 Thread Aleix Pol Gonzalez
--- This is an automatically generated e-mail. To reply, visit: https://git.reviewboard.kde.org/r/128488/ --- Review request for Extra Cmake Modules, KDE Frameworks and David Faure. R

[sysadmin/ci-master-config] /: Change the versions of Qt 5 used on the CI system.

2016-07-20 Thread Ben Cooksley
Git commit 0b3511e4c66cce6206b04294c504cfbf93ae7056 by Ben Cooksley. Committed on 20/07/2016 at 07:58. Pushed by bcooksley into branch 'master'. Change the versions of Qt 5 used on the CI system. As Qt's compatibility between versions usually breaks on the CI, expect many things to break until ev

Re: Review Request 128471: [kio] Some fixes for KNewFileMenu.

2016-07-20 Thread Chinmoy Ranjan Pradhan
> On July 19, 2016, 5:29 p.m., David Faure wrote: > > src/filewidgets/knewfilemenu.cpp, line 789 > > > > > > The QFile::exists check is redundant. QFile::remove will do nothing if > > the file doesn't exist. > >

Re: Review Request 128471: [kio] Some fixes for KNewFileMenu.

2016-07-20 Thread Chinmoy Ranjan Pradhan
--- This is an automatically generated e-mail. To reply, visit: https://git.reviewboard.kde.org/r/128471/ --- (Updated July 20, 2016, 7:09 a.m.) Review request for KDE Frameworks and