KF6 is a long time out so I'm abandoning this.

https://gitlab.kitware.com/cmake/cmake/issues/19615

Thanks!

D23262

Yep, KTextEditor is broken for me, too.

This seems to cause the build of sonnet to fail: https://invent.kde.org/snippets/394

LGTM

typo--

bump version again

Thanks!

Patch as is fine with me.
But no expert on both flags itself, no idea if there could be sideeffects of having those two now injected as ON to all projects. Possibly best to do global rebuilds of all projects on CI once this lands.

I'm happy with this, +2 from my side.

@kossebau @vkrause @apol something else to change?

This shall be compliant with the minimum ECM version set in the project.

Marco explained privately he doesn't want this so I'll move it to my codebase for now until the alternative better thing appears (maybe I'll try to make it appear soon).

In D22958#507299, @nicolasfella wrote:

Why do you restrict it to SVGs? Wouldn't you be able to use PNGs as well?

Why do you restrict it to SVGs? Wouldn't you be able to use PNGs as well?

The patch makes sense. +1

This is the culmination of a patch series that makes About dialogs in Kirigami work reasonably across platforms:

The code is mostly copy-and-pasted together from ECMAddAppIcon.cmake and ECMInstallIcons.cmake.

Ping

Ping

Seems people so far are fine with this, any comments were rather positive. No changes requested.

document behavior and when it was introduced, also enable autouic

Indeed, both good points.

+1, same for AUTOUIC probably?

Just a small principal comment with "supporting-old-versions" hat on.
Cannot give feedback on autorcc itself, never used, so no idea how useful/reliable it is.

+1 I've been using it in several project, it works good. I don't see how this could be misused.

+1 LGTM

• escape strings to be used as strings in C++ code
• use Qt5Core version to decide whether checking only for QtUiPlugin is enough

add more options: DEFAULT_HEADER_CASE, DEFAULT_HEADER_EXTENSION, DEFAULT_ICON_DIR

Accidentally uploaded the wrong diff.

• more code comments
• fix icon file existance check

• API DOX: explain arguments
• add argument to make it work for Okteta's extra code
• avoid rebuilding if generated file did not change
• place generated files in subdir for namespacing

Update CMake minimum required version to 3.1 (for target_sources).

I

I'm not sure if its beneficial to use cmake_parse_arguments here. It seems like it would make the macro harder to understand compared to the current change. I'm also not entirely sure how it would work? I haven't worked with cmake_parse_arguments before.

How about pass it as a TARGET argument using CMakeParseArguments.
You can see documentation about it here: https://cmake.org/cmake/help/latest/command/cmake_parse_arguments.html

LGTM

address cgiboudeaux's comments

In D22580#499015, @cgiboudeaux wrote:

KDE_INSTALL_USE_QT_SYS_PATHS has no effect on XDG_CONFIG_DIRS and XDG_DATA_DIRS or even the binary installation dir.

Running prefix.sh is useful even if KDE_INSTALL_USE_QT_SYS_PATHS is true.

Good idea

+1 Makes a lot of sense to me. In KDevelop we are already setting Debug by default.

Talked with Aleix, we're deciding to drop this for the moment

KDE_INSTALL_USE_QT_SYS_PATHS has no effect on XDG_CONFIG_DIRS and XDG_DATA_DIRS or even the binary installation dir.

Support older cmakes

Improve wording

In D22317#492656, @vatra wrote:

Just out of curiosity, why don't you use the cmake toolchain from Android NDK ? is located in ndk-bundle/build/cmake/android.toolchain.cmake

Just out of curiosity, why don't you use the cmake toolchain from Android NDK ? is located in ndk-bundle/build/cmake/android.toolchain.cmake

In D22156#488219, @cgiboudeaux wrote:

how many users are there?

how many users are there?

LGTM +1