From 5523f64c6f939ced94d55c569f592320442e8eb9 Mon Sep 17 00:00:00 2001 From: Christian Duerr Date: Mon, 1 Apr 2019 22:27:27 +0000 Subject: Revert daemon removal This reverts the removal of the call to libc::daemon and instead adds an annotation to ignore the deprecation warnings on macos. This will not be an issue in the future since macOS is only discouraging the use of `daemon`, but I'm not aware of any intention to actually remove it. This fixes #2211. --- src/util.rs | 11 ++++------- 1 file changed, 4 insertions(+), 7 deletions(-) (limited to 'src') diff --git a/src/util.rs b/src/util.rs index 06e8436d..b70f2f16 100644 --- a/src/util.rs +++ b/src/util.rs @@ -14,7 +14,6 @@ use std::ffi::OsStr; use std::process::Command; -use std::process::Stdio; use std::{cmp, io}; #[cfg(not(windows))] @@ -23,6 +22,8 @@ use std::os::unix::process::CommandExt; #[cfg(windows)] use std::os::windows::process::CommandExt; #[cfg(windows)] +use std::process::Stdio; +#[cfg(windows)] use winapi::um::winbase::{CREATE_NEW_PROCESS_GROUP, CREATE_NO_WINDOW}; /// Threading utilities @@ -90,13 +91,9 @@ where { Command::new(program) .args(args) - .stdin(Stdio::null()) - .stdout(Stdio::null()) - .stderr(Stdio::null()) .before_exec(|| unsafe { - if ::libc::fork() != 0 { - std::process::exit(0); - } + #[allow(deprecated)] + libc::daemon(1, 0); Ok(()) }) .spawn()? -- cgit v1.2.3-54-g00ecf