<div dir="ltr"><div class="gmail_extra"><div class="gmail_quote">On Sun, Dec 29, 2013 at 8:10 PM, 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 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 it<br>
> > > on<br>
> > > the rest of modules.<br>
> ><br>
> > Looks good to me. I like the fact that we can't forget to add forwarding<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 of<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 still<br>
> to install the normal headers manually.<br>
<br>
</div>Well, Aleix's patch removes the installation of the lowercase headers, doesn't<br>
it?<br>
<div class="im"><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 5NES Power Glove<br>
<br>
</div><div class=""><div class="h5">_______________________________________________<br>
Kde-frameworks-devel mailing list<br>
<a href="mailto:Kde-frameworks-devel@kde.org">Kde-frameworks-devel@kde.org</a><br>
<a href="https://mail.kde.org/mailman/listinfo/kde-frameworks-devel" target="_blank">https://mail.kde.org/mailman/listinfo/kde-frameworks-devel</a><br>
</div></div></blockquote></div><br></div><div class="gmail_extra">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. </div>

<div class="gmail_extra"><br></div><div class="gmail_extra">Please take a look at the function documentation, in case it's not clear enough (it's at the top of the ECMGenerateHeaders.cmake file).</div><div class="gmail_extra">

<br></div><div class="gmail_extra">Aleix</div></div>