From ced1857637bbe97fcb90c806a6b4e5f97e29f2c9 Mon Sep 17 00:00:00 2001 From: bflatastic Date: Fri, 16 Oct 2026 18:39:03 +0700 Subject: [PATCH] Templates: the hard-link check covers every Unix, not only Linux Co-Authored-By: Claude Opus 5.5 --- apps/effectcraft-cli/src/template.rs | 2 +- apps/effectcraft-cli/tests/qa_template.rs | 4 ++-- 2 files changed, 3 insertions(+), 3 deletions(-) diff --git a/apps/effectcraft-cli/src/template.rs b/apps/effectcraft-cli/src/template.rs index a4247f2a..f84303dd 100644 --- a/apps/effectcraft-cli/src/template.rs +++ b/apps/effectcraft-cli/src/template.rs @@ -40,7 +40,7 @@ pub(super) fn validate(cmd: &str, args: &Args) -> Result<(), Failure> { if path == output { return true; } - #[cfg(target_os = "linux")] + #[cfg(unix)] { use std::os::unix::fs::MetadataExt; // Distinct canonical paths can still name the same file through hard links. diff --git a/apps/effectcraft-cli/tests/qa_template.rs b/apps/effectcraft-cli/tests/qa_template.rs index ccab23be..2765b163 100644 --- a/apps/effectcraft-cli/tests/qa_template.rs +++ b/apps/effectcraft-cli/tests/qa_template.rs @@ -654,7 +654,7 @@ fn template_missing_output_does_not_replace_extracted_media() { let _ = std::fs::remove_dir_all(&dir); } -#[cfg(target_os = "linux")] +#[cfg(unix)] #[test] fn template_frame_output_cannot_alias_its_template_through_a_hard_link() { use std::os::unix::fs::MetadataExt; @@ -672,7 +672,7 @@ fn template_frame_output_cannot_alias_its_template_through_a_hard_link() { let _ = std::fs::remove_dir_all(&dir); } -#[cfg(target_os = "linux")] +#[cfg(unix)] #[test] fn template_frame_output_cannot_alias_its_values_through_a_hard_link() { use std::os::unix::fs::MetadataExt;