[education/minuet] /: Use the same condition for finding KF6IconThemes when linking

Heiko Becker <[email protected]>
Newsgroups gmane.comp.kde.cvs
Message-ID <[email protected]>
Git commit 0876d7f8ec7118cf092a51be433a15d03da0f823 by Heiko Becker.
Committed on 23/07/2026 at 07:42.
Pushed by heikobecker into branch 'master'.

Use the same condition for finding KF6IconThemes when linking

I unfortunately broke this in 9aef20d, being confused about Apple's OSes
and seeing the if(IOS) branch. But it actually turns out that
src/app/CMakeLists.txt does

    if(APPLE AND NOT (CMAKE_SYSTEM_NAME STREQUAL "iOS"))
        target_link_libraries(minuet PRIVATE KF6::IconThemes)
        ...

and src/app/main.cpp:

    #if defined(Q_OS_MACOS)
    #include <KIconTheme>
    #endif

So match this usage in the find_package call.

M  +4    -3    CMakeLists.txt

https://invent.kde.org/education/minuet/-/commit/0876d7f8ec7118cf092a51be433a15d03da0f823

diff --git a/CMakeLists.txt b/CMakeLists.txt
index 16af554..69abb44 100644
--- a/CMakeLists.txt
+++ b/CMakeLists.txt
@@ -189,14 +189,15 @@ if(NOT MINUET_MOBILE_PLATFORM)
         KF6 ${KF_MIN_VERSION} REQUIRED COMPONENTS Crash DocTools
                                                   QQC2DesktopStyle
     )
+    if(APPLE)
+        find_package(KF6 ${KF_MIN_VERSION} REQUIRED COMPONENTS IconThemes)
+    endif()
     ecm_find_qmlmodule(org.kde.desktop REQUIRED)
 else()
     find_package(KF6 ${KF_MIN_VERSION} REQUIRED COMPONENTS ColorScheme)
-    if(IOS)
-        find_package(KF6 ${KF_MIN_VERSION} REQUIRED COMPONENTS IconThemes)
-    endif()
 endif()
 
+
 include_directories(${minuet_SOURCE_DIR}/src/ ${minuet_BINARY_DIR}/src/)
 
 add_subdirectory(src)
lmpx.com only provides a reader for public news (NNTP) servers. It is not affiliated with the servers or forums shown here and is not responsible for the content of articles, which is written by their respective authors.