Skip to content

Commit ff0e1ba

Browse files
branchseerclaude
andcommitted
chore: add changelog and clean up comments
Co-Authored-By: Claude Opus 4.6 (1M context) <noreply@anthropic.com>
1 parent 191d519 commit ff0e1ba

File tree

5 files changed

+6
-6
lines changed

5 files changed

+6
-6
lines changed

.github/workflows/ci.yml

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -164,7 +164,7 @@ jobs:
164164
corepack enable
165165
pnpm install
166166
167-
- run: cargo test --no-fail-fast
167+
- run: cargo test
168168

169169
fmt:
170170
name: Format and Check Deps

CHANGELOG.md

Lines changed: 3 additions & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -0,0 +1,3 @@
1+
# Changelog
2+
3+
- [Added] Musl (Alpine Linux) target support (#273)

crates/fspy/src/lib.rs

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -2,7 +2,7 @@
22
#![feature(once_cell_try)]
33

44
// Persist the injected DLL/shared library somewhere in the filesystem.
5-
// Not needed on musl where LD_PRELOAD is unavailable (seccomp-only tracking).
5+
// Not needed on musl (seccomp-only tracking).
66
#[cfg(not(target_env = "musl"))]
77
mod artifact;
88

crates/fspy_preload_unix/src/lib.rs

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -1,4 +1,4 @@
1-
// On musl targets, fspy_preload_unix is not usable (musl does not support cdylib/LD_PRELOAD).
1+
// On musl targets, fspy_preload_unix is not needed since we can track accesses via seccomp-only.
22
// Compile as an empty crate to avoid build failures from missing libc symbols.
33
#![cfg_attr(not(target_env = "musl"), feature(c_variadic))]
44

crates/fspy_seccomp_unotify/tests/arg_types.rs

Lines changed: 0 additions & 3 deletions
Original file line numberDiff line numberDiff line change
@@ -90,9 +90,6 @@ async fn run_in_pre_exec(
9090
async fn fd_and_path() -> Result<(), Box<dyn Error>> {
9191
let syscalls = run_in_pre_exec(|| {
9292
set_current_dir("/")?;
93-
// Use openat(AT_FDCWD, ...) instead of open() because musl's open()
94-
// uses the native `open` syscall on x86_64, which isn't intercepted by
95-
// the test's openat-only seccomp handler.
9693
let home_fd = openat(AT_FDCWD, c"/home", OFlag::O_PATH, Mode::empty())?;
9794
let _ = openat(home_fd, c"open_at_home", OFlag::O_RDONLY, Mode::empty());
9895
let _ = openat(AT_FDCWD, c"openat_cwd", OFlag::O_RDONLY, Mode::empty());

0 commit comments

Comments
 (0)