Re: KIO directory listing - CPU slows down SSD

2014-05-28 Thread Aaron J. Seigo
On Wednesday, May 28, 2014 21:12:43 Mark Gaiser wrote: > You've written that with the assumption of backwards compatibility. > It's a nice idea, but why should we even try to remain backwards > compatible? The question should be inverted: why *should* we break compatibility? Least change always w

Re: KIO directory listing - CPU slows down SSD

2014-05-28 Thread Mark Gaiser
On Wed, May 28, 2014 at 7:33 PM, Aaron J. Seigo wrote: > On Wednesday, May 28, 2014 16:23:31 Mark Gaiser wrote: >> On Wed, May 28, 2014 at 9:59 AM, David Faure wrote: >> > On Wednesday 14 May 2014 12:50:18 Aaron J. Seigo wrote: >> >> * one entry at a time, serialized >> >> * no changing data once

Re: KIO directory listing - CPU slows down SSD

2014-05-28 Thread Aaron J. Seigo
On Wednesday, May 28, 2014 16:23:31 Mark Gaiser wrote: > On Wed, May 28, 2014 at 9:59 AM, David Faure wrote: > > On Wednesday 14 May 2014 12:50:18 Aaron J. Seigo wrote: > >> * one entry at a time, serialized > >> * no changing data once put there > >> * no getting the data back out > >> > >> the c

Jenkins build is back to stable : kxmlgui_master_qt5 #137

2014-05-28 Thread KDE CI System
See ___ Kde-frameworks-devel mailing list Kde-frameworks-devel@kde.org https://mail.kde.org/mailman/listinfo/kde-frameworks-devel

Review Request 118385: Remove name prefix from autotests

2014-05-28 Thread Alex Merry
--- This is an automatically generated e-mail. To reply, visit: https://git.reviewboard.kde.org/r/118385/ --- Review request for KDE Frameworks. Repository: kdelibs4support Descript

Re: Review Request 118374: Fix finding of Qt5LinguistTools on some systems

2014-05-28 Thread Commit Hook
--- This is an automatically generated e-mail. To reply, visit: https://git.reviewboard.kde.org/r/118374/#review58678 --- This review has been submitted with commit 51c4739231a637ee60

Re: Review Request 118374: Fix finding of Qt5LinguistTools on some systems

2014-05-28 Thread Alex Merry
--- This is an automatically generated e-mail. To reply, visit: https://git.reviewboard.kde.org/r/118374/ --- (Updated May 28, 2014, 3:48 p.m.) Status -- This change has been mar

Review Request 118384: Improve the kgendesignerplugin man page

2014-05-28 Thread Alex Merry
--- This is an automatically generated e-mail. To reply, visit: https://git.reviewboard.kde.org/r/118384/ --- Review request for KDE Frameworks. Repository: kdesignerplugin Descript

Re: Review Request 118374: Fix finding of Qt5LinguistTools on some systems

2014-05-28 Thread Nicolás Alvarez
--- This is an automatically generated e-mail. To reply, visit: https://git.reviewboard.kde.org/r/118374/#review58677 --- Ship it! Ship It! - Nicolás Alvarez On May 28, 2014, 8:32

Re: KIO directory listing - CPU slows down SSD

2014-05-28 Thread Mark Gaiser
On Wed, May 28, 2014 at 9:59 AM, David Faure wrote: > On Wednesday 14 May 2014 12:50:18 Aaron J. Seigo wrote: >> * one entry at a time, serialized >> * no changing data once put there >> * no getting the data back out >> >> the class would then just batch up data into its QByteArray and then shunt

Re: Review Request 118264: Switch to PolkitQt5-1

2014-05-28 Thread Hrvoje Senjan
> On May 28, 2014, 2:56 p.m., Aurélien Gâteau wrote: > > Building with polkit-qt-1 master (bac771e69887c9253f2b0973f6310810db0061f8) > > fails with this error: > > > > [ 55%] Building CXX object > > autotests/CMakeFiles/KAuthHelperTest.dir/HelperTest.cpp.o > > In file included from > > /home/

Re: Review Request 118264: Switch to PolkitQt5-1

2014-05-28 Thread Hrvoje Senjan
--- This is an automatically generated e-mail. To reply, visit: https://git.reviewboard.kde.org/r/118264/ --- (Updated May 28, 2014, 4:01 p.m.) Review request for KDE Frameworks and K

Re: Review Request 118270: [doc] explicitly load external entities (after CVE-2014-0191)

2014-05-28 Thread Jonathan Riddell
--- This is an automatically generated e-mail. To reply, visit: https://git.reviewboard.kde.org/r/118270/#review58671 --- yes working good, update going through ubuntu now https://bug

Review Request 118377: Add an autotest for designer plugin generation

2014-05-28 Thread Alex Merry
--- This is an automatically generated e-mail. To reply, visit: https://git.reviewboard.kde.org/r/118377/ --- Review request for KDE Frameworks. Repository: kdesignerplugin Descript

Re: Review Request 118192: Do not set the INSTALL_RPATH_USE_LINK_PATH and INSTALL_RPATH properties

2014-05-28 Thread Commit Hook
--- This is an automatically generated e-mail. To reply, visit: https://git.reviewboard.kde.org/r/118192/#review58670 --- This review has been submitted with commit 6579620b1b62e216fd

Re: Review Request 118192: Do not set the INSTALL_RPATH_USE_LINK_PATH and INSTALL_RPATH properties

2014-05-28 Thread Commit Hook
--- This is an automatically generated e-mail. To reply, visit: https://git.reviewboard.kde.org/r/118192/#review58669 --- This review has been submitted with commit 6feaf86e6e98c93b49

Re: Review Request 118192: Do not set the INSTALL_RPATH_USE_LINK_PATH and INSTALL_RPATH properties

2014-05-28 Thread Alex Merry
--- This is an automatically generated e-mail. To reply, visit: https://git.reviewboard.kde.org/r/118192/ --- (Updated May 28, 2014, 1:30 p.m.) Status -- This change has been mar

Re: Review Request 118192: Do not set the INSTALL_RPATH_USE_LINK_PATH and INSTALL_RPATH properties

2014-05-28 Thread Aurélien Gâteau
--- This is an automatically generated e-mail. To reply, visit: https://git.reviewboard.kde.org/r/118192/#review58666 --- Ship it! Ship It! - Aurélien Gâteau On May 28, 2014, 2:54

Re: Review Request 118193: Use the macros provided by KDesignerPlugin

2014-05-28 Thread Aurélien Gâteau
--- This is an automatically generated e-mail. To reply, visit: https://git.reviewboard.kde.org/r/118193/#review58667 --- Ship it! Ship It! - Aurélien Gâteau On May 18, 2014, 2:13

Re: Review Request 118264: Switch to PolkitQt5-1

2014-05-28 Thread Aurélien Gâteau
--- This is an automatically generated e-mail. To reply, visit: https://git.reviewboard.kde.org/r/118264/#review58664 --- Building with polkit-qt-1 master (bac771e69887c9253f2b0973f631

Re: Review Request 118192: Do not set the INSTALL_RPATH_USE_LINK_PATH and INSTALL_RPATH properties

2014-05-28 Thread Alex Merry
> On May 28, 2014, 12:41 p.m., Aurélien Gâteau wrote: > > KF5DesignerPluginMacros.cmake, line 83 > > > > > > Can kf5designerplugin_add_plugin be turned into a function? No, because of the find_package() call. I'v

Re: Review Request 118192: Do not set the INSTALL_RPATH_USE_LINK_PATH and INSTALL_RPATH properties

2014-05-28 Thread Alex Merry
--- This is an automatically generated e-mail. To reply, visit: https://git.reviewboard.kde.org/r/118192/ --- (Updated May 28, 2014, 12:54 p.m.) Review request for KDE Frameworks and

Re: Review Request 118193: Use the macros provided by KDesignerPlugin

2014-05-28 Thread Aurélien Gâteau
> On May 28, 2014, 2:23 p.m., Aurélien Gâteau wrote: > > Does not build here: CMake complains that kf5designerplugin_add_plugin() > > does not exist. > > > > If I replace it with kf5designerplugin_add_widget_files() then it says > > "Cannot specify link libraries for target "kdedeprecated" whi

Re: Review Request 118192: Fix/replace CMake macros provided by kdesignerplugin

2014-05-28 Thread Aurélien Gâteau
--- This is an automatically generated e-mail. To reply, visit: https://git.reviewboard.kde.org/r/118192/#review58656 --- The name of the functions are a bit confusing: it is not obvio

Re: Review Request 118193: Use the macros provided by KDesignerPlugin

2014-05-28 Thread Alex Merry
> On May 28, 2014, 12:23 p.m., Aurélien Gâteau wrote: > > Does not build here: CMake complains that kf5designerplugin_add_plugin() > > does not exist. > > > > If I replace it with kf5designerplugin_add_widget_files() then it says > > "Cannot specify link libraries for target "kdedeprecated" wh

Re: Review Request 118193: Use the macros provided by KDesignerPlugin

2014-05-28 Thread Aurélien Gâteau
> On May 28, 2014, 2:23 p.m., Aurélien Gâteau wrote: > > Does not build here: CMake complains that kf5designerplugin_add_plugin() > > does not exist. > > > > If I replace it with kf5designerplugin_add_widget_files() then it says > > "Cannot specify link libraries for target "kdedeprecated" whi

Re: Review Request 118193: Use the macros provided by KDesignerPlugin

2014-05-28 Thread Aurélien Gâteau
--- This is an automatically generated e-mail. To reply, visit: https://git.reviewboard.kde.org/r/118193/#review58654 --- Does not build here: CMake complains that kf5designerplugin_ad

Re: Review Request 118253: Port KWindowSystem::raiseWindow and ::lowerWindow to XCB

2014-05-28 Thread Aurélien Gâteau
--- This is an automatically generated e-mail. To reply, visit: https://git.reviewboard.kde.org/r/118253/#review58651 --- Ship it! I am no expert on the topic, but since very few peop

Review Request 118374: Fix finding of Qt5LinguistTools on some systems

2014-05-28 Thread Alex Merry
--- This is an automatically generated e-mail. To reply, visit: https://git.reviewboard.kde.org/r/118374/ --- Review request for Build System, Extra Cmake Modules and KDE Frameworks.

Re: Issues for Qt5/KF5/KDE/CI system on OSX/MacPorts

2014-05-28 Thread Alex Merry
On 28/05/14 07:47, Ben Cooksley wrote: > On Tue, May 27, 2014 at 9:55 PM, Alex Merry wrote: >> On Monday 26 May 2014 20:37:23 mk-li...@email.de wrote: >>> I hope that Ben can give me a hint about how to make use of proper variable >>> substitution in that cfg file, since the following unfortunatel

Re: Issues for Qt5/KF5/KDE/CI system on OSX/MacPorts

2014-05-28 Thread Alex Merry
On 28/05/14 07:48, Ben Cooksley wrote: > On Tue, May 27, 2014 at 9:59 PM, Alex Merry wrote: >> On Tuesday 27 May 2014 17:30:56 Ben Cooksley wrote: >>> Thanks for those details. I'm not sure what the defaults should be - >>> but I do agree that they should be usable outside the CI system. >>> Are t

Re: Review Request 118373: Do not delete passed objects to createObjectFromComponent

2014-05-28 Thread David Edmundson
--- This is an automatically generated e-mail. To reply, visit: https://git.reviewboard.kde.org/r/118373/ --- (Updated May 28, 2014, 10:55 a.m.) Status -- This change has been ma

Re: Review Request 118373: Do not delete passed objects to createObjectFromComponent

2014-05-28 Thread Commit Hook
--- This is an automatically generated e-mail. To reply, visit: https://git.reviewboard.kde.org/r/118373/#review58646 --- This review has been submitted with commit a93fa67ea8072cc5c4

Re: Review Request 118373: Do not delete passed objects to createObjectFromComponent

2014-05-28 Thread Aleix Pol Gonzalez
--- This is an automatically generated e-mail. To reply, visit: https://git.reviewboard.kde.org/r/118373/#review58643 --- Ship it! Yes, it doesn't make much sense to delete it only in

Review Request 118373: Do not delete passed objects to createObjectFromComponent

2014-05-28 Thread David Edmundson
--- This is an automatically generated e-mail. To reply, visit: https://git.reviewboard.kde.org/r/118373/ --- Review request for KDE Frameworks and Plasma. Repository: kdeclarative

Re: KIO directory listing - CPU slows down SSD

2014-05-28 Thread David Faure
On Wednesday 14 May 2014 12:50:18 Aaron J. Seigo wrote: > * one entry at a time, serialized > * no changing data once put there > * no getting the data back out > > the class would then just batch up data into its QByteArray and then shunt > it across the wire when needed. no middle men, no 100s