Just some small tips.

Thanks @davidedmundson and @ognarb, I was able to replicate the issue, I'll think a bit more in my approach

@cblack Yeah, I was uploading it :)
Output:

Update style

Merged.

@apol done.

In D29202#657702, @patrickelectric wrote:

In D29202#657700, @apol wrote:

Then remove it from CMakeLists.txt as well?

Can you point what exactly should be removed ?

In D29202#657700, @apol wrote:

Then remove it from CMakeLists.txt as well?

It fails to build when QtConcurrent can't be found.

Moved to https://invent.kde.org/kde/minuet/-/merge_requests/7

Moved to https://invent.kde.org/kde/minuet/-/merge_requests/7

Merged.

Merged.

@apol @pino This fix the problem, RUNTIME DESTINATION is already specified by KDE_INSTALL_TARGETS_DEFAULT_ARGS, that's why I was getting the error message.

Hm, Ok, I'll debug it later so.

@pino if you can, please the history if of this patch to see my first and working version.
https://phabricator.kde.org/D29077?id=80827

@pino It was a simple if(apple) message(potato) endif() to test if apple was true and the correct variable was by kdeinstalldir, with everything stashed and only this patch applied:

CMake Error at src/app/CMakeLists.txt:84 (install):
  install TARGETS given no BUNDLE DESTINATION for MACOSX_BUNDLE executable
  target "minuet".

@apol doing so results in

CMake Error at src/app/CMakeLists.txt:87 (install):
  install TARGETS given no BUNDLE DESTINATION for MACOSX_BUNDLE executable
  target "minuet".

@apol is that what you mean ?

@dakon It's @araujoluis first MR and we are still here in phabricator, so I don't see why not merge if we are not in gitlab yet.

Hi @jjazeix this did fix the android build, but now it's failing with libQtZeroConf, and after your fix with:

qrc:/FlightControls.qml:29 module "org.kde.kirogi.video" is not installed