Jenkins build is back to stable : kdelibs_stable #887

2013-11-02 Thread KDE CI System
See ___ Kde-frameworks-devel mailing list Kde-frameworks-devel@kde.org https://mail.kde.org/mailman/listinfo/kde-frameworks-devel

Re: build.kde.org and `ping`

2013-11-02 Thread David Faure
On Sunday 03 November 2013 12:42:24 Ben Cooksley wrote: > On Sun, Nov 3, 2013 at 12:27 PM, David Faure wrote: > > This error: > > > > http://build.kde.org/job/kdelibs_stable/886/testReport/(root)/TestSuite/kp > > typrocesstest/ > > > > is caused by this: > > > > jenkins@slave1:/srv/jenkins/work

Re: build.kde.org and `ping`

2013-11-02 Thread Ben Cooksley
On Sun, Nov 3, 2013 at 12:27 PM, David Faure wrote: > This error: > > http://build.kde.org/job/kdelibs_stable/886/testReport/(root)/TestSuite/kptyprocesstest/ > > is caused by this: > > jenkins@slave1:/srv/jenkins/workspace/kdelibs_master/build> ping -i 0.5 > localhost > ping: icmp open socket: O

Build failed in Jenkins: plasma-framework_master_qt5 #885

2013-11-02 Thread KDE CI System
See Changes: [faure] Fix compilation -- [...truncated 944 lines...] :0: Note: No

build.kde.org and `ping`

2013-11-02 Thread David Faure
This error: http://build.kde.org/job/kdelibs_stable/886/testReport/(root)/TestSuite/kptyprocesstest/ is caused by this: jenkins@slave1:/srv/jenkins/workspace/kdelibs_master/build> ping -i 0.5 localhost ping: icmp open socket: Operation not permitted because the code is doing exactly this: void

Re: plasma-framework buildsys borked

2013-11-02 Thread David Faure
On Saturday 02 November 2013 20:05:56 Gregor Tätzner wrote: > The following imported targets are referenced, but are missing: KDE4__kio > KF5::KParts I just fixed that error. -- David Faure, fa...@kde.org, http://www.davidfaure.fr Working on KDE, in particular KDE Frameworks 5 _

Build failed in Jenkins: plasma-framework_master_qt5 #884

2013-11-02 Thread KDE CI System
See Changes: [faure] kde4support now needs kparts (I sure hope one day we won't have to do this by hand) -- Started by remote host 127.0.0.1 with note: Triggered by commit Building remotel

Re: Review Request 113510: Deprecate methods in kimageio

2013-11-02 Thread David Faure
On Saturday 02 November 2013 22:36:03 Alex Merry wrote: > Moving this to the list. > > On 02/11/13 19:36, David Faure wrote: > > On November 2nd, 2013, 6:21 p.m. UTC, *Alex Merry* wrote: > > Related question: would it make sense to try to implement this by > > directly enumerat

Re: Review Request 113510: Deprecate methods in kimageio

2013-11-02 Thread Alex Merry
Moving this to the list. On 02/11/13 19:36, David Faure wrote: > On November 2nd, 2013, 6:21 p.m. UTC, *Alex Merry* wrote: > Related question: would it make sense to try to implement this by > directly enumerating the QImageIO plugins (and reading their JSON files)? > Possibly with

Jenkins build is back to normal : kdelibs_frameworks_qt5 #1564

2013-11-02 Thread KDE CI System
See ___ Kde-frameworks-devel mailing list Kde-frameworks-devel@kde.org https://mail.kde.org/mailman/listinfo/kde-frameworks-devel

Build failed in Jenkins: kdelibs_frameworks_qt5 #1563

2013-11-02 Thread KDE CI System
See Changes: [faure] Remove config-kio.h, now unused [faure] Move kntlm to be a library in the KIO framework. -- [...truncated 7762 lines...] [ 91%] Building CXX object khtml/src/CMakeFiles/KH

Re: Getting ecm files from the ECM package

2013-11-02 Thread David Faure
On Saturday 02 November 2013 13:44:55 Stephen Kelly wrote: > Alexander Neundorf wrote: > > In case we decide to go this way (i.e. the "my ideal view" plus optional > > downloading), and we should hear Stephens opinion on that, > > My opinion: > > 1) The current situation with ECM and KF5 is just

Jenkins build is back to normal : kdelibs_frameworks_qt5 #1560

2013-11-02 Thread KDE CI System
See ___ Kde-frameworks-devel mailing list Kde-frameworks-devel@kde.org https://mail.kde.org/mailman/listinfo/kde-frameworks-devel

Re: Review Request 113583: Fix build after enabling C++11 standard

2013-11-02 Thread Volker Krause
--- This is an automatically generated e-mail. To reply, visit: http://git.reviewboard.kde.org/r/113583/#review42872 --- Ship it! Ship It! - Volker Krause On Nov. 2, 2013, 7:38 p.m

Re: Review Request 113583: Fix build after enabling C++11 standard

2013-11-02 Thread Christoph Feck
--- This is an automatically generated e-mail. To reply, visit: http://git.reviewboard.kde.org/r/113583/ --- (Updated Nov. 2, 2013, 7:38 p.m.) Status -- This change has been mark

Re: Review Request 113583: Fix build after enabling C++11 standard

2013-11-02 Thread Commit Hook
--- This is an automatically generated e-mail. To reply, visit: http://git.reviewboard.kde.org/r/113583/#review42871 --- This review has been submitted with commit 1bf8a5c5d69d0528fd0

Re: Review Request 113510: Deprecate methods in kimageio

2013-11-02 Thread David Faure
> On Nov. 2, 2013, 8:27 a.m., David Faure wrote: > > Ship It! > > Alex Merry wrote: > Question: would it not make sense to put this in kde4support? > > David Faure wrote: > Sure, that's exactly the idea (I'll do that once this is committed) > > Alex Merry wrote: > I think I'd misse

plasma-framework buildsys borked

2013-11-02 Thread Gregor Tätzner
Some idea whats going on here? I'm following the standard instructions from wiki*. everything including kdelibs-frameworks builds fine * http://community.kde.org/Frameworks/Building # kdesrc-build running: 'cmake' '/home/test/projects/kde-plasma2/plasma-framework' '-DKDE4_BUILD_TESTS=TRUE' '-DLI

Re: Review Request 113583: Fix build after enabling C++11 standard

2013-11-02 Thread David Faure
--- This is an automatically generated e-mail. To reply, visit: http://git.reviewboard.kde.org/r/113583/#review42869 --- Ship it! Ship It! - David Faure On Nov. 2, 2013, 7:04 p.m.,

Re: Review Request 113500: make KNewStuff build on its own

2013-11-02 Thread Jeremy Whiting
> On Nov. 2, 2013, 7 a.m., Alex Merry wrote: > > tier3/knewstuff/CMakeLists.txt, lines 15-19 > > > > > > Is it necessary to list *all* these, or can we just list the ones that > > are explicitly linked against in

Build failed in Jenkins: kdelibs_frameworks_qt5 #1558

2013-11-02 Thread KDE CI System
See Changes: [martin.sandsmark] Add support for retrying to KIO. -- [...truncated 7414 lines...] : In construc

Re: Review Request 113510: Deprecate methods in kimageio

2013-11-02 Thread Commit Hook
--- This is an automatically generated e-mail. To reply, visit: http://git.reviewboard.kde.org/r/113510/#review42866 --- This review has been submitted with commit 4eb8e017268d0e75a60

Re: Review Request 113510: Deprecate methods in kimageio

2013-11-02 Thread Jeremy Whiting
--- This is an automatically generated e-mail. To reply, visit: http://git.reviewboard.kde.org/r/113510/ --- (Updated Nov. 2, 2013, 7:04 p.m.) Status -- This change has been mark

Re: Review Request 113583: Fix build after enabling C++11 standard

2013-11-02 Thread Christoph Feck
--- This is an automatically generated e-mail. To reply, visit: http://git.reviewboard.kde.org/r/113583/ --- (Updated Nov. 2, 2013, 7:04 p.m.) Review request for KDE Frameworks. Cha

Re: Review Request 113342: Add support for retrying to KIO (frameworks branch)

2013-11-02 Thread Martin Tobias Holmedahl Sandsmark
--- This is an automatically generated e-mail. To reply, visit: http://git.reviewboard.kde.org/r/113342/ --- (Updated Nov. 2, 2013, 6:48 p.m.) Status -- This change has been mark

Re: Review Request 113583: Fix build after enabling C++11 standard

2013-11-02 Thread Alexander Richardson
> On Nov. 2, 2013, 5:03 p.m., Christoph Feck wrote: > > Oh wait... the function has been moved to the std:: namespace for C++11, so > > it would break on older compilers. Is there a standard way to check if we > > have a C++11 compiler? How about using namespace std; inside that function? That

Re: Review Request 113510: Deprecate methods in kimageio

2013-11-02 Thread Alex Merry
> On Nov. 2, 2013, 8:27 a.m., David Faure wrote: > > Ship It! > > Alex Merry wrote: > Question: would it not make sense to put this in kde4support? > > David Faure wrote: > Sure, that's exactly the idea (I'll do that once this is committed) I think I'd missed it was still in the "to be

Re: Review Request 113510: Deprecate methods in kimageio

2013-11-02 Thread David Faure
> On Nov. 2, 2013, 8:27 a.m., David Faure wrote: > > Ship It! > > Alex Merry wrote: > Question: would it not make sense to put this in kde4support? Sure, that's exactly the idea (I'll do that once this is committed) - David --- Thi

Build failed in Jenkins: kdelibs_frameworks_qt5 #1557

2013-11-02 Thread KDE CI System
See Changes: [martin.sandsmark] Remove KTextEditSpellInterface -- [...truncated 7416 lines...] :746:59

Re: Review Request 113578: Remove KTextEditSpellInterface

2013-11-02 Thread Martin Tobias Holmedahl Sandsmark
--- This is an automatically generated e-mail. To reply, visit: http://git.reviewboard.kde.org/r/113578/ --- (Updated Nov. 2, 2013, 4:48 p.m.) Status -- This change has been mark

Re: Review Request 113578: Remove KTextEditSpellInterface

2013-11-02 Thread Alex Merry
--- This is an automatically generated e-mail. To reply, visit: http://git.reviewboard.kde.org/r/113578/#review42838 --- Ship it! Ship It! - Alex Merry On Nov. 2, 2013, 3:23 p.m.,

Re: Review Request 113583: Fix build after enabling C++11 standard

2013-11-02 Thread Christoph Feck
--- This is an automatically generated e-mail. To reply, visit: http://git.reviewboard.kde.org/r/113583/#review42837 --- Oh wait... the function has been moved to the std:: namespace f

Review Request 113583: Fix build after enabling C++11 standard

2013-11-02 Thread Christoph Feck
--- This is an automatically generated e-mail. To reply, visit: http://git.reviewboard.kde.org/r/113583/ --- Review request for KDE Frameworks. Repository: kdelibs Description -

Re: Review Request 113578: Remove KTextEditSpellInterface

2013-11-02 Thread Martin Tobias Holmedahl Sandsmark
--- This is an automatically generated e-mail. To reply, visit: http://git.reviewboard.kde.org/r/113578/ --- (Updated Nov. 2, 2013, 3:23 p.m.) Review request for KDE Frameworks. Cha

Re: Getting ecm files from the ECM package

2013-11-02 Thread Stephen Kelly
Stephen Kelly wrote: > Alexander Neundorf wrote: > >> In case we decide to go this way (i.e. the "my ideal view" plus optional >> downloading), and we should hear Stephens opinion on that, > FYI: http://public.kitware.com/Bug/view.php?id=8471 Thanks, Steve. ___

Re: Review Request 113536: General cleanups of kguiaddons/plugins

2013-11-02 Thread Alex Merry
--- This is an automatically generated e-mail. To reply, visit: http://git.reviewboard.kde.org/r/113536/ --- (Updated Nov. 2, 2013, 2:32 p.m.) Review request for KDE Frameworks. Rep

Re: Review Request 113510: Deprecate methods in kimageio

2013-11-02 Thread Alex Merry
> On Nov. 2, 2013, 8:27 a.m., David Faure wrote: > > Ship It! Question: would it not make sense to put this in kde4support? - Alex --- This is an automatically generated e-mail. To reply, visit: http://git.reviewboard.kde.org/r/113510/#

Re: Review Request 113578: Remove KTextEditSpellInterface

2013-11-02 Thread Alex Merry
--- This is an automatically generated e-mail. To reply, visit: http://git.reviewboard.kde.org/r/113578/#review42835 --- Generally fine, apart from the comments. tier3/ktextwidgets/s

Re: Review Request 113522: Improve documentation in ECMInstallIcons.cmake

2013-11-02 Thread Alex Merry
--- This is an automatically generated e-mail. To reply, visit: http://git.reviewboard.kde.org/r/113522/#review42834 --- Ship it! Looks good, modulo the comments below. modules/ECMI

Re: Review Request 113516: Implement queueing directly in KDialogJobUiDelegate

2013-11-02 Thread Alex Merry
--- This is an automatically generated e-mail. To reply, visit: http://git.reviewboard.kde.org/r/113516/#review42831 --- Ship it! Generally looks good, but see the comments below. Al

Re: Review Request 113500: make KNewStuff build on its own

2013-11-02 Thread Alex Merry
--- This is an automatically generated e-mail. To reply, visit: http://git.reviewboard.kde.org/r/113500/#review42829 --- Looks sane to me, and definitely an improvement, but see my com

Re: Review Request 113393: Get KPassivePopup positioning in a semi-working state

2013-11-02 Thread Alex Merry
--- This is an automatically generated e-mail. To reply, visit: http://git.reviewboard.kde.org/r/113393/ --- (Updated Nov. 2, 2013, 12:46 p.m.) Status -- This change has been dis

Re: Getting ecm files from the ECM package

2013-11-02 Thread Stephen Kelly
Alexander Neundorf wrote: > In case we decide to go this way (i.e. the "my ideal view" plus optional > downloading), and we should hear Stephens opinion on that, My opinion: 1) The current situation with ECM and KF5 is just fine. 2) There are other issues to work on which are more relevant, urge

[extra-cmake-modules] find-modules: Remove FindKF5.cmake.

2013-11-02 Thread Stephen Kelly
Git commit ef93f7db4c1f4c432b053ae3b54d0a745d73c00f by Stephen Kelly. Committed on 02/11/2013 at 12:09. Pushed by skelly into branch 'master'. Remove FindKF5.cmake. The functionality of it is now provided by a KF5Config.cmake file. Remember to remove it from your install prefix to avoid future c

Re: Review Request 113373: Enable C++11 support by default.

2013-11-02 Thread Commit Hook
--- This is an automatically generated e-mail. To reply, visit: http://git.reviewboard.kde.org/r/113373/#review42827 --- This review has been submitted with commit b02c357508afdaf2c14

Re: Review Request 113373: Enable C++11 support by default.

2013-11-02 Thread Volker Krause
--- This is an automatically generated e-mail. To reply, visit: http://git.reviewboard.kde.org/r/113373/ --- (Updated Nov. 2, 2013, 11:32 a.m.) Status -- This change has been mar

Jenkins build is back to normal : kdelibs_frameworks_qt5 #1556

2013-11-02 Thread KDE CI System
See ___ Kde-frameworks-devel mailing list Kde-frameworks-devel@kde.org https://mail.kde.org/mailman/listinfo/kde-frameworks-devel

Build failed in Jenkins: kdelibs_frameworks_qt5 #1555

2013-11-02 Thread KDE CI System
See Changes: [faure] Move "speed" test prog to kio framework, renamed listrecursivetest -- [...truncated 2767 lines...] Automatic moc for target ktextbrowsertest [ 15%] Built target ktextbrowse

Build failed in Jenkins: kdelibs_frameworks_qt5 #1554

2013-11-02 Thread KDE CI System
See Changes: [faure] Move httpfilter to kio_http, which is now its only user. [faure] Split KMountPoint test program out of speed.cpp -- [...truncated 6833 lines...] Building CXX object stagi

Re: Exceptions in KDE Frameworks

2013-11-02 Thread Ivan Čukić
> > AFAIK the only reason to avoid exceptions is an increase in code size, > > which doesn't strike me as a strong problem in 2014. But on the other Modern compilers tend to work very well with exceptions if those are used only for error branches and not as goto statements :) > If there are any

Re: Getting ecm files from the ECM package

2013-11-02 Thread Luca Beltrame
Rex Dieter wrote: > I can vouche for fedora/redhat that the buildsystem does not have internet > access. Likewise, the OBS for openSUSE does not have net access during building. -- Luca Beltrame - KDE Forums team KDE Science supporter GPG key ID: 6E1A4E79 _

Re: Review Request 113510: Deprecate methods in kimageio

2013-11-02 Thread David Faure
--- This is an automatically generated e-mail. To reply, visit: http://git.reviewboard.kde.org/r/113510/#review42823 --- Ship it! Ship It! - David Faure On Nov. 1, 2013, 4:51 p.m.,