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

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

Build failed in Jenkins: kdelibs_frameworks_qt5 #1554

2013-11-02 Thread KDE CI System
See http://build.kde.org/job/kdelibs_frameworks_qt5/1554/changes 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

Build failed in Jenkins: kdelibs_frameworks_qt5 #1555

2013-11-02 Thread KDE CI System
See http://build.kde.org/job/kdelibs_frameworks_qt5/1555/changes Changes: [faure] Move speed test prog to kio framework, renamed listrecursivetest -- [...truncated 2767 lines...] Automatic moc for target ktextbrowsertest [ 15%] Built target

Jenkins build is back to normal : kdelibs_frameworks_qt5 #1556

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

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

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, urgent

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

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

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.

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.

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:

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 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.

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 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

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 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

Build failed in Jenkins: kdelibs_frameworks_qt5 #1557

2013-11-02 Thread KDE CI System
See http://build.kde.org/job/kdelibs_frameworks_qt5/1557/changes Changes: [martin.sandsmark] Remove KTextEditSpellInterface -- [...truncated 7416 lines...] http://build.kde.org/job/kdelibs_frameworks_qt5/ws/tier4/kde4support/src/kdecore/kurl.cpp:746:59:

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 --- This is

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 sorted

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 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

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.

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

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

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'

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 missed it was

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

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

Jenkins build is back to normal : kdelibs_frameworks_qt5 #1560

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

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 fine. 2)

Build failed in Jenkins: kdelibs_frameworks_qt5 #1563

2013-11-02 Thread KDE CI System
See http://build.kde.org/job/kdelibs_frameworks_qt5/1563/changes 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

Jenkins build is back to normal : kdelibs_frameworks_qt5 #1564

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

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

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 enumerating the

Build failed in Jenkins: plasma-framework_master_qt5 #884

2013-11-02 Thread KDE CI System
See http://build.kde.org/job/plasma-framework_master_qt5/884/changes 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 remotely

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

Re: build.kde.org and `ping`

2013-11-02 Thread Ben Cooksley
On Sun, Nov 3, 2013 at 12:27 PM, David Faure fa...@kde.org 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

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 fa...@kde.org wrote: This error: http://build.kde.org/job/kdelibs_stable/886/testReport/(root)/TestSuite/kp typrocesstest/ is caused by this:

Jenkins build is back to stable : kdelibs_stable #887

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