<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">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">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>