Fix: target_family unix means also macos and linux; use BSD target_os

This commit is contained in:
veeso
2021-06-21 09:11:28 +02:00
parent 04cafc4181
commit c78fc583b0
10 changed files with 61 additions and 51 deletions

View File

@@ -62,8 +62,8 @@ pretty_assertions = "0.7.2"
github-actions = [] github-actions = []
with-containers = [] with-containers = []
[target."cfg(any(target_family = \"unix\", target_os = \"macos\", target_os = \"linux\"))"] [target."cfg(target_family = \"unix\")"]
[target."cfg(any(target_family = \"unix\", target_os = \"macos\", target_os = \"linux\"))".dependencies] [target."cfg(target_family = \"unix\")".dependencies]
users = "0.11.0" users = "0.11.0"
[target."cfg(any(target_os = \"windows\", target_os = \"macos\"))"] [target."cfg(any(target_os = \"windows\", target_os = \"macos\"))"]

View File

@@ -225,7 +225,7 @@ mod tests {
PathBuf::from(cfg.user_interface.text_editor.file_name().unwrap()), // NOTE: since edit 0.1.3 real path is used PathBuf::from(cfg.user_interface.text_editor.file_name().unwrap()), // NOTE: since edit 0.1.3 real path is used
PathBuf::from("vim.EXE") PathBuf::from("vim.EXE")
); );
#[cfg(any(target_family = "unix", target_os = "macos", target_os = "linux"))] #[cfg(target_family = "unix")]
assert_eq!( assert_eq!(
PathBuf::from(cfg.user_interface.text_editor.file_name().unwrap()), // NOTE: since edit 0.1.3 real path is used PathBuf::from(cfg.user_interface.text_editor.file_name().unwrap()), // NOTE: since edit 0.1.3 real path is used
PathBuf::from("vim") PathBuf::from("vim")

View File

@@ -73,7 +73,7 @@ impl FtpFileTransfer {
PathBuf::from(path_slash::PathExt::to_slash_lossy(p).as_str()) PathBuf::from(path_slash::PathExt::to_slash_lossy(p).as_str())
} }
#[cfg(any(target_family = "unix", target_os = "macos", target_os = "linux"))] #[cfg(target_family = "unix")]
fn resolve(p: &Path) -> PathBuf { fn resolve(p: &Path) -> PathBuf {
p.to_path_buf() p.to_path_buf()
} }

View File

@@ -77,7 +77,7 @@ impl ScpFileTransfer {
PathBuf::from(path_slash::PathExt::to_slash_lossy(p).as_str()) PathBuf::from(path_slash::PathExt::to_slash_lossy(p).as_str())
} }
#[cfg(any(target_family = "unix", target_os = "macos", target_os = "linux"))] #[cfg(target_family = "unix")]
fn resolve(p: &Path) -> PathBuf { fn resolve(p: &Path) -> PathBuf {
p.to_path_buf() p.to_path_buf()
} }

View File

@@ -28,7 +28,7 @@
// Deps // Deps
extern crate bytesize; extern crate bytesize;
extern crate regex; extern crate regex;
#[cfg(any(target_family = "unix", target_os = "macos", target_os = "linux"))] #[cfg(target_family = "unix")]
extern crate users; extern crate users;
// Locals // Locals
use super::FsEntry; use super::FsEntry;
@@ -36,7 +36,7 @@ use crate::utils::fmt::{fmt_path_elide, fmt_pex, fmt_time};
// Ext // Ext
use bytesize::ByteSize; use bytesize::ByteSize;
use regex::Regex; use regex::Regex;
#[cfg(any(target_family = "unix", target_os = "macos", target_os = "linux"))] #[cfg(target_family = "unix")]
use users::{get_group_by_gid, get_user_by_uid}; use users::{get_group_by_gid, get_user_by_uid};
// Types // Types
// FmtCallback: Formatter, fsentry: &FsEntry, cur_str, prefix, length, extra // FmtCallback: Formatter, fsentry: &FsEntry, cur_str, prefix, length, extra
@@ -251,7 +251,7 @@ impl Formatter {
_fmt_extra: Option<&String>, _fmt_extra: Option<&String>,
) -> String { ) -> String {
// Get username // Get username
#[cfg(any(target_family = "unix", target_os = "macos", target_os = "linux"))] #[cfg(target_family = "unix")]
let group: String = match fsentry.get_group() { let group: String = match fsentry.get_group() {
Some(gid) => match get_group_by_gid(gid) { Some(gid) => match get_group_by_gid(gid) {
Some(user) => user.name().to_string_lossy().to_string(), Some(user) => user.name().to_string_lossy().to_string(),
@@ -431,7 +431,7 @@ impl Formatter {
_fmt_extra: Option<&String>, _fmt_extra: Option<&String>,
) -> String { ) -> String {
// Get username // Get username
#[cfg(any(target_family = "unix", target_os = "macos", target_os = "linux"))] #[cfg(target_family = "unix")]
let username: String = match fsentry.get_user() { let username: String = match fsentry.get_user() {
Some(uid) => match get_user_by_uid(uid) { Some(uid) => match get_user_by_uid(uid) {
Some(user) => user.name().to_string_lossy().to_string(), Some(user) => user.name().to_string_lossy().to_string(),
@@ -605,7 +605,7 @@ mod tests {
group: Some(0), // UNIX only group: Some(0), // UNIX only
unix_pex: Some((6, 4, 4)), // UNIX only unix_pex: Some((6, 4, 4)), // UNIX only
}); });
#[cfg(any(target_family = "unix", target_os = "macos", target_os = "linux"))] #[cfg(target_family = "unix")]
assert_eq!( assert_eq!(
formatter.fmt(&entry), formatter.fmt(&entry),
format!( format!(
@@ -636,7 +636,7 @@ mod tests {
group: Some(0), // UNIX only group: Some(0), // UNIX only
unix_pex: Some((6, 4, 4)), // UNIX only unix_pex: Some((6, 4, 4)), // UNIX only
}); });
#[cfg(any(target_family = "unix", target_os = "macos", target_os = "linux"))] #[cfg(target_family = "unix")]
assert_eq!( assert_eq!(
formatter.fmt(&entry), formatter.fmt(&entry),
format!( format!(
@@ -667,7 +667,7 @@ mod tests {
group: Some(0), // UNIX only group: Some(0), // UNIX only
unix_pex: None, // UNIX only unix_pex: None, // UNIX only
}); });
#[cfg(any(target_family = "unix", target_os = "macos", target_os = "linux"))] #[cfg(target_family = "unix")]
assert_eq!( assert_eq!(
formatter.fmt(&entry), formatter.fmt(&entry),
format!( format!(
@@ -698,7 +698,7 @@ mod tests {
group: Some(0), // UNIX only group: Some(0), // UNIX only
unix_pex: None, // UNIX only unix_pex: None, // UNIX only
}); });
#[cfg(any(target_family = "unix", target_os = "macos", target_os = "linux"))] #[cfg(target_family = "unix")]
assert_eq!( assert_eq!(
formatter.fmt(&entry), formatter.fmt(&entry),
format!( format!(
@@ -734,7 +734,7 @@ mod tests {
group: Some(0), // UNIX only group: Some(0), // UNIX only
unix_pex: Some((7, 5, 5)), // UNIX only unix_pex: Some((7, 5, 5)), // UNIX only
}); });
#[cfg(any(target_family = "unix", target_os = "macos", target_os = "linux"))] #[cfg(target_family = "unix")]
assert_eq!( assert_eq!(
formatter.fmt(&entry), formatter.fmt(&entry),
format!( format!(
@@ -763,7 +763,7 @@ mod tests {
group: Some(0), // UNIX only group: Some(0), // UNIX only
unix_pex: None, // UNIX only unix_pex: None, // UNIX only
}); });
#[cfg(any(target_family = "unix", target_os = "macos", target_os = "linux"))] #[cfg(target_family = "unix")]
assert_eq!( assert_eq!(
formatter.fmt(&entry), formatter.fmt(&entry),
format!( format!(

View File

@@ -595,7 +595,7 @@ mod tests {
group: Some(0), // UNIX only group: Some(0), // UNIX only
unix_pex: Some((6, 4, 4)), // UNIX only unix_pex: Some((6, 4, 4)), // UNIX only
}); });
#[cfg(any(target_family = "unix", target_os = "macos", target_os = "linux"))] #[cfg(target_family = "unix")]
assert_eq!( assert_eq!(
explorer.fmt_file(&entry), explorer.fmt_file(&entry),
format!( format!(

View File

@@ -34,9 +34,9 @@ use std::time::SystemTime;
use thiserror::Error; use thiserror::Error;
use wildmatch::WildMatch; use wildmatch::WildMatch;
// Metadata ext // Metadata ext
#[cfg(any(target_family = "unix", target_os = "macos", target_os = "linux"))] #[cfg(target_family = "unix")]
use std::fs::set_permissions; use std::fs::set_permissions;
#[cfg(any(target_family = "unix", target_os = "macos", target_os = "linux"))] #[cfg(target_family = "unix")]
use std::os::unix::fs::{MetadataExt, PermissionsExt}; use std::os::unix::fs::{MetadataExt, PermissionsExt};
// Locals // Locals
@@ -439,7 +439,7 @@ impl Localhost {
/// ### stat /// ### stat
/// ///
/// Stat file and create a FsEntry /// Stat file and create a FsEntry
#[cfg(any(target_family = "unix", target_os = "macos", target_os = "linux"))] #[cfg(target_family = "unix")]
pub fn stat(&self, path: &Path) -> Result<FsEntry, HostError> { pub fn stat(&self, path: &Path) -> Result<FsEntry, HostError> {
info!("Stating file {}", path.display()); info!("Stating file {}", path.display());
let path: PathBuf = self.to_abs_path(path); let path: PathBuf = self.to_abs_path(path);
@@ -605,7 +605,7 @@ impl Localhost {
/// ### chmod /// ### chmod
/// ///
/// Change file mode to file, according to UNIX permissions /// Change file mode to file, according to UNIX permissions
#[cfg(any(target_family = "unix", target_os = "macos", target_os = "linux"))] #[cfg(target_family = "unix")]
pub fn chmod(&self, path: &Path, pex: (u8, u8, u8)) -> Result<(), HostError> { pub fn chmod(&self, path: &Path, pex: (u8, u8, u8)) -> Result<(), HostError> {
let path: PathBuf = self.to_abs_path(path); let path: PathBuf = self.to_abs_path(path);
// Get metadta // Get metadta
@@ -790,7 +790,7 @@ impl Localhost {
/// ### u32_to_mode /// ### u32_to_mode
/// ///
/// Return string with format xxxxxx to tuple of permissions (user, group, others) /// Return string with format xxxxxx to tuple of permissions (user, group, others)
#[cfg(any(target_family = "unix", target_os = "macos", target_os = "linux"))] #[cfg(target_family = "unix")]
fn u32_to_mode(&self, mode: u32) -> (u8, u8, u8) { fn u32_to_mode(&self, mode: u32) -> (u8, u8, u8) {
let user: u8 = ((mode >> 6) & 0x7) as u8; let user: u8 = ((mode >> 6) & 0x7) as u8;
let group: u8 = ((mode >> 3) & 0x7) as u8; let group: u8 = ((mode >> 3) & 0x7) as u8;
@@ -801,7 +801,7 @@ impl Localhost {
/// mode_to_u32 /// mode_to_u32
/// ///
/// Convert owner,group,others to u32 /// Convert owner,group,others to u32
#[cfg(any(target_family = "unix", target_os = "macos", target_os = "linux"))] #[cfg(target_family = "unix")]
fn mode_to_u32(&self, mode: (u8, u8, u8)) -> u32 { fn mode_to_u32(&self, mode: (u8, u8, u8)) -> u32 {
((mode.0 as u32) << 6) + ((mode.1 as u32) << 3) + mode.2 as u32 ((mode.0 as u32) << 6) + ((mode.1 as u32) << 3) + mode.2 as u32
} }
@@ -826,17 +826,17 @@ impl Localhost {
mod tests { mod tests {
use super::*; use super::*;
#[cfg(any(target_family = "unix", target_os = "macos", target_os = "linux"))] #[cfg(target_family = "unix")]
use crate::utils::test_helpers::{create_sample_file, make_fsentry}; use crate::utils::test_helpers::{create_sample_file, make_fsentry};
use crate::utils::test_helpers::{make_dir_at, make_file_at}; use crate::utils::test_helpers::{make_dir_at, make_file_at};
use pretty_assertions::assert_eq; use pretty_assertions::assert_eq;
#[cfg(any(target_family = "unix", target_os = "macos", target_os = "linux"))] #[cfg(target_family = "unix")]
use std::fs::File; use std::fs::File;
#[cfg(any(target_family = "unix", target_os = "macos", target_os = "linux"))] #[cfg(target_family = "unix")]
use std::io::Write; use std::io::Write;
#[cfg(any(target_family = "unix", target_os = "macos", target_os = "linux"))] #[cfg(target_family = "unix")]
use std::os::unix::fs::{symlink, PermissionsExt}; use std::os::unix::fs::{symlink, PermissionsExt};
#[test] #[test]
@@ -848,7 +848,7 @@ mod tests {
} }
#[test] #[test]
#[cfg(any(target_family = "unix", target_os = "macos", target_os = "linux"))] #[cfg(target_family = "unix")]
fn test_host_localhost_new() { fn test_host_localhost_new() {
let host: Localhost = Localhost::new(PathBuf::from("/dev")).ok().unwrap(); let host: Localhost = Localhost::new(PathBuf::from("/dev")).ok().unwrap();
assert_eq!(host.wrkdir, PathBuf::from("/dev")); assert_eq!(host.wrkdir, PathBuf::from("/dev"));
@@ -884,14 +884,14 @@ mod tests {
} }
#[test] #[test]
#[cfg(any(target_family = "unix", target_os = "macos", target_os = "linux"))] #[cfg(target_family = "unix")]
fn test_host_localhost_pwd() { fn test_host_localhost_pwd() {
let host: Localhost = Localhost::new(PathBuf::from("/dev")).ok().unwrap(); let host: Localhost = Localhost::new(PathBuf::from("/dev")).ok().unwrap();
assert_eq!(host.pwd(), PathBuf::from("/dev")); assert_eq!(host.pwd(), PathBuf::from("/dev"));
} }
#[test] #[test]
#[cfg(any(target_family = "unix", target_os = "macos", target_os = "linux"))] #[cfg(target_family = "unix")]
fn test_host_localhost_list_files() { fn test_host_localhost_list_files() {
let host: Localhost = Localhost::new(PathBuf::from("/dev")).ok().unwrap(); let host: Localhost = Localhost::new(PathBuf::from("/dev")).ok().unwrap();
// Scan dir // Scan dir
@@ -904,7 +904,7 @@ mod tests {
} }
#[test] #[test]
#[cfg(any(target_family = "unix", target_os = "macos", target_os = "linux"))] #[cfg(target_family = "unix")]
fn test_host_localhost_change_dir() { fn test_host_localhost_change_dir() {
let mut host: Localhost = Localhost::new(PathBuf::from("/dev")).ok().unwrap(); let mut host: Localhost = Localhost::new(PathBuf::from("/dev")).ok().unwrap();
let new_dir: PathBuf = PathBuf::from("/dev"); let new_dir: PathBuf = PathBuf::from("/dev");
@@ -920,7 +920,7 @@ mod tests {
} }
#[test] #[test]
#[cfg(any(target_family = "unix", target_os = "macos", target_os = "linux"))] #[cfg(target_family = "unix")]
#[should_panic] #[should_panic]
fn test_host_localhost_change_dir_failed() { fn test_host_localhost_change_dir_failed() {
let mut host: Localhost = Localhost::new(PathBuf::from("/dev")).ok().unwrap(); let mut host: Localhost = Localhost::new(PathBuf::from("/dev")).ok().unwrap();
@@ -929,7 +929,7 @@ mod tests {
} }
#[test] #[test]
#[cfg(any(target_family = "unix", target_os = "macos", target_os = "linux"))] #[cfg(target_family = "unix")]
fn test_host_localhost_open_read() { fn test_host_localhost_open_read() {
let host: Localhost = Localhost::new(PathBuf::from("/dev")).ok().unwrap(); let host: Localhost = Localhost::new(PathBuf::from("/dev")).ok().unwrap();
// Create temp file // Create temp file
@@ -938,7 +938,7 @@ mod tests {
} }
#[test] #[test]
#[cfg(any(target_family = "unix", target_os = "macos", target_os = "linux"))] #[cfg(target_family = "unix")]
#[should_panic] #[should_panic]
fn test_host_localhost_open_read_err_no_such_file() { fn test_host_localhost_open_read_err_no_such_file() {
let host: Localhost = Localhost::new(PathBuf::from("/dev")).ok().unwrap(); let host: Localhost = Localhost::new(PathBuf::from("/dev")).ok().unwrap();
@@ -959,7 +959,7 @@ mod tests {
} }
#[test] #[test]
#[cfg(any(target_family = "unix", target_os = "macos", target_os = "linux"))] #[cfg(target_family = "unix")]
fn test_host_localhost_open_write() { fn test_host_localhost_open_write() {
let host: Localhost = Localhost::new(PathBuf::from("/dev")).ok().unwrap(); let host: Localhost = Localhost::new(PathBuf::from("/dev")).ok().unwrap();
// Create temp file // Create temp file
@@ -978,7 +978,7 @@ mod tests {
assert!(host.open_file_write(file.path()).is_err()); assert!(host.open_file_write(file.path()).is_err());
} }
#[cfg(any(target_family = "unix", target_os = "macos", target_os = "linux"))] #[cfg(target_family = "unix")]
#[test] #[test]
fn test_host_localhost_symlinks() { fn test_host_localhost_symlinks() {
let tmpdir: tempfile::TempDir = tempfile::TempDir::new().unwrap(); let tmpdir: tempfile::TempDir = tempfile::TempDir::new().unwrap();
@@ -1026,7 +1026,7 @@ mod tests {
} }
#[test] #[test]
#[cfg(any(target_family = "unix", target_os = "macos", target_os = "linux"))] #[cfg(target_family = "unix")]
fn test_host_localhost_mkdir() { fn test_host_localhost_mkdir() {
let tmpdir: tempfile::TempDir = tempfile::TempDir::new().unwrap(); let tmpdir: tempfile::TempDir = tempfile::TempDir::new().unwrap();
let mut host: Localhost = Localhost::new(PathBuf::from(tmpdir.path())).ok().unwrap(); let mut host: Localhost = Localhost::new(PathBuf::from(tmpdir.path())).ok().unwrap();
@@ -1051,7 +1051,7 @@ mod tests {
} }
#[test] #[test]
#[cfg(any(target_family = "unix", target_os = "macos", target_os = "linux"))] #[cfg(target_family = "unix")]
fn test_host_localhost_remove() { fn test_host_localhost_remove() {
let tmpdir: tempfile::TempDir = tempfile::TempDir::new().unwrap(); let tmpdir: tempfile::TempDir = tempfile::TempDir::new().unwrap();
// Create sample file // Create sample file
@@ -1080,7 +1080,7 @@ mod tests {
} }
#[test] #[test]
#[cfg(any(target_family = "unix", target_os = "macos", target_os = "linux"))] #[cfg(target_family = "unix")]
fn test_host_localhost_rename() { fn test_host_localhost_rename() {
let tmpdir: tempfile::TempDir = tempfile::TempDir::new().unwrap(); let tmpdir: tempfile::TempDir = tempfile::TempDir::new().unwrap();
// Create sample file // Create sample file
@@ -1108,7 +1108,7 @@ mod tests {
.is_err()); .is_err());
} }
#[cfg(any(target_family = "unix", target_os = "macos", target_os = "linux"))] #[cfg(target_family = "unix")]
#[test] #[test]
fn test_host_chmod() { fn test_host_chmod() {
let tmpdir: tempfile::TempDir = tempfile::TempDir::new().unwrap(); let tmpdir: tempfile::TempDir = tempfile::TempDir::new().unwrap();
@@ -1127,7 +1127,7 @@ mod tests {
.is_err()); .is_err());
} }
#[cfg(any(target_family = "unix", target_os = "macos", target_os = "linux"))] #[cfg(target_family = "unix")]
#[test] #[test]
fn test_host_copy_file_absolute() { fn test_host_copy_file_absolute() {
let tmpdir: tempfile::TempDir = tempfile::TempDir::new().unwrap(); let tmpdir: tempfile::TempDir = tempfile::TempDir::new().unwrap();
@@ -1157,7 +1157,7 @@ mod tests {
.is_err()); .is_err());
} }
#[cfg(any(target_family = "unix", target_os = "macos", target_os = "linux"))] #[cfg(target_family = "unix")]
#[test] #[test]
fn test_host_copy_file_relative() { fn test_host_copy_file_relative() {
let tmpdir: tempfile::TempDir = tempfile::TempDir::new().unwrap(); let tmpdir: tempfile::TempDir = tempfile::TempDir::new().unwrap();
@@ -1179,7 +1179,7 @@ mod tests {
assert_eq!(host.files.len(), 2); assert_eq!(host.files.len(), 2);
} }
#[cfg(any(target_family = "unix", target_os = "macos", target_os = "linux"))] #[cfg(target_family = "unix")]
#[test] #[test]
fn test_host_copy_directory_absolute() { fn test_host_copy_directory_absolute() {
let tmpdir: tempfile::TempDir = tempfile::TempDir::new().unwrap(); let tmpdir: tempfile::TempDir = tempfile::TempDir::new().unwrap();
@@ -1210,7 +1210,7 @@ mod tests {
assert!(host.stat(test_file_path.as_path()).is_ok()); assert!(host.stat(test_file_path.as_path()).is_ok());
} }
#[cfg(any(target_family = "unix", target_os = "macos", target_os = "linux"))] #[cfg(target_family = "unix")]
#[test] #[test]
fn test_host_copy_directory_relative() { fn test_host_copy_directory_relative() {
let tmpdir: tempfile::TempDir = tempfile::TempDir::new().unwrap(); let tmpdir: tempfile::TempDir = tempfile::TempDir::new().unwrap();
@@ -1245,7 +1245,7 @@ mod tests {
let tmpdir: tempfile::TempDir = tempfile::TempDir::new().unwrap(); let tmpdir: tempfile::TempDir = tempfile::TempDir::new().unwrap();
let host: Localhost = Localhost::new(PathBuf::from(tmpdir.path())).ok().unwrap(); let host: Localhost = Localhost::new(PathBuf::from(tmpdir.path())).ok().unwrap();
// Execute // Execute
#[cfg(any(target_family = "unix", target_os = "macos", target_os = "linux"))] #[cfg(target_family = "unix")]
assert_eq!(host.exec("echo 5").ok().unwrap().as_str(), "5\n"); assert_eq!(host.exec("echo 5").ok().unwrap().as_str(), "5\n");
#[cfg(target_os = "windows")] #[cfg(target_os = "windows")]
assert_eq!(host.exec("echo 5").ok().unwrap().as_str(), "5\r\n"); assert_eq!(host.exec("echo 5").ok().unwrap().as_str(), "5\r\n");

View File

@@ -92,7 +92,12 @@ impl BookmarksClient {
} }
}; };
// Make a key storage (linux / unix) // Make a key storage (linux / unix)
#[cfg(any(target_os = "linux", target_family = "unix"))] #[cfg(any(
target_os = "linux",
target_os = "freebsd",
target_os = "netbsd",
target_os = "netbsd"
))]
let (key_storage, service_id): (Box<dyn KeyStorage>, &str) = { let (key_storage, service_id): (Box<dyn KeyStorage>, &str) = {
#[cfg(not(test))] #[cfg(not(test))]
let app_name: &str = "bookmarks"; let app_name: &str = "bookmarks";
@@ -446,7 +451,12 @@ mod tests {
} }
#[test] #[test]
#[cfg(any(target_family = "unix", target_os = "linux"))] #[cfg(any(
target_os = "linux",
target_os = "freebsd",
target_os = "netbsd",
target_os = "netbsd"
))]
fn test_system_bookmarks_new_err() { fn test_system_bookmarks_new_err() {
assert!(BookmarksClient::new( assert!(BookmarksClient::new(
Path::new("/tmp/oifoif/omar"), Path::new("/tmp/oifoif/omar"),

View File

@@ -28,7 +28,7 @@
// Deps // Deps
extern crate bytesize; extern crate bytesize;
extern crate hostname; extern crate hostname;
#[cfg(any(target_family = "unix", target_os = "macos", target_os = "linux"))] #[cfg(target_family = "unix")]
extern crate users; extern crate users;
// locals // locals
use super::{browser::FileExplorerTab, Context, FileTransferActivity}; use super::{browser::FileExplorerTab, Context, FileTransferActivity};
@@ -59,7 +59,7 @@ use tuirealm::tui::{
style::Color, style::Color,
widgets::{BorderType, Borders, Clear}, widgets::{BorderType, Borders, Clear},
}; };
#[cfg(any(target_family = "unix", target_os = "macos", target_os = "linux"))] #[cfg(target_family = "unix")]
use users::{get_group_by_gid, get_user_by_uid}; use users::{get_group_by_gid, get_user_by_uid};
impl FileTransferActivity { impl FileTransferActivity {
@@ -813,7 +813,7 @@ impl FileTransferActivity {
.build(), .build(),
); );
// User // User
#[cfg(any(target_family = "unix", target_os = "macos", target_os = "linux"))] #[cfg(target_family = "unix")]
let username: String = match file.get_user() { let username: String = match file.get_user() {
Some(uid) => match get_user_by_uid(uid) { Some(uid) => match get_user_by_uid(uid) {
Some(user) => user.name().to_string_lossy().to_string(), Some(user) => user.name().to_string_lossy().to_string(),
@@ -824,7 +824,7 @@ impl FileTransferActivity {
#[cfg(target_os = "windows")] #[cfg(target_os = "windows")]
let username: String = format!("{}", file.get_user().unwrap_or(0)); let username: String = format!("{}", file.get_user().unwrap_or(0));
// Group // Group
#[cfg(any(target_family = "unix", target_os = "macos", target_os = "linux"))] #[cfg(target_family = "unix")]
let group: String = match file.get_group() { let group: String = match file.get_group() {
Some(gid) => match get_group_by_gid(gid) { Some(gid) => match get_group_by_gid(gid) {
Some(group) => group.name().to_string_lossy().to_string(), Some(group) => group.name().to_string_lossy().to_string(),

View File

@@ -216,7 +216,7 @@ mod tests {
} }
#[test] #[test]
#[cfg(any(target_family = "unix", target_os = "linux", target_os = "macos"))] #[cfg(target_family = "unix")]
fn test_utils_fmt_path_elide() { fn test_utils_fmt_path_elide() {
let p: &Path = &Path::new("/develop/pippo"); let p: &Path = &Path::new("/develop/pippo");
// Under max size // Under max size