<div dir="ltr"><div>The KATE_VERSION variable was used in some source files but by passing the version as a command line define via CMake allowed me to remove the config.h file completely.</div><div><br></div><div>One more question: I have a few in progress patches for extra-cmake-modules and ki18n on Phabricator that will allow me to streamline kate's CMake configuration even more. Can these be used immediately after they are merged or does each KDE application only use features from the latest stable release? I'm asking this because I was surprised Kate built without problems from source even though I only have the latest stable release of all other KDE libraries installed on my system (instead of the git versions).</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 Wed, 24 Jul 2019 at 08:28, 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-24 00:10, Daan De Meyer wrote:<br>
> Also as a follow-up question, I moved the two defines from kate's<br>
> config.h into the CMake scripts using `target_compile_definitions` and<br>
> the only thing left in it is the version. Does that mean the config.h<br>
> file can be removed or is there code that depends on the version<br>
> defined in the config.h file?<br>
<br>
Hi,<br>
don't think 3.1 should be a problem. config.h is Not required elsewhere <br>
I assume.<br>
<br>
Greetings<br>
Christoph<br>
<br>
> <br>
> Regards,<br>
> <br>
> Daan<br>
> <br>
> On Tue, 23 Jul 2019 at 23:55, Daan De Meyer <<a href="mailto:daan.j.demeyer@gmail.com" target="_blank">daan.j.demeyer@gmail.com</a>><br>
> wrote:<br>
> <br>
>> I've almost finished a pretty big cleanup of all the CMake files<br>
>> including addons. I standardized everything on 2 spaces.<br>
>> <br>
>> Should I just make a single revision on Phabricator or try to split<br>
>> it up into multiple revisions?<br>
>> <br>
>> I also upped the minimum CMake version to 3.1 (from 3.0) to have<br>
>> access to `target_sources`<br>
>> (<a href="https://cmake.org/cmake/help/latest/command/target_sources.html" rel="noreferrer" target="_blank">https://cmake.org/cmake/help/latest/command/target_sources.html</a>).<br>
>> Is that a problem?<br>
>> <br>
>> Regards,<br>
>> <br>
>> Daan<br>
>> <br>
>> On Tue, 23 Jul 2019 at 22:08, Christoph Cullmann<br>
>> <<a href="mailto:christoph@cullmann.io" target="_blank">christoph@cullmann.io</a>> wrote:<br>
>> <br>
>>> 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<br>
>>> what<br>
>>>> style to use. Generally, CMake projects (outside of KDE) tend to<br>
>>> use 2<br>
>>>> space indentation but I'm seeing both 2 and 4 spaces indentation<br>
>>> in<br>
>>>> the CMake files (and see the .kateconfig specifying 4 spaces).<br>
>>> 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>
<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>