On Monday 30 December 2013 14:59:51 Aleix Pol wrote:
> It only has the lowercase headers, uppercase headers are installed from
> another command:
> install(DIRECTORY ${CMAKE_CURRENT_BINARY_DIR}/KCoreAddons DESTINATION
> ${INCLUDE_INSTALL_DIR} COMPONENT Devel )
Ah, right.
Sounds good to me as it
On Mon, Dec 30, 2013 at 11:42 AM, David Faure wrote:
> On Monday 30 December 2013 01:26:52 Aleix Pol wrote:
> > On Sun, Dec 29, 2013 at 8:10 PM, David Faure wrote:
> > > On Sunday 29 December 2013 16:18:23 Christoph Cullmann wrote:
> > > > > On Friday 27 December 2013 19:36:57 Aleix Pol wrote:
>
On Monday 30 December 2013 01:26:52 Aleix Pol wrote:
> On Sun, Dec 29, 2013 at 8:10 PM, David Faure wrote:
> > On Sunday 29 December 2013 16:18:23 Christoph Cullmann wrote:
> > > > On Friday 27 December 2013 19:36:57 Aleix Pol wrote:
> > > > > Hi,
> > > > > Here's a patch adding CamelCase headers
On 30/12/13 00:26, Aleix Pol wrote:
> No, it creates a variable called KCoreAddons_HEADERS which is passed to
> the install() command. ECM_GENERATE_HEADERS doesn't install anything, we
> decided to do it like that to minimize the magic factor.
>
> Please take a look at the function documentation,
On Sun, Dec 29, 2013 at 8:10 PM, David Faure wrote:
> On Sunday 29 December 2013 16:18:23 Christoph Cullmann wrote:
> > > On Friday 27 December 2013 19:36:57 Aleix Pol wrote:
> > > > Hi,
> > > > Here's a patch adding CamelCase headers to KCoreAddons (attached).
> > > >
> > > > If we agree that it
On Sunday 29 December 2013 16:18:23 Christoph Cullmann wrote:
> > On Friday 27 December 2013 19:36:57 Aleix Pol wrote:
> > > 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
> >
> On Friday 27 December 2013 19:36:57 Aleix Pol wrote:
> > 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.
>
> Looks good to me. I like the fact that we can't for
On Friday 27 December 2013 19:36:57 Aleix Pol wrote:
> 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.
Looks good to me. I like the fact that we can't forget to add forwa
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
+++ b/CMakeLists.txt
