fix(hero_embedderd): make Path import unconditional #34

Merged

zaelgohary merged 1 commit from development_fix_path_import_linux into development

2026-05-03 13:17:36 +00:00

zaelgohary commented

2026-05-03 13:10:22 +00:00

Member

Summary

hero_embedderd use of Path::new(path) runs on both macOS and Linux (the linux branch also iterates fallback library paths) but the use std::path::Path; import was gated behind #[cfg(target_os = "macos")], so the linux release build failed with use of undeclared type Path. This re-applies the same fix from 3f9e8c3 (omarz, earlier today) that was reverted in 37ab134 without explanation. The release make install step now produces a binary on linux.

N/A — surfaced as an install-step build failure on linux during deploy.

Changes

dropped the #[cfg(target_os = "macos")] gate from use std::path::Path;

Test Results

cargo build --release -p hero_embedderd passes clean.

## Summary `hero_embedderd` use of `Path::new(path)` runs on both macOS and Linux (the linux branch also iterates fallback library paths) but the `use std::path::Path;` import was gated behind `#[cfg(target_os = "macos")]`, so the linux release build failed with `use of undeclared type Path`. This re-applies the same fix from 3f9e8c3 (omarz, earlier today) that was reverted in 37ab134 without explanation. The release `make install` step now produces a binary on linux. ## Related Issue N/A — surfaced as an install-step build failure on linux during deploy. ## Changes - dropped the `#[cfg(target_os = "macos")]` gate from `use std::path::Path;` ## Test Results cargo build --release -p hero_embedderd passes clean.