fix(parse_size): Add cross-platform total_physical_memory implementations

- Implements total_physical_memory() for macOS (hw.memsize)
- Implements total_physical_memory() for FreeBSD (hw.physmem)
- Implements total_physical_memory() for OpenBSD (hw.physmem)
- Implements total_physical_memory() for NetBSD (hw.physmem64)
- Enables 'sort -S 50%' to work on all BSD-based platforms
- Updates tests to include percentage buffer sizes for BSD platforms

Fixes #7257

Author: naoNao89

src/uucore/src/lib/features/parser/parse_size.rs

@@ -2,7 +2,7 @@
 //
 // For the full copyright and license information, please view the LICENSE
 // file that was distributed with this source code.
-// spell-checker:ignore (ToDO) hdsf ghead gtail ACDBK hexdigit
+// spell-checker:ignore (ToDO) hdsf ghead gtail ACDBK hexdigit memsize physmem
 
 //! Parser for sizes in SI or IEC units (multiples of 1000 or 1024 bytes).
 
@@ -18,7 +18,7 @@ use procfs::{Current, Meminfo};
 enum SystemError {
     IOError,
     ParseError,
-    #[cfg(not(target_os = "linux"))]
+    #[allow(dead_code)]
     NotFound,
 }
 
@@ -77,10 +77,76 @@ pub fn available_memory_bytes() -> Option<u128> {
     None
 }
 
-/// Get the total number of bytes of physical memory.
+/// Get the total number of bytes of physical memory on macOS.
 ///
-/// TODO Implement this for non-Linux systems.
-#[cfg(not(target_os = "linux"))]
+/// Uses the `sysctl` command to query `hw.memsize`.
+#[cfg(target_os = "macos")]
+fn total_physical_memory() -> Result<u128, SystemError> {
+    let output = std::process::Command::new("sysctl")
+        .arg("-n")
+        .arg("hw.memsize")
+        .output()?;
+
+    let mem_str = String::from_utf8_lossy(&output.stdout);
+    let mem_bytes: u128 = mem_str.trim().parse()?;
+    Ok(mem_bytes)
+}
+
+/// Get the total number of bytes of physical memory on FreeBSD.
+///
+/// Uses the `sysctl` command to query `hw.physmem`.
+#[cfg(target_os = "freebsd")]
+fn total_physical_memory() -> Result<u128, SystemError> {
+    let output = std::process::Command::new("sysctl")
+        .arg("-n")
+        .arg("hw.physmem")
+        .output()?;
+
+    let mem_str = String::from_utf8_lossy(&output.stdout);
+    let mem_bytes: u128 = mem_str.trim().parse()?;
+    Ok(mem_bytes)
+}
+
+/// Get the total number of bytes of physical memory on OpenBSD.
+///
+/// Uses the `sysctl` command to query `hw.physmem`.
+#[cfg(target_os = "openbsd")]
+fn total_physical_memory() -> Result<u128, SystemError> {
+    let output = std::process::Command::new("sysctl")
+        .arg("-n")
+        .arg("hw.physmem")
+        .output()?;
+
+    let mem_str = String::from_utf8_lossy(&output.stdout);
+    let mem_bytes: u128 = mem_str.trim().parse()?;
+    Ok(mem_bytes)
+}
+
+/// Get the total number of bytes of physical memory on NetBSD.
+///
+/// Uses the `sysctl` command to query `hw.physmem64`.
+#[cfg(target_os = "netbsd")]
+fn total_physical_memory() -> Result<u128, SystemError> {
+    let output = std::process::Command::new("sysctl")
+        .arg("-n")
+        .arg("hw.physmem64")
+        .output()?;
+
+    let mem_str = String::from_utf8_lossy(&output.stdout);
+    let mem_bytes: u128 = mem_str.trim().parse()?;
+    Ok(mem_bytes)
+}
+
+/// Get the total number of bytes of physical memory (fallback for unsupported platforms).
+///
+/// Returns an error for platforms where we haven't implemented memory detection yet.
+#[cfg(not(any(
+    target_os = "linux",
+    target_os = "macos",
+    target_os = "freebsd",
+    target_os = "openbsd",
+    target_os = "netbsd"
+)))]
 fn total_physical_memory() -> Result<u128, SystemError> {
     Err(SystemError::NotFound)
 }

tests/by-util/test_sort.rs

@@ -33,10 +33,21 @@ fn test_helper(file_name: &str, possible_args: &[&str]) {
 
 #[test]
 fn test_buffer_sizes() {
-    #[cfg(target_os = "linux")]
-    let buffer_sizes = ["0", "50K", "50k", "1M", "100M", "0%", "10%"];
-    // TODO Percentage sizes are not yet supported beyond Linux.
-    #[cfg(not(target_os = "linux"))]
+    #[cfg(any(
+        target_os = "linux",
+        target_os = "macos",
+        target_os = "freebsd",
+        target_os = "openbsd",
+        target_os = "netbsd"
+    ))]
+    let buffer_sizes = ["0", "50K", "50k", "1M", "100M", "10%", "50%"];
+    #[cfg(not(any(
+        target_os = "linux",
+        target_os = "macos",
+        target_os = "freebsd",
+        target_os = "openbsd",
+        target_os = "netbsd"
+    )))]
     let buffer_sizes = ["0", "50K", "50k", "1M", "100M"];
     for buffer_size in &buffer_sizes {
         new_ucmd!()