Skip to content

Commit 810ebe0

Browse files
authored
fix: restrict windows-sys to Windows target (#8522)
I attempted to build codex on LoongArch Linux and encountered compilation errors. After investigation, the errors were traced to certain `windows-sys` features which rely on platform-specific cfgs that only support x86 and aarch64. With this change applied, the project now builds and runs successfully on my platform: - OS: AOSC OS (loongarch64) - Kernel: Linux 6.17 - CPU: Loongson-3A6000 Please let me know if this approach is reasonable, or if there is a better way to support additional platforms.
1 parent bf73260 commit 810ebe0

File tree

1 file changed

+1
-1
lines changed

1 file changed

+1
-1
lines changed

codex-rs/windows-sandbox-rs/Cargo.toml

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -47,7 +47,7 @@ version = "0.8"
4747
[dependencies.dirs-next]
4848
version = "2.0"
4949

50-
[dependencies.windows-sys]
50+
[target.'cfg(windows)'.dependencies.windows-sys]
5151
features = [
5252
"Win32_Foundation",
5353
"Win32_System_Diagnostics_Debug",

0 commit comments

Comments
 (0)