<div dir="ltr">On Fri, Nov 15, 2013 at 5:24 PM, Aurélien Gâteau <span dir="ltr"><<a href="mailto:agateau@kde.org" target="_blank">agateau@kde.org</a>></span> wrote:<br><div class="gmail_extra"><div class="gmail_quote">
<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 Thu, 14 Nov 2013 17:55:42 +0100, Kevin Ottens 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">
On Thursday 14 November 2013 18:40:04 Aurélien Gâteau 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">
Looks like we have an agreement then. Let me recap:<br>
<br>
# Installation dir<br>
<br>
All header files are installed in $PREFIX/include/KF5/$<u></u>Framework.<br>
This includes 'k' prefixed headers like kfoo and KFoo,<br>
</blockquote>
<br>
I assume you meant kfoo.h here.<br>
<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">
non-prefixed headers such as bar.h and Bar, as well as special headers<br>
such as ${framework}_export.h and ${framework}_version.h.<br>
<br>
# Include path<br>
<br>
For 'k' prefixed headers, include path is<br>
$PREFIX/include/KF5/$<u></u>Framework.<br>
<br>
For non-prefixed headers, include path is $PREFIX/include/KF5.<br>
<br>
Is this correct?<br>
</blockquote>
<br>
Looks correct to me.<br>
</blockquote>
<br></div>
David pointed out on IRC this scheme would need a significant change for<br>
frameworks like KIO or Solid: includes to <kio/foo.h> would need to be<br>
changed to <KIO/foo.h>, same with <solid/*> includes. Are you fine with<br>
this?<div class=""><div class="h5"><br>
<br>
Aurélien<br>
<br>
______________________________<u></u>_________________<br>
Kde-frameworks-devel mailing list<br>
<a href="mailto:Kde-frameworks-devel@kde.org" target="_blank">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/<u></u>listinfo/kde-frameworks-devel</a><br>
</div></div></blockquote></div><br></div><div class="gmail_extra">I think that we're making this too complex. I'd say that it should always be prefixed, then the module can decide to export the module directory as well.</div>
<div class="gmail_extra">Not mixing headers from different frameworks in a same directory sounds like a good feature as well.</div><div class="gmail_extra"><br></div><div class="gmail_extra">Also I'd say that clearly there's a differentiation between camelcase and lowercase. IMHO:</div>
<div class="gmail_extra">good:</div><div class="gmail_extra">#include <kio/job.h></div><div class="gmail_extra">#include <Solid/Device><br></div><div class="gmail_extra"><br></div><div class="gmail_extra">weird:</div>
<div class="gmail_extra">#include <Solid/device.h></div><div class="gmail_extra"><br></div><div class="gmail_extra">Aleix</div><div class="gmail_extra"><br></div><div class="gmail_extra">PS: I would stop adding even more things to kde4support if it's not extremely needed, it's crowded already.</div>
<div class="gmail_extra"><br></div></div>