<div dir="ltr"><div class="gmail_extra"><div class="gmail_quote">On Mon, Dec 30, 2013 at 11:42 AM, David Faure <span dir="ltr"><<a href="mailto:faure@kde.org" target="_blank">faure@kde.org</a>></span> wrote:<br><blockquote class="gmail_quote" style="margin:0px 0px 0px 0.8ex;border-left-width:1px;border-left-color:rgb(204,204,204);border-left-style:solid;padding-left:1ex">

<div class="im">On Monday 30 December 2013 01:26:52 Aleix Pol wrote:<br>
> On Sun, Dec 29, 2013 at 8:10 PM, David Faure <<a href="mailto:faure@kde.org">faure@kde.org</a>> wrote:<br>
> > On Sunday 29 December 2013 16:18:23 Christoph Cullmann wrote:<br>
> > > > On Friday 27 December 2013 19:36:57 Aleix Pol wrote:<br>
> > > > > Hi,<br>
> > > > > Here's a patch adding CamelCase headers to KCoreAddons (attached).<br>
> > > > ><br>
> > > > > If we agree that it's the proper way of doing it, I'll proceed to do<br>
> ><br>
> > it<br>
> ><br>
> > > > > on<br>
> > > > > the rest of modules.<br>
> > > ><br>
> > > > Looks good to me. I like the fact that we can't forget to add<br>
> ><br>
> > forwarding<br>
> ><br>
> > > > headers like before - if we do, the lowercase header isn't installed<br>
> > > > either, so we'll notice faster :)<br>
> > > > I mean, a single operation is needed to install a new header, instead<br>
> ><br>
> > of<br>
> ><br>
> > > > two, one of which I always forgot.<br>
> > ><br>
> > > Hmm, does it really install the normal headers, too?<br>
> > ><br>
> > > I use it now in kate.git/ktexteditor/include/CMakelists.txt and need<br>
> ><br>
> > still<br>
> ><br>
> > > to install the normal headers manually.<br>
> ><br>
> > Well, Aleix's patch removes the installation of the lowercase headers,<br>
> > doesn't<br>
> > it?<br>
><br>
</div><div class="im">> No, it creates a variable called KCoreAddons_HEADERS which is passed to the<br>
> install() command. ECM_GENERATE_HEADERS doesn't install anything, we<br>
> decided to do it like that to minimize the magic factor.<br>
<br>
</div>Haha, by remove I meant "removes the explicit code that installed the<br>
lowercase headers, because it is now done semi-automatically".<br>
<br>
So if KCoreAddons_HEADERS contains both the lowercase and camelcase headers,<br>
then we fully agree :)<br>
<div class=""><div class="h5"><br>
--<br>
David Faure, <a href="mailto:faure@kde.org">faure@kde.org</a>, <a href="http://www.davidfaure.fr" target="_blank">http://www.davidfaure.fr</a><br>
Working on KDE, in particular KDE Frameworks 5<br>
<br>
</div></div></blockquote></div><br></div><div class="gmail_extra">It only has the lowercase headers, uppercase headers are installed from another command:</div><div class="gmail_extra"><div class="gmail_extra">install(DIRECTORY ${CMAKE_CURRENT_BINARY_DIR}/KCoreAddons DESTINATION  ${INCLUDE_INSTALL_DIR} COMPONENT Devel )</div>

<div><br></div><div>Aleix</div></div></div>