Directory organization for frameworks with a namespace

[David Faure]

On Wednesday 07 November 2012 17:41:15 Kevin Ottens wrote:
> On Wednesday 07 November 2012 16:50:39 David Faure wrote:
> > On Monday 29 October 2012 07:40:29 Kevin Ottens wrote:
> > > Generated forward headers definitely have my preference as they indeed
> > > separate the include organization from the "implementation detail" of
> > > how
> > > the sources are organized.
> > 
> > In fact we don't even *have* to generate them. Unlike the CamelCase Qt
> > headers, I'm thinking of lowercase forwarding headers here, not installed,
> > so no need for them to be ever changed-before-install.
> 
> Right, they just need to be generated before build.

Done now, with a few lines of CMake code.

> > This makes it rather simple:
> > 
> > src/include/kio/job.h: #include "../../../core/job.h"
> > 
> > Well I can ship a script to create them, but still commit them (not have
> > cmake run the script). Better for dependency handling, and simpler
> > overall.
> 
> Also more error prone, it'd mean people would have to manually run a script
> when they add a public header. I think it's a case where automation would be
> better to be honest.

OK, you convinced me.

> > > > Any other ideas?
> > > 
> > > The only alternative I see would be:
> > [...]
> > Sorry for being so pragmatic, but I think we have enough work ahead of us
> > without creating even more work for ourselves.
> 
> Well, you asked for other ideas, that's what I provided. I just meant it was
> the last option not yet on the table to be evaluated. I don't think it's
> practical either.

LOL, I'll make sure to specify "valid" before "ideas" next time :-)

-- 
David Faure, faure at kde.org, http://www.davidfaure.fr
Working on KDE, in particular KDE Frameworks 5
-------------- next part --------------
A non-text attachment was scrubbed...
Name: signature.asc
Type: application/pgp-signature
Size: 190 bytes
Desc: This is a digitally signed message part.
URL: <http://mail.kde.org/pipermail/kde-frameworks-devel/attachments/20121121/b0a1c597/attachment.sig>