Re: pkg-config files

2013-12-27 Thread Alexander Neundorf
On Thursday 26 December 2013, Alex Merry wrote: How much do we care about users of frameworks who don't use CMake? Do we want to provide pkg-config files? How easy is it to do so? (My personal answers would be: enough to put in a little effort, but not a huge one; yes, if it's easy; there

Re: kde4support: kstandarddirs_unittest fix

2013-12-27 Thread Alex Merry
On 27/12/13 11:04, Kevin Ottens wrote: On Monday 23 December 2013 14:51:19 Alex Merry wrote: Actually, I think we should just install application files directly in share/applications. (ie: change the value of XDG_APPS_INSTALL_DIR in KDEInstallDirs.cmake). It makes everyone's lives more

Re: kde4support: kstandarddirs_unittest fix

2013-12-27 Thread Kevin Ottens
On Friday 27 December 2013 11:22:57 Alex Merry wrote: On 27/12/13 11:04, Kevin Ottens wrote: On Monday 23 December 2013 14:51:19 Alex Merry wrote: Actually, I think we should just install application files directly in share/applications. (ie: change the value of XDG_APPS_INSTALL_DIR in

Re: kde4support: kstandarddirs_unittest fix

2013-12-27 Thread Alex Merry
On 27/12/13 11:30, Kevin Ottens wrote: On Friday 27 December 2013 11:22:57 Alex Merry wrote: On 27/12/13 11:04, Kevin Ottens wrote: On Monday 23 December 2013 14:51:19 Alex Merry wrote: Actually, I think we should just install application files directly in share/applications. (ie: change the

Re: Review Request 114478: Add kshareddatacache.h include

2013-12-27 Thread Alex Merry
--- This is an automatically generated e-mail. To reply, visit: https://git.reviewboard.kde.org/r/114478/#review46182 --- Ship it! As per discussion on kde-frameworks list. I'll

Re: Review Request 114478: Add kshareddatacache.h include

2013-12-27 Thread Commit Hook
--- This is an automatically generated e-mail. To reply, visit: https://git.reviewboard.kde.org/r/114478/#review46183 --- This review has been submitted with commit

Re: Review Request 114478: Add kshareddatacache.h include

2013-12-27 Thread Albert Astals Cid
--- This is an automatically generated e-mail. To reply, visit: https://git.reviewboard.kde.org/r/114478/ --- (Updated Dec. 27, 2013, 12:13 p.m.) Status -- This change has been

Review Request 114683: Clean up target_link_libraries for KF5Plasma

2013-12-27 Thread Alex Merry
--- This is an automatically generated e-mail. To reply, visit: https://git.reviewboard.kde.org/r/114683/ --- Review request for Build System, KDE Frameworks and Plasma. Repository:

Re: Using KCompressionDevice with QSaveFile

2013-12-27 Thread Dominik Haumann
On Tuesday 10 December 2013 23:31:33 David Faure wrote: On Tuesday 10 December 2013 21:48:09 Dominik Haumann wrote: It seems that the behavior for None is different. Interestingly, KSaveFile also behaved like this (KFilterDev did not change after all I guess). I'm surprised, because

Re: Review Request 114683: Clean up target_link_libraries for KF5Plasma

2013-12-27 Thread Aleix Pol Gonzalez
--- This is an automatically generated e-mail. To reply, visit: https://git.reviewboard.kde.org/r/114683/#review46193 --- Ship it! Ship It! - Aleix Pol Gonzalez On Dec. 27, 2013,

Re: Review Request 114683: Clean up target_link_libraries for KF5Plasma

2013-12-27 Thread Commit Hook
--- This is an automatically generated e-mail. To reply, visit: https://git.reviewboard.kde.org/r/114683/#review46194 --- This review has been submitted with commit

Forward includes

2013-12-27 Thread Aleix Pol
Hi, I've been going through the kde4support forward includes, since I wanted to start making the modules I decided we'd better make sure all of them are working properly. After some research, I found that I don't have these available, can somebody please tell me if I'm missing some dependency or

KCoreAddons forward headers

2013-12-27 Thread Aleix Pol
Hi, Here's a patch adding CamelCase headers to KCoreAddons (attached). If we agree that it's the proper way of doing it, I'll proceed to do it on the rest of modules. Cheers! Aleix diff --git a/CMakeLists.txt b/CMakeLists.txt index 389245c..ba6644a 100644 --- a/CMakeLists.txt +++

Re: Forward includes

2013-12-27 Thread Àlex Fiestas
On Friday 27 December 2013 19:00:14 Aleix Pol wrote: Hi, I've been going through the kde4support forward includes, since I wanted to start making the modules I decided we'd better make sure all of them are working properly. After some research, I found that I don't have these available, can