<div dir="ltr"><div>Also as a follow-up question, I moved the two defines from kate's config.h into the CMake scripts using `target_compile_definitions` and the only thing left in it is the version. Does that mean the config.h file can be removed or is there code that depends on the version defined in the config.h file?</div><div><br></div><div>Regards,</div><div><br></div><div>Daan<br></div><br></div><br><div class="gmail_quote"><div dir="ltr" class="gmail_attr">On Tue, 23 Jul 2019 at 23:55, Daan De Meyer <<a href="mailto:daan.j.demeyer@gmail.com">daan.j.demeyer@gmail.com</a>> wrote:<br></div><blockquote class="gmail_quote" style="margin:0px 0px 0px 0.8ex;border-left:1px solid rgb(204,204,204);padding-left:1ex"><div dir="ltr"><div>I've almost finished a pretty big cleanup of all the CMake files including addons. I standardized everything on 2 spaces. <br></div><div><br></div><div>Should I just make a single revision on Phabricator or try to split it up into multiple revisions?</div><div><br></div><div>I also upped the minimum CMake version to 3.1 (from 3.0) to have access to `target_sources` (<a href="https://cmake.org/cmake/help/latest/command/target_sources.html" target="_blank">https://cmake.org/cmake/help/latest/command/target_sources.html</a>).  Is that a problem?</div><div><br></div><div>Regards,</div><div><br></div><div>Daan<br></div></div><br><div class="gmail_quote"><div dir="ltr" class="gmail_attr">On Tue, 23 Jul 2019 at 22:08, Christoph Cullmann <<a href="mailto:christoph@cullmann.io" target="_blank">christoph@cullmann.io</a>> wrote:<br></div><blockquote class="gmail_quote" style="margin:0px 0px 0px 0.8ex;border-left:1px solid rgb(204,204,204);padding-left:1ex">On 2019-07-23 18:04, Daan De Meyer wrote:<br>
> Hi,<br>
> <br>
> I'm cleaning up the CMake scripts a little and was wondering what<br>
> style to use. Generally, CMake projects (outside of KDE) tend to use 2<br>
> space indentation but I'm seeing both 2 and 4 spaces indentation in<br>
> the CMake files (and see the .kateconfig specifying 4 spaces). Is<br>
> there a guideline I should follow here?<br>
> <br>
> Regards,<br>
> <br>
> Daan<br>
Hi,<br>
<br>
Think we are flexible, would keep 2 spaces, if<br>
our files are that already.<br>
<br>
Greetings<br>
Christoph<br>
<br>
-- <br>
Ignorance is bliss...<br>
<a href="https://cullmann.io" rel="noreferrer" target="_blank">https://cullmann.io</a> | <a href="https://kate-editor.org" rel="noreferrer" target="_blank">https://kate-editor.org</a><br>
</blockquote></div>
</blockquote></div>