Remove usage of kernel32-sys crate

Cargo.toml

@@ -110,11 +110,17 @@ daemonize = "0.3"
 tokio-uds = "0.2"
 
 [target.'cfg(windows)'.dependencies]
-kernel32-sys = "0.2.2"
-winapi = { version = "0.3", features = ["winnls"] }
 tokio-named-pipes = "0.1"
 tokio-reactor = "0.1"
 
+[target.'cfg(windows)'.dependencies.winapi]
+version = "0.3"
+features = [
+    "fileapi",
+    "handleapi",
+    "winnls",
+]
+
 [features]
 default = ["dist-client", "s3"]
 all = ["dist-client", "redis", "s3", "memcached", "gcs", "azure"]

src/commands.rs

@@ -159,6 +159,7 @@ fn run_server_process() -> Result<ServerStartup> {
     use winapi::um::winbase::{
         CREATE_NEW_PROCESS_GROUP, CREATE_UNICODE_ENVIRONMENT, DETACHED_PROCESS,
     };
+    use winapi::um::handleapi::CloseHandle;
 
     trace!("run_server_process");
 
@@ -231,8 +232,8 @@ fn run_server_process() -> Result<ServerStartup> {
         ) == TRUE
     } {
         unsafe {
-            kernel32::CloseHandle(pi.hProcess);
-            kernel32::CloseHandle(pi.hThread);
+            CloseHandle(pi.hProcess);
+            CloseHandle(pi.hThread);
         }
     } else {
         return Err(io::Error::last_os_error().into());

src/compiler/msvc.rs

@@ -429,25 +429,22 @@ pub fn parse_arguments(arguments: &[OsString], cwd: &Path, is_clang: bool) -> Co
 #[cfg(windows)]
 fn normpath(path: &str) -> String {
     use std::os::windows::ffi::OsStringExt;
-    use std::ptr;
     use std::os::windows::io::AsRawHandle;
+    use std::ptr;
+    use winapi::um::fileapi::GetFinalPathNameByHandleW;
     File::open(path)
         .and_then(|f| {
             let handle = f.as_raw_handle();
-            let size = unsafe { kernel32::GetFinalPathNameByHandleW(handle,
-                                                         ptr::null_mut(),
-                                                         0,
-                                                         0)
-            };
+            let size = unsafe { GetFinalPathNameByHandleW(handle, ptr::null_mut(), 0, 0) };
             if size == 0 {
                 return Err(io::Error::last_os_error());
             }
             let mut wchars = Vec::with_capacity(size as usize);
             wchars.resize(size as usize, 0);
-            if unsafe { kernel32::GetFinalPathNameByHandleW(handle,
-                                                            wchars.as_mut_ptr(),
-                                                            wchars.len() as u32,
-                                                            0) } == 0 {
+            if unsafe {
+                GetFinalPathNameByHandleW(handle, wchars.as_mut_ptr(), wchars.len() as u32, 0)
+            } == 0
+            {
                 return Err(io::Error::last_os_error());
             }
             // The return value of GetFinalPathNameByHandleW uses the

src/compiler/rust.rs

@@ -1525,9 +1525,9 @@ impl OutputsRewriter for RustOutputsRewriter {
 #[test]
 #[cfg(all(feature = "dist-client", target_os = "windows"))]
 fn test_rust_outputs_rewriter() {
-    use compiler::compiler::OutputsRewriter;
+    use crate::compiler::compiler::OutputsRewriter;
     use std::io::Write;
-    use test::utils::create_file;
+    use crate::test::utils::create_file;
 
     let mut pt = dist::PathTransformer::new();
     pt.to_dist(Path::new("c:\\")).unwrap();