refactor(sort): restrict nix::libc and fd_soft_limit to Linux

mattsu2020 · mattsu2020 · commit dc309e6965ee · 2025-11-29T21:10:23.000+09:00
Update conditional compilation attributes from #[cfg(unix)] to #[cfg(target_os = "linux")]
for the nix::libc import and fd_soft_limit function implementations, ensuring these
features are only enabled on Linux systems to improve portability and avoid issues
on other Unix-like platforms.
diff --git a/src/uu/sort/src/sort.rs b/src/uu/sort/src/sort.rs
@@ -25,7 +25,7 @@ use clap::{Arg, ArgAction, Command};
 use custom_str_cmp::custom_str_cmp;
 use ext_sort::ext_sort;
 use fnv::FnvHasher;
-#[cfg(unix)]
+#[cfg(target_os = "linux")]
 use nix::libc;
 use numeric_str_cmp::{NumInfo, NumInfoParseSettings, human_numeric_str_cmp, numeric_str_cmp};
 use rand::{Rng, rng};
@@ -1072,7 +1072,7 @@ fn make_sort_mode_arg(mode: &'static str, short: char, help: String) -> Arg {
         )
 }
 
-#[cfg(unix)]
+#[cfg(target_os = "linux")]
 pub(crate) fn fd_soft_limit() -> Option<usize> {
     let mut limit = libc::rlimit {
         rlim_cur: 0,
@@ -1092,7 +1092,7 @@ pub(crate) fn fd_soft_limit() -> Option<usize> {
     }
 }
 
-#[cfg(not(unix))]
+#[cfg(not(target_os = "linux"))]
 pub(crate) fn fd_soft_limit() -> Option<usize> {
     None
 }

Original file line number	Diff line number	Diff line change
`@@ -25,7 +25,7 @@ use clap::{Arg, ArgAction, Command};`
`25`	`25`	`use custom_str_cmp::custom_str_cmp;`
`26`	`26`	`use ext_sort::ext_sort;`
`27`	`27`	`use fnv::FnvHasher;`
`28`		`-#[cfg(unix)]`
	`28`	`+#[cfg(target_os = "linux")]`
`29`	`29`	`use nix::libc;`
`30`	`30`	`use numeric_str_cmp::{NumInfo, NumInfoParseSettings, human_numeric_str_cmp, numeric_str_cmp};`
`31`	`31`	`use rand::{Rng, rng};`
`@@ -1072,7 +1072,7 @@ fn make_sort_mode_arg(mode: &'static str, short: char, help: String) -> Arg {`
`1072`	`1072`	`)`
`1073`	`1073`	`}`
`1074`	`1074`
`1075`		`-#[cfg(unix)]`
	`1075`	`+#[cfg(target_os = "linux")]`
`1076`	`1076`	`pub(crate) fn fd_soft_limit() -> Option<usize> {`
`1077`	`1077`	`let mut limit = libc::rlimit {`
`1078`	`1078`	`rlim_cur: 0,`
`@@ -1092,7 +1092,7 @@ pub(crate) fn fd_soft_limit() -> Option<usize> {`
`1092`	`1092`	`}`
`1093`	`1093`	`}`
`1094`	`1094`
`1095`		`-#[cfg(not(unix))]`
	`1095`	`+#[cfg(not(target_os = "linux"))]`
`1096`	`1096`	`pub(crate) fn fd_soft_limit() -> Option<usize> {`
`1097`	`1097`	`None`
`1098`	`1098`	`}`