<table><tr><td style="">daandemeyer created this revision.<br />daandemeyer added a reviewer: Kate.<br />daandemeyer added a project: Kate.<br />Herald added a subscriber: kwrite-devel.<br />daandemeyer 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/D22721">View Revision</a></tr></table><br /><div><strong>REVISION SUMMARY</strong><div><p>This is the first in a series of revisions modernizing the Kate CMake scripts. <br />
I'm submitting the changes to the addons directory first as they form a <br />
nicely separated list of changes.</p>

<p>Note that if <a href="https://phabricator.kde.org/D22698" class="remarkup-link" target="_blank" rel="noreferrer">https://phabricator.kde.org/D22698</a> and <br />
<a href="https://phabricator.kde.org/D22699" class="remarkup-link" target="_blank" rel="noreferrer">https://phabricator.kde.org/D22699</a> are merged, most if not all of the<br />
remaining source lists in the addons directory can be removed in favor of<br />
working with targets.</p>

<p>List of changes made:</p>

<ul class="remarkup-list">
<li class="remarkup-list-item">Standardize style on two spaces and no whitespace between commands and</li>
</ul>

<p>arguments.</p>

<ul class="remarkup-list">
<li class="remarkup-list-item">Remove directory commands in favor of target-based commands as</li>
</ul>

<p>recommended by modern CMake.</p>

<ul class="remarkup-list">
<li class="remarkup-list-item">Remove usage of <tt style="background: #ebebeb; font-size: 13px;">qt5_add_resources</tt> in favor of <tt style="background: #ebebeb; font-size: 13px;">CMAKE_AUTORCC</tt> and</li>
</ul>

<p>adding .qrc files directly to a target's sources.</p>

<ul class="remarkup-list">
<li class="remarkup-list-item">Remove usage of source lists (where possible) in favor of CMake 3.1's</li>
</ul>

<p><tt style="background: #ebebeb; font-size: 13px;">target_sources</tt> command.</p>

<ul class="remarkup-list">
<li class="remarkup-list-item">Remove the CMake binary directory as an include directory from most</li>
</ul>

<p>addons.</p>

<ul class="remarkup-list">
<li class="remarkup-list-item">Remove unnecessary comments, CMake project calls, config files, etc</li>
</ul>

<p>from addon CMake scripts.</p>

<ul class="remarkup-list">
<li class="remarkup-list-item">Remove <tt style="background: #ebebeb; font-size: 13px;">HAVE_CTERMID</tt> from config.h and move the check and addition</li>
</ul>

<p>to compile definitions to the project addon CMake script as its only<br />
used in the project plugin sources.</p></div></div><br /><div><strong>TEST PLAN</strong><div><p>Apply changes and verify Kate still builds and works exactly as before.</p></div></div><br /><div><strong>REPOSITORY</strong><div><div>R40 Kate</div></div></div><br /><div><strong>REVISION DETAIL</strong><div><a href="https://phabricator.kde.org/D22721">https://phabricator.kde.org/D22721</a></div></div><br /><div><strong>AFFECTED FILES</strong><div><div>CMakeLists.txt<br />
addons/CMakeLists.txt<br />
addons/backtracebrowser/CMakeLists.txt<br />
addons/backtracebrowser/autotests/CMakeLists.txt<br />
addons/close-except-like/CMakeLists.txt<br />
addons/close-except-like/close_except_plugin.cpp<br />
addons/close-except-like/config.h.in<br />
addons/filebrowser/CMakeLists.txt<br />
addons/filetree/CMakeLists.txt<br />
addons/filetree/autotests/CMakeLists.txt<br />
addons/gdbplugin/CMakeLists.txt<br />
addons/kate-ctags/CMakeLists.txt<br />
addons/katebuild-plugin/CMakeLists.txt<br />
addons/katesql/CMakeLists.txt<br />
addons/konsole/CMakeLists.txt<br />
addons/lspclient/CMakeLists.txt<br />
addons/lspclient/tests/CMakeLists.txt<br />
addons/lumen/CMakeLists.txt<br />
addons/openheader/CMakeLists.txt<br />
addons/preview/CMakeLists.txt<br />
addons/project/CMakeLists.txt<br />
addons/project/autotests/CMakeLists.txt<br />
addons/project/kateprojectplugin.cpp<br />
addons/replicode/CMakeLists.txt<br />
addons/rustcompletion/CMakeLists.txt<br />
addons/search/CMakeLists.txt<br />
addons/sessionapplet/CMakeLists.txt<br />
addons/sessionapplet/engine/CMakeLists.txt<br />
addons/snippets/CMakeLists.txt<br />
addons/symbolviewer/CMakeLists.txt<br />
addons/tabswitcher/CMakeLists.txt<br />
addons/tabswitcher/autotests/CMakeLists.txt<br />
addons/tabswitcher/tests/CMakeLists.txt<br />
addons/textfilter/CMakeLists.txt<br />
addons/xmlcheck/CMakeLists.txt<br />
addons/xmltools/CMakeLists.txt<br />
config.h.cmake</div></div></div><br /><div><strong>To: </strong>daandemeyer, Kate<br /><strong>Cc: </strong>kwrite-devel, domson, michaelh, ngraham, demsking, cullmann, sars, dhaumann<br /></div>