Jenkins-kde-ci: kdoctools master stable-kf5-qt5 » OSX,clang - Build # 9 - Fixed!

2015-05-30 Thread no-reply
We thank you for fixing your build!___ Kde-frameworks-devel mailing list Kde-frameworks-devel@kde.org https://mail.kde.org/mailman/listinfo/kde-frameworks-devel

Jenkins-kde-ci: kdoctools master kf5-qt5 » OSX,clang - Build # 11 - Fixed!

2015-05-30 Thread no-reply
We thank you for fixing your build!___ Kde-frameworks-devel mailing list Kde-frameworks-devel@kde.org https://mail.kde.org/mailman/listinfo/kde-frameworks-devel

Jenkins-kde-ci: kdoctools master stable-kf5-qt5 » OSX,clang - Build # 8 - Failure!

2015-05-30 Thread no-reply
Check console output at https://build.kde.org/job/kdoctools%20master%20stable-kf5-qt5/PLATFORM=OSX,compiler=clang/8/ to view the results. ___ Kde-frameworks-devel mailing list Kde-frameworks-devel@kde.org https://mail.kde.org/mailman/listinfo/kde

Jenkins-kde-ci: kdoctools master kf5-qt5 » OSX,clang - Build # 10 - Failure!

2015-05-30 Thread no-reply
Check console output at https://build.kde.org/job/kdoctools%20master%20kf5-qt5/PLATFORM=OSX,compiler=clang/10/ to view the results. ___ Kde-frameworks-devel mailing list Kde-frameworks-devel@kde.org https://mail.kde.org/mailman/listinfo/kde-frame

Re: Review Request 123946: Add api to disable plot grid lines

2015-05-30 Thread Mark Gaiser
--- This is an automatically generated e-mail. To reply, visit: https://git.reviewboard.kde.org/r/123946/#review80988 --- +1 for the api addition. However, the naming of the api addit

Jenkins-kde-ci: kauth master stable-kf5-qt5 » OSX,clang - Build # 2 - Failure!

2015-05-30 Thread no-reply
Check console output at https://build.kde.org/job/kauth%20master%20stable-kf5-qt5/PLATFORM=OSX,compiler=clang/2/ to view the results. ___ Kde-frameworks-devel mailing list Kde-frameworks-devel@kde.org https://mail.kde.org/mailman/listinfo/kde-fra

Jenkins-kde-ci: kauth master kf5-qt5 » OSX,clang - Build # 2 - Failure!

2015-05-30 Thread no-reply
Check console output at https://build.kde.org/job/kauth%20master%20kf5-qt5/PLATFORM=OSX,compiler=clang/2/ to view the results. ___ Kde-frameworks-devel mailing list Kde-frameworks-devel@kde.org https://mail.kde.org/mailman/listinfo/kde-frameworks

Re: Review Request 123951: Change from GPL with KDE exception to LGPL to match the rest of the library

2015-05-30 Thread Diane Trout
--- This is an automatically generated e-mail. To reply, visit: https://git.reviewboard.kde.org/r/123951/ --- (Updated May 30, 2015, 2:26 p.m.) Review request for KDE Frameworks and K

Review Request 123951: Change from GPL with KDE exception to LGPL to match the rest of the library

2015-05-30 Thread Diane Trout
--- This is an automatically generated e-mail. To reply, visit: https://git.reviewboard.kde.org/r/123951/ --- Review request for KDE Frameworks and KDEPIM. Repository: kpeople Descr

Review Request 123946: Add api to disable plot grid lines

2015-05-30 Thread Marco Martin
--- This is an automatically generated e-mail. To reply, visit: https://git.reviewboard.kde.org/r/123946/ --- Review request for KDE Frameworks, Plasma and Thomas Pfeiffer. Repository

Re: Review Request 123438: Make KAUTH_HELPER_INSTALL_ABSOLUTE_DIR available to all KAuth users

2015-05-30 Thread Hrvoje Senjan
--- This is an automatically generated e-mail. To reply, visit: https://git.reviewboard.kde.org/r/123438/ --- (Updated May 30, 2015, 5:15 p.m.) Status -- This change has been mar

Re: Review Request 123930: Reduce temporary allocations.

2015-05-30 Thread Milian Wolff
> On May 29, 2015, 7:26 p.m., Kåre Särs wrote: > > Having a static cache sounds reasonable to me and I think it might be a > > good change, but I'll let people with more experience with sonnet give the > > thumbs up or down. > > > > Meanwhile I wonder why ktexteditor calls > > m_backgroundChe

Re: Review Request 123930: Reduce temporary allocations.

2015-05-30 Thread Milian Wolff
--- This is an automatically generated e-mail. To reply, visit: https://git.reviewboard.kde.org/r/123930/ --- (Updated May 30, 2015, 5:13 p.m.) Review request for KDE Frameworks and M

Re: Review Request 123929: If double-click is on empty area, then invoke back. (kfilewidgets)

2015-05-30 Thread Arjun AK
> On May 29, 2015, 6:15 p.m., David Faure wrote: > > I'm not sure this is a good idea, usability wise. Miss an icon by one > > pixel, and the effect is to go up, making the whole set of icons disappear? > > Ashish Bansal wrote: > Well about usability there's already discussion going on dolp

Re: Review Request 123929: If double-click is on empty area, then invoke back. (kfilewidgets)

2015-05-30 Thread Ashish Bansal
--- This is an automatically generated e-mail. To reply, visit: https://git.reviewboard.kde.org/r/123929/ --- (Updated May 30, 2015, 12:42 p.m.) Review request for Dolphin, KDE Framew

Re: Review Request 123940: Use KPluginLoader::factory() when loading KIO::DndPopupMenuPlugin

2015-05-30 Thread Ragnar Thomsen
--- This is an automatically generated e-mail. To reply, visit: https://git.reviewboard.kde.org/r/123940/ --- (Updated May 30, 2015, 2:39 p.m.) Review request for Dolphin, KDE Framewo

Re: Review Request 123940: Use KPluginLoader::factory() when loading KIO::DndPopupMenuPlugin

2015-05-30 Thread Emmanuel Pescosta
--- This is an automatically generated e-mail. To reply, visit: https://git.reviewboard.kde.org/r/123940/#review80967 --- Ship it! Looks good otherwise. :) src/widgets/dropjob.cpp (

Re: Review Request 123929: If double-click is on empty area, then invoke back. (kfilewidgets)

2015-05-30 Thread Ashish Bansal
> On May 29, 2015, 12:45 p.m., David Faure wrote: > > I'm not sure this is a good idea, usability wise. Miss an icon by one > > pixel, and the effect is to go up, making the whole set of icons disappear? > > Ashish Bansal wrote: > Well about usability there's already discussion going on dol

Re: Review Request 123929: If double-click is on empty area, then invoke back. (kfilewidgets)

2015-05-30 Thread David Faure
> On May 29, 2015, 12:45 p.m., David Faure wrote: > > I'm not sure this is a good idea, usability wise. Miss an icon by one > > pixel, and the effect is to go up, making the whole set of icons disappear? > > Ashish Bansal wrote: > Well about usability there's already discussion going on dol

Re: Review Request 123929: If double-click is on empty area, then invoke back. (kfilewidgets)

2015-05-30 Thread David Faure
> On May 29, 2015, 12:45 p.m., David Faure wrote: > > I'm not sure this is a good idea, usability wise. Miss an icon by one > > pixel, and the effect is to go up, making the whole set of icons disappear? > > Ashish Bansal wrote: > Well about usability there's already discussion going on dol