From 9d4c7683dcac0b6aae9103e471d519a4fa4cfe03 Mon Sep 17 00:00:00 2001
From: KhooLy <73142442+KhooLy@users.noreply.github.com>
Date: Tue, 28 Jul 2026 02:33:08 +0300
Subject: [PATCH] fix(core): also enable rquickjs bindgen for iOS targets

rquickjs-sys ships no prebuilt bindings for any Apple *-ios target
either (only desktop aarch64/x86_64-apple-darwin), and fluxa_core's
"ios" feature also pulls in plugin-js-engine -> rquickjs. Scoping
bindgen to Android only would have left iOS builds hitting the same
missing-bindings error the moment they actually cross-compiled with
that feature set. Widen the target cfg to cover both.

Could not verify by actually cross-compiling for an iOS target from
this Linux sandbox (no Apple toolchain available) -- verified via the
prebuilt bindings list instead (src/bindings/ in the rquickjs-sys
source has no ios entries) and confirmed the desktop/Android builds
are unaffected by the wider cfg.
---
 Cargo.toml | 2 +-
 1 file changed, 1 insertion(+), 1 deletion(-)

diff --git a/Cargo.toml b/Cargo.toml
index 84fc754..5a29d54 100644
--- a/Cargo.toml
+++ b/Cargo.toml
@@ -108,7 +108,7 @@ uniffi = { version = "0.31.1", optional = true }
 wasm-bindgen = { version = "0.2", optional = true }
 web-time = "1"
 
-[target.'cfg(target_os = "android")'.dependencies]
+[target.'cfg(any(target_os = "android", target_os = "ios"))'.dependencies]
 rquickjs = { version = "0.12.1", default-features = false, features = ["std", "chrono", "loader", "dyn-load", "either", "indexmap", "futures", "bindgen"], optional = true }
 
 [dev-dependencies]