Fixing non-Windows compile

haimgel · Oct 18, 2021 · 1c53510 · 1c53510
1 parent 7bf1158
commit 1c53510
Show file tree

Hide file tree

Showing 2 changed files with 7 additions and 6 deletions.
diff --git a/.github/workflows/build.yml b/.github/workflows/build.yml
@@ -33,10 +33,10 @@ jobs:
           key: ${{ runner.os }}-cargo-${{ hashFiles('**/Cargo.lock') }}
 
       - name: Build
-        run: cargo build --verbose --release
+        run: cargo build --release
 
       - name: Run tests
-        run: cargo test --verbose
+        run: cargo test
 
       - name: Run executable
         run: ./target/release/display_switch || true # todo: --version or --help when those exist

diff --git a/src/main.rs b/src/main.rs
@@ -8,6 +8,8 @@
 extern crate log;
 
 use anyhow::Result;
+
+#[cfg(target_os = "windows")]
 use winapi::um::wincon::{AttachConsole, ATTACH_PARENT_PROCESS};
 
 mod app;
@@ -21,10 +23,9 @@ mod usb;
 /// On Windows, re-attach the console, if parent process has the console. This allows
 /// to see the log output when run from the command line.
 fn attach_console() {
-    if cfg!(windows) {
-        unsafe {
-            AttachConsole(ATTACH_PARENT_PROCESS);
-        }
+    #[cfg(target_os = "windows")]
+    unsafe {
+        AttachConsole(ATTACH_PARENT_PROCESS);
     }
 }