KCoreAddons forward headers

[Aleix Pol]

On Sun, Dec 29, 2013 at 8:10 PM, David Faure <faure at kde.org> 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'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
> forwarding
> > > headers like before - if we do, the lowercase header isn't installed
> > > either, so we'll notice faster :)
> > > I mean, a single operation is needed to install a new header, instead
> of
> > > two, one of which I always forgot.
> >
> > Hmm, does it really install the normal headers, too?
> >
> > I use it now in kate.git/ktexteditor/include/CMakelists.txt and need
> still
> > to install the normal headers manually.
>
> Well, Aleix's patch removes the installation of the lowercase headers,
> doesn't
> it?
>
> --
> David Faure, faure at kde.org, http://www.davidfaure.fr
> Working on KDE, in particular KDE Frameworks 5NES Power Glove
>
> _______________________________________________
> Kde-frameworks-devel mailing list
> Kde-frameworks-devel at kde.org
> https://mail.kde.org/mailman/listinfo/kde-frameworks-devel
>

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, in case it's not clear
enough (it's at the top of the ECMGenerateHeaders.cmake file).

Aleix
-------------- next part --------------
An HTML attachment was scrubbed...
URL: <http://mail.kde.org/pipermail/kde-frameworks-devel/attachments/20131230/bdbd6741/attachment.html>