Re: Getting ecm files from the ECM package

On Wednesday 30 October 2013, Stephen Kelly wrote: > Hello, > > Soon I will be replacing code like > > find_package(KF5 REQUIRED CMake Compiler InstallDirs) > > with > > include(KDEInstallDirs) > include(KDECMakeSettings) > include(KDECompilerSettings) > > As I've said before, it makes no

Review Request 113511: Install KSslSettings header

--- This is an automatically generated e-mail. To reply, visit: http://git.reviewboard.kde.org/r/113511/ --- Review request for KDE Frameworks. Repository: kdelibs Description -

Build failed in Jenkins: plasma-framework_master_qt5 #875

See Changes: [notmart] removing panels works -- Started by remote host 127.0.0.1 with note: Triggered by commit Building remotely on LinuxSlave - 3 in workspace

Build failed in Jenkins: plasma-framework_master_qt5 #874

See -- Started by remote host 127.0.0.1 with note: Triggered by commit Building remotely on LinuxSlave - 3 in workspace Running Prebuild s

Review Request 113510: Deprecate methods in kimageio

--- This is an automatically generated e-mail. To reply, visit: http://git.reviewboard.kde.org/r/113510/ --- Review request for KDE Frameworks. Repository: kdelibs Description -

Re: Review Request 113505: Add unit tests for KTranscript

--- This is an automatically generated e-mail. To reply, visit: http://git.reviewboard.kde.org/r/113505/#review42710 --- Ship it! Most definitely. The test semantics looks good. I'll

Review Request 113507: Use ecm_mark_as_test for test-only targets

--- This is an automatically generated e-mail. To reply, visit: http://git.reviewboard.kde.org/r/113507/ --- Review request for KDE Frameworks. Repository: kdelibs Description -

Re: Using KSSLSettings from KHTML

On Tuesday 29 October 2013 15:22:18 David Edmundson wrote: > Currently KHTMLPart uses KSSLSettings from KIOCore > > It uses the methods: > warnOnUnencrypted > setWarnOnUnencrypted > > This class does not have the header exported in KIOCore, so whilst it > builds now it will fail when we split. >

Review Request 113506: Add a BUILD_ALL option to superbuild

--- This is an automatically generated e-mail. To reply, visit: http://git.reviewboard.kde.org/r/113506/ --- Review request for KDE Frameworks and Alexander Neundorf. Repository: kdel

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

--- This is an automatically generated e-mail. To reply, visit: http://git.reviewboard.kde.org/r/113500/ --- (Updated Oct. 30, 2013, 9:41 a.m.) Review request for KDE Frameworks. Ch

Review Request 113505: Add unit tests for KTranscript

--- This is an automatically generated e-mail. To reply, visit: http://git.reviewboard.kde.org/r/113505/ --- Review request for KDE Frameworks and Chusslove Illich. Repository: kdelib

Re: Review Request 113432: Install FindDBusMenuQt5.cmake

> On Oct. 30, 2013, 2:21 p.m., Stephen Kelly wrote: > > I still don't understand why a Config file is not added upstream, so I'll > > not 'shipit'. Oops, I thought this was updated, not discarded. - Stephen --- This is an automatically

Re: Review Request 113432: Install FindDBusMenuQt5.cmake

--- This is an automatically generated e-mail. To reply, visit: http://git.reviewboard.kde.org/r/113432/#review42706 --- I still don't understand why a Config file is not added upstrea

Re: Review Request 113432: Install FindDBusMenuQt5.cmake

--- This is an automatically generated e-mail. To reply, visit: http://git.reviewboard.kde.org/r/113432/ --- (Updated Oct. 30, 2013, 3:15 p.m.) Status -- This change has been dis

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

--- This is an automatically generated e-mail. To reply, visit: http://git.reviewboard.kde.org/r/113500/#review42705 --- Patch does not apply here. Can you refresh it to apply on lates

Re: Review Request 113498: Fix KCompletion standalone build

--- This is an automatically generated e-mail. To reply, visit: http://git.reviewboard.kde.org/r/113498/ --- (Updated Oct. 30, 2013, 2:26 p.m.) Status -- This change has been mar

Getting ecm files from the ECM package

Hello, Soon I will be replacing code like find_package(KF5 REQUIRED CMake Compiler InstallDirs) with include(KDEInstallDirs) include(KDECMakeSettings) include(KDECompilerSettings) As I've said before, it makes no sense that we have to find KF5 in order to build KF5, so I'm going to 'fix

Re: cmake and linking issues

Giorgos Tsiapaliokas wrote: > find_package(KDE4Support REQUIRED NO_MODULE) > find_package(KDELibs4 REQUIRED NO_MODULE) > > So should the find_package be written like this? > Is it normal to fail otherwise? There are likely two issues: 1) a bug in cmake fixed in master with http://cmake.org/

Re: Review Request 113406: Add a macro to automatically generate forward headers

--- This is an automatically generated e-mail. To reply, visit: http://git.reviewboard.kde.org/r/113406/#review42700 --- Looks good to me (apart from the expired pastebin links in the

cmake and linking issues

Hello, I have came across to some buildsystem issues. if I don't have the KDE4Support and KDELibs4 with this *exact* order, cmake will fail. Without this order cmake fails in plasma-framework and in plasmate. I have attached the output from cmake. find_package(KDE4Support REQUIRED NO_MODULE) fin

Re: Review Request 113503: make dbus dependency optional in JobWidgets

--- This is an automatically generated e-mail. To reply, visit: http://git.reviewboard.kde.org/r/113503/ --- (Updated Oct. 30, 2013, 10:08 a.m.) Review request for KDE Frameworks, kde

Jenkins build became unstable: kdelibs_stable #879

See ___ Kde-frameworks-devel mailing list Kde-frameworks-devel@kde.org https://mail.kde.org/mailman/listinfo/kde-frameworks-devel

Re: Review Request 113503: make dbus dependency optional in JobWidgets

--- This is an automatically generated e-mail. To reply, visit: http://git.reviewboard.kde.org/r/113503/#review42696 --- for some reason the cmake magic that is supposed to ensure the

Jenkins build is back to stable : kdelibs_stable #878

See ___ Kde-frameworks-devel mailing list Kde-frameworks-devel@kde.org https://mail.kde.org/mailman/listinfo/kde-frameworks-devel