Merge pull request #41 from andrewdavidmackenzie/pidpath

pidpath - with some extra edits

Author: andrewdavidmackenzie

README.md

@@ -36,7 +36,7 @@ pub fn pidinfo<T: PIDInfo>(pid : i32, arg: u64) -> Result<T, String> (macos)
 pub fn regionfilename(pid: i32, address: u64) -> Result<String, String> (macos)
 ```
 ```
-pub fn pidpath(pid : i32) -> Result<String, String> (macos)
+pub fn pidpath(pid : i32) -> Result<String, String> (macos) (linux)
 ```
 ```
 pub fn libversion() -> Result<(i32, i32), String> (macos)

src/libproc/proc_pid.rs

@@ -13,7 +13,7 @@ use std::{mem, process};
 use std::path::PathBuf;
 use std::ptr;
 
-#[cfg(not(target_os = "macos"))]
+#[cfg(target_os = "linux")]
 use libc::PATH_MAX;
 
 use libc::pid_t;
@@ -24,7 +24,9 @@ use crate::libproc::task_info::{TaskAllInfo, TaskInfo};
 use crate::libproc::thread_info::ThreadInfo;
 use crate::libproc::work_queue_info::WorkQueueInfo;
 
-use self::libc::{c_char, c_int, c_void, readlink};
+#[cfg(target_os = "linux")]
+use self::libc::{c_char, readlink};
+use self::libc::{c_int, c_void};
 
 // Since we cannot access C macros for constants from Rust - I have had to redefine this, based on Apple's source code
 // See http://opensource.apple.com/source/Libc/Libc-594.9.4/darwin/libproc.c
@@ -572,7 +574,6 @@ mod test {
         }
     }
 
-   // 
     #[test]
     // This checks that it cannot find the path of the process with pid -1 and returns correct error messaage
     fn pidpath_test_unknown_pid() {
@@ -598,4 +599,4 @@ mod test {
     fn test_pidcwd_of_self() {
         assert_eq!(env::current_dir().unwrap(), pidcwd(process::id() as i32).unwrap());
     }
-}
+}