<div dir="ltr"><div><span style="font-size:12.8px">Hi Scarlet,</span></div><span style="font-size:12.8px"><div><span style="font-size:12.8px"><br></span></div>-#include <Akonadi/Contact/</span><span style="font-size:12.8px">ContactSearchJob></span><br style="font-size:12.8px"><span style="font-size:12.8px">+#include <KF5/akonadi/contact/</span><span style="font-size:12.8px">contactsearchjob.h></span><br><div><span style="font-size:12.8px"><br></span></div><div><span style="font-size:12.8px">This patch sound like a wrong way to handle frameworks header. After all i never use </span><span style="font-size:12.8px">#include <KF5/...> syntax, and there is no reason to do it.</span></div><div><span style="font-size:12.8px"><br></span></div><div><span style="font-size:12.8px">Using the .h file instead the camelcase version is not a problem, but both cases must work well in client application.</span></div><div><span style="font-size:12.8px"><br></span></div><div><span style="font-size:12.8px">Gilles Caulier</span></div></div><div class="gmail_extra"><br><div class="gmail_quote">2016-04-07 3:01 GMT+02:00 Scarlett Clark <span dir="ltr"><<a href="mailto:scarlett.gately.clark@gmail.com" target="_blank">scarlett.gately.clark@gmail.com</a>></span>:<br><blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex"><div dir="ltr"><div><div><div><div><div>It has to do with /path/to/include/KF5/path/to/camelcase , the key is actually the KF5 folder. <br>If I patch  include/KF5/Path/To/Camelcase file it works, however the camel case file does not find its own header. <br><pre>/srv/jenkins/install/ubuntu/x86_64/g++/kf5-qt5/kde/kdepimlibs/inst/usr/include/KF5/Akonadi/Contact/ContactSearchJob:1:46: fatal error: akonadi/contact/contactsearchjob.h: No such file or directory
<span><b>22:47:37</b> </span>compilation terminated.</pre>I can only assume the KF5 folder is involved again. So I tried to add KF5 to the ecm_generate_headers(PREFIX) but that has the effect of lower case kf5 which also does not work, obviously.<br><br></div><div>So I turned back to Digikam and patched:<br><br>-#include <Akonadi/Contact/ContactSearchJob><br>+#include <KF5/akonadi/contact/contactsearchjob.h><br><br></div><div>And finally we have a successful build.<br><br><a href="https://build.kde.org/job/digikam%20master%20kf5-qt5/PLATFORM=Linux,compiler=gcc/68/console" target="_blank">https://build.kde.org/job/digikam%20master%20kf5-qt5/PLATFORM=Linux,compiler=gcc/68/console</a><br></div><div><br></div>I do notice KDE_INSTALL_INCLUDEDIR_KF5 in kdepimlibs is likely what puts in this KF5 folder?<br><br></div>I do not know who is responsible here, but someone needs to fix something :) I have exhausted my current C++ skills to get this far lol.<br><br></div>I have added digikam to the conversation.<br></div>Cheers,<br></div>Scarlett<br><div><div><div><br></div></div></div></div><div class="gmail_extra"><br><div class="gmail_quote">On Thu, Mar 31, 2016 at 11:48 PM, Ben Cooksley <span dir="ltr"><<a href="mailto:bcooksley@kde.org" target="_blank">bcooksley@kde.org</a>></span> wrote:<br><blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex"><div><div>On Thu, Mar 31, 2016 at 11:44 PM, Christophe Giboudeaux<br>
<<a href="mailto:cgiboudeaux@gmx.com" target="_blank">cgiboudeaux@gmx.com</a>> wrote:<span class=""><br>
> On mercredi 30 mars 2016 20:19:29 CEST Ben Cooksley wrote:<br>
>> On Tue, Mar 29, 2016 at 10:46 PM, Sandro Knauß <<a href="mailto:sknauss@kde.org" target="_blank">sknauss@kde.org</a>> wrote:<br>
>><br></span><span class="">
>> Those are the header generation and installation routines - which do<br>
>> indeed look.<br>
>> What is missing in this case is the include path specification for the<br>
>> KF5::AkonadiContact target which CMake provides.<br>
>><br>
><br></span>
> The exported target is correct.<br>
> From KF5AkonadiContactTargets.cmake :<br>
><br>
> set_target_properties(KF5::AkonadiContact PROPERTIES<br>
> INTERFACE_INCLUDE_DIRECTORIES<br>
>  "${_IMPORT_PREFIX}/include/KF5/Akonadi/Contact;<br>
>   ${_IMPORT_PREFIX}/include/KF5;<br>
>   ${_IMPORT_PREFIX}/include/KF5/akonadi/contact;<br>
>   ${_IMPORT_PREFIX}/include/KF5"<br>
>   INTERFACE_LINK_LIBRARIES "KF5::AkonadiCore;KF5::Contacts;Qt5::Widgets"<br>
> )<br>
<br>
</div></div>Hmm, indeed that does look correct.<br>
I wonder why Digikam would fail to compile then:<br>
<br>
19:24:09 /home/jenkins/sources/digikam/kf5-qt5/utilities/kdesupport/akonadi/akonadiiface.cpp:47:44:<br>
fatal error: Akonadi/Contact/ContactSearchJob: No such file or<br>
directory<br>
<br>
Any ideas? (I assume Digikam is trying using to use AkonadiContact correctly)<br>
<br>
Cheers,<br>
Ben<br>
<br>
><br>
> _______________________________________________<br>
> Kde-buildsystem mailing list<br>
> <a href="mailto:Kde-buildsystem@kde.org" target="_blank">Kde-buildsystem@kde.org</a><br>
> <a href="https://mail.kde.org/mailman/listinfo/kde-buildsystem" rel="noreferrer" target="_blank">https://mail.kde.org/mailman/listinfo/kde-buildsystem</a><span class=""><br>
<div><div>><br>
_______________________________________________<br>
KDE PIM mailing list <a href="mailto:kde-pim@kde.org" target="_blank">kde-pim@kde.org</a><br>
<a href="https://mail.kde.org/mailman/listinfo/kde-pim" rel="noreferrer" target="_blank">https://mail.kde.org/mailman/listinfo/kde-pim</a><br>
KDE PIM home page at <a href="http://pim.kde.org/" rel="noreferrer" target="_blank">http://pim.kde.org/</a></div></div></span></blockquote></div><br></div>
<br>_______________________________________________<br>
Digikam-devel mailing list<br>
<a href="mailto:Digikam-devel@kde.org">Digikam-devel@kde.org</a><br>
<a href="https://mail.kde.org/mailman/listinfo/digikam-devel" rel="noreferrer" target="_blank">https://mail.kde.org/mailman/listinfo/digikam-devel</a><br>
<br></blockquote></div><br></div>