Re: Fwd: RFC Rules for installation of header files

2013-11-24 Thread Martin Klapetek
On Sun, Nov 24, 2013 at 11:52 PM, Stephen Kelly wrote: > Aurélien Gâteau wrote: > > > My webmail is screwed up and by default sets the From > > field to the sender of the mail I am replying to. > > This is the most hilarious email-related feature I've ever heard of :). > That's actually the mail

Build failed in Jenkins: plasma-framework_master_qt5 #914

2013-11-24 Thread KDE CI System
See Changes: [scripty] SVN_SILENT made messages (.desktop file) -- Started by remote host 127.0.0.1 with note: Triggered by commit Building remotely on LinuxSlave - 3 in workspace

Re: RFC Rules for installation of header files

2013-11-24 Thread Stephen Kelly
Aurélien Gâteau wrote: > Looks like we have an agreement then. Let me recap: > > # Installation dir > > All header files are installed in $PREFIX/include/KF5/$Framework. > This includes 'k' prefixed headers like kfoo and KFoo, > non-prefixed headers such as bar.h and Bar, as well as special heade

Re: Review Request 114076: Set KDE4_INCLUDES in KDE4SupportConfig

2013-11-24 Thread David Faure
--- This is an automatically generated e-mail. To reply, visit: http://git.reviewboard.kde.org/r/114076/ --- (Updated Nov. 24, 2013, 11:27 p.m.) Status -- This change has been di

Jenkins build became unstable: kdelibs_frameworks_qt5 #1746

2013-11-24 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 114076: Set KDE4_INCLUDES in KDE4SupportConfig

2013-11-24 Thread David Faure
On Monday 25 November 2013 00:00:38 Stephen Kelly wrote: > David Faure wrote: > > On Sunday 24 November 2013 23:11:25 Stephen Kelly wrote: > >> David Faure wrote: > >> > It doesn't link to kde4support, and KConfigCore doesn't install > >> > forwarding headers yet. So this is a temporary measure, th

Re: Review Request 114076: Set KDE4_INCLUDES in KDE4SupportConfig

2013-11-24 Thread Stephen Kelly
David Faure wrote: > On Sunday 24 November 2013 23:11:25 Stephen Kelly wrote: >> David Faure wrote: >> > It doesn't link to kde4support, and KConfigCore doesn't install >> > forwarding headers yet. So this is a temporary measure, the real fix >> > being to generate and install forwarding headers;

Re: Review Request 114076: Set KDE4_INCLUDES in KDE4SupportConfig

2013-11-24 Thread David Faure
On Sunday 24 November 2013 23:11:25 Stephen Kelly wrote: > David Faure wrote: > > It doesn't link to kde4support, and KConfigCore doesn't install forwarding > > headers yet. So this is a temporary measure, the real fix being to > > generate and install forwarding headers; but this fix allows to mov

Re: Fwd: RFC Rules for installation of header files

2013-11-24 Thread Stephen Kelly
Aurélien Gâteau wrote: > My webmail is screwed up and by default sets the From > field to the sender of the mail I am replying to. This is the most hilarious email-related feature I've ever heard of :). Thanks, Steve. ___ Kde-frameworks-devel mailin

Review Request 114090: [tier2/ki18n] Improve dependency specification

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

Re: Review Request 114076: Set KDE4_INCLUDES in KDE4SupportConfig

2013-11-24 Thread Stephen Kelly
David Faure wrote: > It doesn't link to kde4support, and KConfigCore doesn't install forwarding > headers yet. So this is a temporary measure, the real fix being to > generate and install forwarding headers; but this fix allows to move the > two things forward independently. Given the issues that

Re: Review Request 114043: [tier3/kunitconversion] Improve dependency specification

2013-11-24 Thread Johannes Huber
--- This is an automatically generated e-mail. To reply, visit: http://git.reviewboard.kde.org/r/114043/ --- (Updated Nov. 24, 2013, 10:07 p.m.) Status -- This change has been ma

Re: Review Request 114043: [tier3/kunitconversion] Improve dependency specification

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

Re: Review Request 114076: Set KDE4_INCLUDES in KDE4SupportConfig

2013-11-24 Thread David Faure
> On Nov. 24, 2013, 9:20 p.m., Stephen Kelly wrote: > > tier4/kde4support/KDE4SupportConfig.cmake.in, line 38 > > > > > > Why is it necessary to set KDE4_INCLUDES at all? The correct include > > dirs should all co

Re: Review Request 113910: Add basic KArchive examples

2013-11-24 Thread Stephen Kelly
--- This is an automatically generated e-mail. To reply, visit: http://git.reviewboard.kde.org/r/113910/#review44355 --- > Looking for feedback on: > License? Currently BSD. BSD is on

Re: Review Request 114076: Set KDE4_INCLUDES in KDE4SupportConfig

2013-11-24 Thread Stephen Kelly
> On Nov. 24, 2013, 9:20 p.m., Stephen Kelly wrote: > > Nothing should need find_package(KDELibs4) at all anymore. Uses of it should be removed (or replaced with kde4support, if that's not already there), and then KDELibs4Config.cmake should be removed from kdelibs.git/frameworks. - Stephen

Re: Review Request 114076: Set KDE4_INCLUDES in KDE4SupportConfig

2013-11-24 Thread Stephen Kelly
--- This is an automatically generated e-mail. To reply, visit: http://git.reviewboard.kde.org/r/114076/#review44353 --- tier4/kde4support/KDE4SupportConfig.cmake.in

Re: K4AboutData

2013-11-24 Thread Stephen Kelly
David Faure wrote: > Conclusion: > I think it's a lot simpler to keep things as is, I can't say I understand fully, but thanks for looking into it! Steve. ___ Kde-frameworks-devel mailing list Kde-frameworks-devel@kde.org https://mail.kde.org/mailman

Re: further coinstallability

2013-11-24 Thread Stephen Kelly
Jonathan Riddell wrote: > developer files matter to some people to be co-installable. > Header files and others can probably be moved to a > separate directory Can you be more specific about why this is easy with header files? Does this refer to installing to a include/KF5 directory? > but c

Re: [Kde-pim] PIM Sprint: Porting PIM to Frameworks

2013-11-24 Thread Stephen Kelly
Kevin Ottens wrote: >> Tiers above 1 are not especially relevant. A tier is a fluid label. It >> doesn't matter when we are using split repos. > > If you could stop spreading that it'd be nice. It's not because you see no > value in tier 2 and 3 distinction that this value doesn't exist. I think

Re: Review Request 114076: Set KDE4_INCLUDES in KDE4SupportConfig

2013-11-24 Thread Alexander Richardson
--- This is an automatically generated e-mail. To reply, visit: http://git.reviewboard.kde.org/r/114076/#review44349 --- I can confirm that it works with okteta, too - Alexander Richa

Re: Dot story on KDE contributions to Qt 5.2

2013-11-24 Thread Mark Gaiser
On Sun, Nov 24, 2013 at 7:57 PM, David Faure wrote: > On Sunday 24 November 2013 19:48:33 Mark Gaiser wrote: >> On Sun, Nov 24, 2013 at 7:34 PM, David Faure wrote: >> > On Saturday 16 November 2013 10:54:53 John Layt wrote: >> >> Hi, >> >> >> >> There was a discussion on the promo list a couple o

Re: Review Request 114044: Remove unused dependencies in kcoreaddons

2013-11-24 Thread David Faure
--- This is an automatically generated e-mail. To reply, visit: http://git.reviewboard.kde.org/r/114044/#review44347 --- Ship it! Probably historical. Well spotted! - David Faure O

Re: Dot story on KDE contributions to Qt 5.2

2013-11-24 Thread John Layt
On 24 November 2013 19:34, David Faure wrote: > > Is this supposed to be "added for Qt 5.2" only, or "added in Qt 5.0, 5.1 and > 5.2"? > This email says the former, but the document contains a lot of the earlier > stuff (mimetypes, qstandardpaths, ...) > > See the above doc for more, I wrote more

Re: Review Request 114043: [tier3/kunitconversion] Improve dependency specification

2013-11-24 Thread David Faure
--- This is an automatically generated e-mail. To reply, visit: http://git.reviewboard.kde.org/r/114043/#review44342 --- Ship it! Ship It! - David Faure On Nov. 22, 2013, 6:45 p.m.

Re: Review Request 113975: Make some KSSL files public API

2013-11-24 Thread David Faure
--- This is an automatically generated e-mail. To reply, visit: http://git.reviewboard.kde.org/r/113975/#review44341 --- Fixed differently, see a2bd9380ecc03347293943300c03c81e62b90ec8

Re: Dot story on KDE contributions to Qt 5.2

2013-11-24 Thread David Faure
On Sunday 24 November 2013 19:48:33 Mark Gaiser wrote: > On Sun, Nov 24, 2013 at 7:34 PM, David Faure wrote: > > On Saturday 16 November 2013 10:54:53 John Layt wrote: > >> Hi, > >> > >> There was a discussion on the promo list a couple of weeks back about > >> doing a Dot story to coincide with

Re: Dot story on KDE contributions to Qt 5.2

2013-11-24 Thread Mark Gaiser
On Sun, Nov 24, 2013 at 7:34 PM, David Faure wrote: > On Saturday 16 November 2013 10:54:53 John Layt wrote: >> Hi, >> >> There was a discussion on the promo list a couple of weeks back about >> doing a Dot story to coincide with the release of Qt 5.2 highlighting >> KDE's contributions. Jos star

K4AboutData

2013-11-24 Thread David Faure
Hi Stephen, as promised I had another look at the KAboutData vs K4AboutData situation, in order to attempt to minimize the SIC. Context: KCmdLineArgs uses K4AboutData rather than KAboutData: K4AboutData keeps the old mechanism for delayed translations (I18N_NOOP), while KAboutData is the one u

Re: Dot story on KDE contributions to Qt 5.2

2013-11-24 Thread David Faure
On Saturday 16 November 2013 10:54:53 John Layt wrote: > Hi, > > There was a discussion on the promo list a couple of weeks back about > doing a Dot story to coincide with the release of Qt 5.2 highlighting > KDE's contributions. Jos started an Etherpad at > https://notes.kde.org/p/Platform5DotSt

Re: [Kde-pim] PIM Sprint: Porting PIM to Frameworks

2013-11-24 Thread Kevin Ottens
Hello, On Tuesday 19 November 2013 22:59:15 Stephen Kelly wrote: > John Layt wrote: > > It was agreed that the ideal plan for kdepimlibs was to split all the > > libraries up and make them all stand-alone Frameworks, aiming for Tier > > 2 or even Tier 1 where possible. > > Tiers above 1 are not es

further coinstallability

2013-11-24 Thread Jonathan Riddell
I had a chat here at Munich BSP with Maxy from Debian and Sput from Gentoo about coinstallability. It was pointed out that dbus interfaces will clash at runtime if the interface has changed at all or it expects to talk to a different daemon or something. org.kde.kded has been renamed to org.kde.kd

Re: Dot story on KDE contributions to Qt 5.2

2013-11-24 Thread Jos Poortvliet
On Saturday 16 November 2013 10:54:53 John Layt wrote: > Hi, > > There was a discussion on the promo list a couple of weeks back about > doing a Dot story to coincide with the release of Qt 5.2 highlighting > KDE's contributions. Jos started an Etherpad at > https://notes.kde.org/p/Platform5DotSt

Re: further coinstallability

2013-11-24 Thread Johannes Huber
Am Samstag, 23. November 2013, 18:19:53 schrieb Jonathan Riddell: > I had a chat here at Munich BSP with Maxy from Debian and Sput from Gentoo > about coinstallability. > > It was pointed out that dbus interfaces will clash at runtime if the > interface has changed at all or it expects to talk to a

Review Request 114076: Set KDE4_INCLUDES in KDE4SupportConfig

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

Re: Review Request 113805: Do not change the build types available with cmake

2013-11-24 Thread David Faure
> On Nov. 12, 2013, 7:24 p.m., Alexander Neundorf wrote: > > IMO the patch as it is is not good. > > > > Several things: > > 1) This file, is not mandatory at all with KDE frameworks. > > You can build applications using KDE frameworks libraries without it. You > > (the developer of the applica

Re: Review Request 114038: Integrate KAboutData and QCommandLineParser

2013-11-24 Thread David Faure
--- This is an automatically generated e-mail. To reply, visit: http://git.reviewboard.kde.org/r/114038/#review44296 --- Great idea, thanks. tier1/kcoreaddons/src/lib/kaboutdata.h

Re: Review Request 113935: Clean up FindOpenEXR.cmake and move to tier1/kguiaddons/cmake

2013-11-24 Thread David Faure
--- This is an automatically generated e-mail. To reply, visit: http://git.reviewboard.kde.org/r/113935/#review44301 --- kde-runtime/kioslave/thumbnail/CMakeLists.txt needs this too.

Re: Review Request 114052: Skip tests which need a window manager running

2013-11-24 Thread Martin Gräßlin
--- This is an automatically generated e-mail. To reply, visit: http://git.reviewboard.kde.org/r/114052/ --- (Updated Nov. 24, 2013, 11:35 a.m.) Status -- This change has been di