Jenkins build is back to normal : plasma-framework_master_qt5 #883

2013-11-01 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: Exceptions in KDE Frameworks

2013-11-01 Thread Mirko Boehm
-BEGIN PGP SIGNED MESSAGE- Hash: SHA1 On 11/01/2013 11:41 PM, Michael Pyne wrote: >> 1) Are we sure we want to disable exceptions in libraries released in 2014? >> > 2) What is the _BLBAFASEL_EXCEPTIONS_ENABLED_ preprocessor variable that I >> > can use to implemented the exception- and no

Re: Getting ecm files from the ECM package

2013-11-01 Thread Rex Dieter
Mirko Boehm wrote: > Pretty strong language. Not much proof. Do you *know* how Red Hat or SuSE > build their packages? I can vouche for fedora/redhat that the buildsystem does not have internet access. -- rex ___ Kde-frameworks-devel mailing list Kde

Re: Exceptions in KDE Frameworks

2013-11-01 Thread Michael Pyne
On Fri, November 1, 2013 17:30:02 Mirko Boehm wrote: > Hi, > > in ThreadWeaver, I would like to be able to catch exceptions thrown from the > run() method of Jobs. You know we cannot trust the user :-) I can work > around if exceptions are disabled in the compiler if I know about it. What > I woul

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

2013-11-01 Thread Martin Tobias Holmedahl Sandsmark
--- This is an automatically generated e-mail. To reply, visit: http://git.reviewboard.kde.org/r/113342/ --- (Updated Nov. 1, 2013, 11:31 p.m.) Review request for KDE Frameworks, kdel

Review Request 113578: Remove KTextEditSpellInterface

2013-11-01 Thread Martin Tobias Holmedahl Sandsmark
--- This is an automatically generated e-mail. To reply, visit: http://git.reviewboard.kde.org/r/113578/ --- Review request for KDE Frameworks. Repository: kdelibs Description -

Frameworks in api.kde.org

2013-11-01 Thread Albert Astals Cid
Hi guys, frameworks seem to be broken in api.kde.org I got to http://api.kde.org/frameworks/kdelibs-apidocs/ but then i wanted to recommend the awesome KConfig and ended in http://api.kde.org/frameworks/kdelibs-apidocs/kdecore/html/classKConfig.html Which is a sad 404. Any chance we can fix t

Re: Getting ecm files from the ECM package

2013-11-01 Thread Michael Pyne
On Fri, November 1, 2013 22:41:29 Mirko Boehm wrote: > In my opinion, a central repository of community maintained find module > packages has a chance of making a real difference. I'm not at all opposed to a centralized module of Find* cmake modules, in case I left that impression. But this centr

Re: Getting ecm files from the ECM package

2013-11-01 Thread Sune Vuorela
On 2013-11-01, Mirko Boehm wrote: > To me, "build systems should not download anything" sounds like a movie > from the 80ies. To me, "build systems fetches code and executes it" sounds like a bad horror movie. > For example, from a developers point of view, what is the difference between > me

Re: Getting ecm files from the ECM package

2013-11-01 Thread Sune Vuorela
On 2013-11-01, Alexander Neundorf wrote: > Anyway, attached is a quick experiment, which adds the 3 KDE*.cmake files= > from=20 > extra-cmake-modules/ to kf5umbrella/, by that turning it into tier0/, wit= > h the=20 > optional ability (-DWITH_ECM) to download ECM and install ECM when buildi= > ng

Re: Getting ecm files from the ECM package

2013-11-01 Thread Nicolás Alvarez
2013/11/1 Mirko Boehm : >> I'm also surprised at "Almost everybody has internet access for build >> machines". Is there *any* Linux distro where that's the case?? > > Pretty strong language. Not much proof. Do you *know* how Red Hat or SuSE > build their packages? No I don't. I didn't say "no dis

Re: Getting ecm files from the ECM package

2013-11-01 Thread Mirko Boehm
-BEGIN PGP SIGNED MESSAGE- Hash: SHA1 On 11/01/2013 06:46 PM, Nicolás Alvarez wrote: >>> and so forth. That would be a real breakthrough. It is related to the >>> >> approach taken by Maven and others. All it takes is a built-in way for >>> >> CMake to download the find_modules into a cach

Exceptions in KDE Frameworks

2013-11-01 Thread Mirko Boehm
-BEGIN PGP SIGNED MESSAGE- Hash: SHA1 Hi, in ThreadWeaver, I would like to be able to catch exceptions thrown from the run() method of Jobs. You know we cannot trust the user :-) I can work around if exceptions are disabled in the compiler if I know about it. What I would like to be a

Re: Getting ecm files from the ECM package

2013-11-01 Thread Alexander Neundorf
On Friday 01 November 2013, Michael Pyne wrote: > On Fri, November 1, 2013 20:06:43 Alexander Neundorf wrote: > > Anyway, attached is a quick experiment, which adds the 3 KDE*.cmake files > > from extra-cmake-modules/ to kf5umbrella/, by that turning it into > > tier0/, with the optional ability (-

Re: Getting ecm files from the ECM package

2013-11-01 Thread Michael Pyne
On Fri, November 1, 2013 20:06:43 Alexander Neundorf wrote: > Anyway, attached is a quick experiment, which adds the 3 KDE*.cmake files > from extra-cmake-modules/ to kf5umbrella/, by that turning it into tier0/, > with the optional ability (-DWITH_ECM) to download ECM and install ECM when > buildi

Re: Getting ecm files from the ECM package

2013-11-01 Thread Alexander Neundorf
On Friday 01 November 2013, Alexander Neundorf wrote: ... > If that option was enabled when installing kf5umbrella, kf5umbrella will > load that embedded ECM when asked for ECM. > In this quick hack attached here > find_package(KF5Umbrella) > unconditionally loads KDECMakeSettings.cmake. > The vers

Re: Getting ecm files from the ECM package

2013-11-01 Thread Alexander Neundorf
On Friday 01 November 2013, Treeve Jelbert wrote: > On Friday 01 November 2013 15:46:30 Nicolás Alvarez wrote: ... > > Maven is a disgusting monstrosity used by the Java crowd where > > backwards compatibility rarely exists, and the approach to make things > > not break is to make packages depend o

Re: Getting ecm files from the ECM package

2013-11-01 Thread Michael Pyne
> On 11/01/2013 10:46 AM, Alexander Neundorf wrote: > > [1] Why not merge that into CMake ? For quicker releases, and even easier > > contributing. Also Bill once said that in hindsight he would have prefered > > if > > cmake would not ship any find-modules itself at all. So one could imagine > > t

Build failed in Jenkins: plasma-framework_master_qt5 #882

2013-11-01 Thread KDE CI System
See Changes: [christoph] Fix build with latest ThreadWeaver -- [...truncated 275 lines...]

Re: Review Request 113535: Fix build with latest ThreadWeaver

2013-11-01 Thread Christoph Feck
--- This is an automatically generated e-mail. To reply, visit: http://git.reviewboard.kde.org/r/113535/ --- (Updated Nov. 1, 2013, 6:10 p.m.) Status -- This change has been mark

Re: Review Request 113535: Fix build with latest ThreadWeaver

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

Re: Getting ecm files from the ECM package

2013-11-01 Thread Treeve Jelbert
On Friday 01 November 2013 15:46:30 Nicolás Alvarez wrote: > 2013/11/1 Kevin Ottens : > > Hello, > > > > On Friday 01 November 2013 11:23:14 Mirko Boehm wrote: > >> On 11/01/2013 10:46 AM, Alexander Neundorf wrote: > >> > [1] Why not merge that into CMake ? For quicker releases, and even > >> > ea

Re: Getting ecm files from the ECM package

2013-11-01 Thread Nicolás Alvarez
2013/11/1 Kevin Ottens : > Hello, > > On Friday 01 November 2013 11:23:14 Mirko Boehm wrote: >> On 11/01/2013 10:46 AM, Alexander Neundorf wrote: >> > [1] Why not merge that into CMake ? For quicker releases, and even easier >> > contributing. Also Bill once said that in hindsight he would have pre

Re: Review Request 113530: Adapt KDED to use DBus to communicate with KSplash

2013-11-01 Thread Àlex Fiestas
--- This is an automatically generated e-mail. To reply, visit: http://git.reviewboard.kde.org/r/113530/#review42776 --- Ship it! Awesome! It looks good. - Àlex Fiestas On Nov. 1,

Re: Review Request 113533: Adapt KCMInit to use DBus to communicate with KSplash

2013-11-01 Thread Àlex Fiestas
--- This is an automatically generated e-mail. To reply, visit: http://git.reviewboard.kde.org/r/113533/#review42775 --- Ship it! Looks good! - Àlex Fiestas On Nov. 1, 2013, 12:55

Re: Review Request 113510: Deprecate methods in kimageio

2013-11-01 Thread Jeremy Whiting
--- This is an automatically generated e-mail. To reply, visit: http://git.reviewboard.kde.org/r/113510/ --- (Updated Nov. 1, 2013, 10:51 a.m.) Review request for KDE Frameworks. Ch

Jenkins build became unstable: kdelibs_stable #885

2013-11-01 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: Getting ecm files from the ECM package

2013-11-01 Thread Kevin Ottens
On Friday 01 November 2013 13:37:56 Kevin Ottens wrote: > On Friday 01 November 2013 12:53:17 Alexander Neundorf wrote: > > On Friday 01 November 2013, Kevin Ottens wrote: > > > Hello, > > > > > > On Friday 01 November 2013 11:23:14 Mirko Boehm wrote: > > > > On 11/01/2013 10:46 AM, Alexander Neun

Re: Getting ecm files from the ECM package

2013-11-01 Thread Kevin Ottens
On Friday 01 November 2013 17:04:12 Sebastian Kügler wrote: > On Friday, November 01, 2013 11:57:35 Kevin Ottens wrote: > > > Then it is time to think of a way to integrate cmake with the separate > > > source of find_modules. Algorithmically, it would look like > > > > > > PROJECT(MyApplication)

Re: Getting ecm files from the ECM package

2013-11-01 Thread Sebastian Kügler
On Friday, November 01, 2013 11:57:35 Kevin Ottens wrote: > > Then it is time to think of a way to integrate cmake with the separate > > source of find_modules. Algorithmically, it would look like > > > > PROJECT(MyApplication) > > FIND_MODULES_REPOSITORY("http://ecm.kde.org";) > > FIND_PACKAGES(KF

Review Request 113536: General cleanups of kguiaddons/plugins

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

Re: Review Request 113535: Fix build with latest ThreadWeaver

2013-11-01 Thread Mirko Boehm
--- This is an automatically generated e-mail. To reply, visit: http://git.reviewboard.kde.org/r/113535/#review42771 --- Ship it! Looks good to me, approved. - Mirko Boehm On Nov.

Re: Getting ecm files from the ECM package

2013-11-01 Thread Alexander Neundorf
On Friday 01 November 2013, Mirko Boehm wrote: > On 11/01/2013 01:53 PM, Sune Vuorela wrote: > >> So far we chose the "have it in cmake/ecm" route. If we had what Mirko = > >> > >> > refers=20 > >> > to, then that'd open the door to another solution. > > > > And it would open the first door towar

Re: Review Request 113535: Fix build with latest ThreadWeaver

2013-11-01 Thread Christoph Feck
--- This is an automatically generated e-mail. To reply, visit: http://git.reviewboard.kde.org/r/113535/ --- (Updated Nov. 1, 2013, 1:31 p.m.) Review request for KDE Frameworks and Pl

Review Request 113535: Fix build with latest ThreadWeaver

2013-11-01 Thread Christoph Feck
--- This is an automatically generated e-mail. To reply, visit: http://git.reviewboard.kde.org/r/113535/ --- Review request for KDE Frameworks and Plasma. Repository: plasma-framework

Re: Getting ecm files from the ECM package

2013-11-01 Thread Sune Vuorela
On 2013-11-01, Mirko Boehm wrote: > Anyone up for hacking this up next week? I am available starting Monday > afternoon. Before you start hacking, please consider the following: http vs https? should http even be allowed? certificate handling? how to do ssl? a library? will cmake accept a

Re: Getting ecm files from the ECM package

2013-11-01 Thread Alexander Neundorf
On Friday 01 November 2013, Mirko Boehm wrote: > On 11/01/2013 01:37 PM, Kevin Ottens wrote: > >> if a package foo uses ecm (or that non existing tier0 cmake package) > >> itself, > >> > >> > this does not mean that another package which uses foo, also needs ecm > >> > (or that tier0 package) at b

Re: Getting ecm files from the ECM package

2013-11-01 Thread Mirko Boehm
-BEGIN PGP SIGNED MESSAGE- Hash: SHA1 On 11/01/2013 01:53 PM, Sune Vuorela wrote: >> So far we chose the "have it in cmake/ecm" route. If we had what Mirko = >> > refers=20 >> > to, then that'd open the door to another solution. > And it would open the first door towards alienating linux d

Review Request 113533: Adapt KCMInit to use DBus to communicate with KSplash

2013-11-01 Thread Martin Klapetek
--- This is an automatically generated e-mail. To reply, visit: http://git.reviewboard.kde.org/r/113533/ --- Review request for kde-workspace and KDE Frameworks. Repository: kde-works

Re: Getting ecm files from the ECM package

2013-11-01 Thread Sune Vuorela
On 2013-11-01, Kevin Ottens wrote: > So far we chose the "have it in cmake/ecm" route. If we had what Mirko = > refers=20 > to, then that'd open the door to another solution. And it would open the first door towards alienating linux distributions. Of course, we could say "and so what?". But that

Jenkins build is back to normal : kdelibs_frameworks_qt5 #1552

2013-11-01 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: Getting ecm files from the ECM package

2013-11-01 Thread Mirko Boehm
-BEGIN PGP SIGNED MESSAGE- Hash: SHA1 On 11/01/2013 01:37 PM, Kevin Ottens wrote: >> if a package foo uses ecm (or that non existing tier0 cmake package) itself, >> > this does not mean that another package which uses foo, also needs ecm (or >> > that tier0 package) at buildtime. > Well, i

Build failed in Jenkins: kdelibs_frameworks_qt5 #1551

2013-11-01 Thread KDE CI System
See Changes: [mklapetek] Fix build -- [...truncated 3538 lines...] [ 28%] Building CXX object tier1/kjs/src/kjs/CMakeFiles/icemaker.dir/bytecode/generator/lexer.cpp.o [ 28%] Building CXX objec

Re: Getting ecm files from the ECM package

2013-11-01 Thread Kevin Ottens
On Friday 01 November 2013 12:53:17 Alexander Neundorf wrote: > On Friday 01 November 2013, Kevin Ottens wrote: > > Hello, > > > > On Friday 01 November 2013 11:23:14 Mirko Boehm wrote: > > > On 11/01/2013 10:46 AM, Alexander Neundorf wrote: > > > > [1] Why not merge that into CMake ? For quicker

Re: Review Request 113530: Adapt KDED to use DBus to communicate with KSplash

2013-11-01 Thread Martin Klapetek
--- This is an automatically generated e-mail. To reply, visit: http://git.reviewboard.kde.org/r/113530/ --- (Updated Nov. 1, 2013, 12:38 p.m.) Review request for KDE Frameworks. Ch

Re: Review Request 113530: Adapt KDED to use DBus to communicate with KSplash

2013-11-01 Thread Martin Klapetek
--- This is an automatically generated e-mail. To reply, visit: http://git.reviewboard.kde.org/r/113530/ --- (Updated Nov. 1, 2013, 12:37 p.m.) Review request for KDE Frameworks. Re

Build failed in Jenkins: kdelibs_frameworks_qt5 #1550

2013-11-01 Thread KDE CI System
See Changes: [mklapetek] Don't build UDisk backends if UDev libs weren't found [mklapetek] Don't print useless CMake variables while configuring KNewStuff3 -- [...truncated 611 lines...] "ht

Re: Getting ecm files from the ECM package

2013-11-01 Thread Mirko Boehm
-BEGIN PGP SIGNED MESSAGE- Hash: SHA1 On 11/01/2013 12:56 PM, Alexander Neundorf wrote: >> As for internet access, I doubt this will be a problem. > there would have to be a way to do this without internet access. > There are setups, where developers simply don't want to depend on some >

Build failed in Jenkins: kdelibs_frameworks_qt5 #1549

2013-11-01 Thread KDE CI System
See Changes: [steveire] Cleanup knewstuff a bit. [steveire] There's no such KIOCore package. -- [...truncated 612 lines...] "httpheadertokenizetest". Use the target name directly with add

Review Request 113530: Adapt KDED to use DBus to communicate with KSplash

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

Re: Getting ecm files from the ECM package

2013-11-01 Thread Stephen Kelly
Alexander Neundorf wrote: > if a package foo uses ecm (or that non existing tier0 cmake package) > itself, this does not mean that another package which uses foo, also needs > ecm (or that tier0 package) at buildtime. Plasma does currently, but that might be a bug. Thanks, Steve.

Build failed in Jenkins: plasma-framework_master_qt5 #881

2013-11-01 Thread KDE CI System
See Changes: [stephen.kelly] Find the new KF5Config file, not the old FindKF5 file. -- [...truncated 417 lines...] Scanning dependencies of target fullmodelaccesstest Generating moc_appleti

Re: Getting ecm files from the ECM package

2013-11-01 Thread Alexander Neundorf
On Friday 01 November 2013, Mirko Boehm wrote: > On 11/01/2013 11:25 AM, Sune Vuorela wrote: > >> and so forth. That would be a real breakthrough. It is related to the > >> approach taken by Maven and others. All it takes is a built-in way for > >> CMake to download the find_modules into a cache lo

Re: Getting ecm files from the ECM package

2013-11-01 Thread Alexander Neundorf
On Friday 01 November 2013, Kevin Ottens wrote: > Hello, > > On Friday 01 November 2013 11:23:14 Mirko Boehm wrote: > > On 11/01/2013 10:46 AM, Alexander Neundorf wrote: > > > [1] Why not merge that into CMake ? For quicker releases, and even > > > easier contributing. Also Bill once said that in

Re: Getting ecm files from the ECM package

2013-11-01 Thread Mirko Boehm
-BEGIN PGP SIGNED MESSAGE- Hash: SHA1 On 11/01/2013 11:25 AM, Sune Vuorela wrote: >> and so forth. That would be a real breakthrough. It is related to the >> approach taken by Maven and others. All it takes is a built-in way for CMake >> to download the find_modules into a cache location

Re: Getting ecm files from the ECM package

2013-11-01 Thread Sune Vuorela
On 2013-11-01, Mirko Boehm wrote: > and so forth. That would be a real breakthrough. It is related to the > approach taken by Maven and others. All it takes is a built-in way for CMake > to download the find_modules into a cache location and update them when > needed, or on request. > > This w

Re: Getting ecm files from the ECM package

2013-11-01 Thread Kevin Ottens
Hello, On Friday 01 November 2013 11:23:14 Mirko Boehm wrote: > On 11/01/2013 10:46 AM, Alexander Neundorf wrote: > > [1] Why not merge that into CMake ? For quicker releases, and even easier > > contributing. Also Bill once said that in hindsight he would have prefered > > if cmake would not ship

Re: Getting ecm files from the ECM package

2013-11-01 Thread Mirko Boehm
-BEGIN PGP SIGNED MESSAGE- Hash: SHA1 On 11/01/2013 10:46 AM, Alexander Neundorf wrote: > [1] Why not merge that into CMake ? For quicker releases, and even easier > contributing. Also Bill once said that in hindsight he would have prefered if > cmake would not ship any find-modules itse

Re: Getting ecm files from the ECM package

2013-11-01 Thread Alexander Neundorf
On Friday 01 November 2013, Alexander Neundorf wrote: > As a fact, all KF5 frameworks on these three files, including the ...all KF5 frameworks *depend* on these three files, including the... ___ Kde-frameworks-devel mailing list Kde-frameworks-devel@kd

Re: Review Request 113506: Add a sb_all target and sb_$framework targets to make it easier to build frameworks standalone

2013-11-01 Thread Aurélien Gâteau
--- This is an automatically generated e-mail. To reply, visit: http://git.reviewboard.kde.org/r/113506/ --- (Updated Nov. 1, 2013, 10:58 a.m.) Status -- This change has been dis

Re: Getting ecm files from the ECM package

2013-11-01 Thread Alexander Neundorf
On Thursday 31 October 2013, David Faure wrote: > On Thursday 31 October 2013 17:54:06 Alexander Neundorf wrote: > > IOW, those three files exist only for KF5, and no other reason. > > This contradicts "use KDECMakeSettings.cmake in gammaray if you want > automatic RPATH handling" as you told me s

Re: Review Request 113510: Deprecate methods in kimageio

2013-11-01 Thread David Faure
--- This is an automatically generated e-mail. To reply, visit: http://git.reviewboard.kde.org/r/113510/#review42758 --- Missing #ifndef KDE_NO_DEPRECATED around all the deprecated met

Re: Flaky kcodecs test depending on qt configuration

2013-11-01 Thread Sune Vuorela
On 2013-11-01, David Faure wrote: > On Wednesday 23 October 2013 21:43:59 Sune Vuorela wrote: >> Hi >> >> Depending on the Qt configuration (built with or without ICU), the >> KCharsetsTest::testEncodingNames() test fails (in the #if) block. >> >> If Qt built with ICU, the tests succeeds and the

Re: Review Request 113158: Implement queueing directly in KDialogJobUiDelegate

2013-11-01 Thread David Faure
> On Oct. 14, 2013, 9:05 a.m., Kevin Ottens wrote: > > I'm not sold on bastardizing QErrorMessage for that feature. The intent was > > more to have some code similar to what KDialogQueue (moved to KDE4Support) > > was doing directly in KDialogJobUiDelegate implementation. I'm surprised by this

Re: Flaky kcodecs test depending on qt configuration

2013-11-01 Thread David Faure
On Wednesday 23 October 2013 21:43:59 Sune Vuorela wrote: > Hi > > Depending on the Qt configuration (built with or without ICU), the > KCharsetsTest::testEncodingNames() test fails (in the #if) block. > > If Qt built with ICU, the tests succeeds and the ISO 8859-16, jis7 and > winsami2 codecs ar