From c1749a46886e5759c87ef9f9cd576e2ef16e1e79 Mon Sep 17 00:00:00 2001
From: Vasilito <adminpupkin@gmail.com>
Date: Fri, 1 May 2026 03:00:49 +0100
Subject: [PATCH] =?UTF-8?q?fix:=20KWin=20recipe=20=E2=80=94=20remove=20if?=
 =?UTF-8?q?=20block,=20cmake=20fails=20fast?=
MIME-Version: 1.0
Content-Type: text/plain; charset=UTF-8
Content-Transfer-Encoding: 8bit

Removed "if cmake ...; then ... fi" wrapper that allowed silent
cmake configure failure. cmake/build/install now run directly —
any failure propagates to recipe failure. No silent fallback.
---
 local/recipes/kde/kwin/recipe.toml | 9 ++++-----
 1 file changed, 4 insertions(+), 5 deletions(-)

diff --git a/local/recipes/kde/kwin/recipe.toml b/local/recipes/kde/kwin/recipe.toml
index e06d23fd..c343ea4b 100644
--- a/local/recipes/kde/kwin/recipe.toml
+++ b/local/recipes/kde/kwin/recipe.toml
@@ -59,7 +59,7 @@ sed -i '/include(ECMQmlModule)/s/^/#/' "${COOKBOOK_SOURCE}/CMakeLists.txt" 2>/de
 rm -f CMakeCache.txt
 rm -rf CMakeFiles
 
-if cmake "${COOKBOOK_SOURCE}" \
+cmake "${COOKBOOK_SOURCE}" \
     -DCMAKE_TOOLCHAIN_FILE="${COOKBOOK_ROOT}/local/recipes/qt/redox-toolchain.cmake" \
     -DQT_HOST_PATH="${HOST_BUILD}" \
     -DCMAKE_INSTALL_PREFIX=/usr \
@@ -74,9 +74,9 @@ if cmake "${COOKBOOK_SOURCE}" \
     -DKWIN_BUILD_GLOBALSHORTCUTS=OFF \
     -DKWIN_BUILD_RUNNERS=OFF \
     -DKWIN_BUILD_NOTIFICATIONS=OFF \
-    -Wno-dev 2>&1; then
-    cmake --build . -j${COOKBOOK_MAKE_JOBS} 2>&1
-    cmake --install . --prefix "${STAGE}" 2>&1
+    -Wno-dev 2>&1
+cmake --build . -j${COOKBOOK_MAKE_JOBS} 2>&1
+cmake --install . --prefix "${STAGE}" 2>&1
 
     # Downstream cmake configs for KF6WindowSystem/KF6Config (needed by plasma-framework, plasma-workspace, plasma-desktop — not KWin itself)
     mkdir -p "${STAGE}/lib/cmake/KF6WindowSystem" "${STAGE}/lib/cmake/KF6Config"
@@ -88,7 +88,6 @@ EOFCMAKE
 find_package(Qt6 REQUIRED COMPONENTS Core)
 set(KF6Config_LIBRARIES Qt6::Core)
 EOFCMAKE
-fi
 """
 
 [package]