<table><tr><td style="">dcaliste created this revision.<br />dcaliste added reviewers: leinir, Calligra: 3.0.<br />dcaliste added a project: Calligra: 3.0.<br />Herald added a subscriber: Calligra-Devel-list.<br />dcaliste 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/D15236">View Revision</a></tr></table><br /><div><strong>REVISION SUMMARY</strong><div><p>Add missing dependencies in target_link_libraries() cmake macro here and there. Added dependencies comes from #include<> directives for specific headers, like for instance having #include<kmessagebox.h> but not specifying KF5::WidgetsAddons.</p>

<p>I'm still a bit puzzled know how it can compile without these modifications since required '-I path' compiler options are then missing… Maybe because I'm using non standard pathes for compilation.</p>

<p>Additionally, I've moved the add_includedirs() macro in sheets/CMakeLists.txt inside the ifdef PART_SHEETS, otherwise the eigen3 include directory variable may not be set, when configuring without part_sheets and cmake fails in that case.</p></div></div><br /><div><strong>REPOSITORY</strong><div><div>R8 Calligra</div></div></div><br /><div><strong>REVISION DETAIL</strong><div><a href="https://phabricator.kde.org/D15236">https://phabricator.kde.org/D15236</a></div></div><br /><div><strong>AFFECTED FILES</strong><div><div>libs/flake/CMakeLists.txt<br />
libs/main/CMakeLists.txt<br />
libs/odf/CMakeLists.txt<br />
libs/store/CMakeLists.txt<br />
libs/widgets/CMakeLists.txt<br />
plugins/textshape/CMakeLists.txt<br />
sheets/CMakeLists.txt<br />
sheets/shape/CMakeLists.txt<br />
stage/part/CMakeLists.txt<br />
words/part/CMakeLists.txt</div></div></div><br /><div><strong>To: </strong>dcaliste, leinir, Calligra: 3.0<br /><strong>Cc: </strong>Calligra-Devel-list, cochise, vandenoever<br /></div>