<table><tr><td style="">joselema created this revision.<br />joselema added reviewers: KDE PIM, KDE PIM: KMail.<br />joselema added projects: KDE PIM: KMail, KDE PIM.<br />joselema requested review of this revision.
</td><a style="text-decoration: none; padding: 4px 8px; margin: 0 8px 8px; float: right; color: #464C5C; font-weight: bold; border-radius: 3px; background-color: #F7F7F9; background-image: linear-gradient(to bottom,#fff,#f1f0f1); display: inline-block; border: 1px solid rgba(71,87,120,.2);" href="https://phabricator.kde.org/D11508">View Revision</a></tr></table><br /><div><strong>REVISION SUMMARY</strong><div><p>Fix messagelist core includes.</p>

<p>There is a test in Kubuntu's packaging which builds the installed library headers alone, detecting problems such as having other headers #include'd which can't be found.</p>

<p>Right now, without this patch, the compilation of 'widgetbase.h' would fail because it won't be able to find '#include <core/enums.h>', so I replaced this with '#include <messagelist/enums.h>' and I also applied the same change to other headers which otherwise wouldn't be found.</p>

<p>It would be nice to get this problem sorted out in KDE's code, so this way we won't have to patch it in the packaging.</p></div></div><br /><div><strong>TEST PLAN</strong><div><p>I have built the Kubuntu's messagelist package (17.12.3) with the patch and now the headers test pass. Also I made a complete rebuild of KDE Applications 17.12.3 from the packaging against this patched messagelib in order to make sure it doesn't break the compilation of other PIM components; everything seems to build fine.</p></div></div><br /><div><strong>REPOSITORY</strong><div><div>R94 PIM: Message Library</div></div></div><br /><div><strong>REVISION DETAIL</strong><div><a href="https://phabricator.kde.org/D11508">https://phabricator.kde.org/D11508</a></div></div><br /><div><strong>AFFECTED FILES</strong><div><div>messagelist/src/CMakeLists.txt<br />
messagelist/src/core/aggregation.h<br />
messagelist/src/core/sortorder.h<br />
messagelist/src/core/widgetbase.h</div></div></div><br /><div><strong>To: </strong>joselema, KDE PIM, KDE PIM: KMail<br /><strong>Cc: </strong>joselema, rikmills, dvasin, hrouis, ach, winterz, vkrause, mlaurent, knauss, dvratil<br /></div>