Merge pull request #385 from joshrotenberg/fix/musl-build-keyring

fix: remove musl target from cargo-dist due to keyring incompatibility

Author: joshrotenberg

Cargo.toml

@@ -34,7 +34,8 @@ ci = "github"
 # The installers to generate for each app
 installers = ["shell", "powershell"]
 # Target platforms to build apps for (Rust target-triple syntax)
-targets = ["aarch64-apple-darwin", "x86_64-apple-darwin", "x86_64-unknown-linux-gnu", "x86_64-unknown-linux-musl", "x86_64-pc-windows-msvc"]
+# Note: x86_64-unknown-linux-musl removed due to keyring incompatibility with musl
+targets = ["aarch64-apple-darwin", "x86_64-apple-darwin", "x86_64-unknown-linux-gnu", "x86_64-pc-windows-msvc"]
 # Which actions to run on pull requests
 pr-run-mode = "plan"
 # Whether dist should create a Github Release or use an existing draft

crates/redisctl/Cargo.toml

@@ -66,6 +66,8 @@ full = ["cloud", "enterprise", "upload"]
 cloud = []
 enterprise = []
 upload = ["dep:files-sdk"]
+# Note: secure-storage may not work on musl targets due to keyring native dependencies
+# Build with --no-default-features --features full for musl
 secure-storage = ["dep:keyring"]
 
 [dev-dependencies]