fix shutdown-signal for non-unix

README.md

@@ -45,6 +45,9 @@ SKUNK_UI_DEV=1
 HOSTAPD_CC=US
 ```
 
+You can also put your environment variables in a `.env` file.
+
+
 [1]: https://en.wikipedia.org/wiki/Man-in-the-middle_attack
 [2]: https://github.com/jgraef/skunk/issues
 [3]: https://trunkrs.dev/

skunk-cli/src/app.rs

@@ -358,7 +358,7 @@ fn cancel_on_ctrlc_or_sigterm() -> CancellationToken {
             .await;
 
         #[cfg(not(unix))]
-        futures::future::pending().await;
+        std::future::pending::<()>().await;
     }
 
     tokio::spawn({

skunk-util/src/trigger.rs

@@ -1,11 +1,4 @@
-use std::{
-    future::Future,
-    pin::Pin,
-    task::{
-        Context,
-        Poll,
-    },
-};
+use std::future::pending;
 
 use tokio::sync::watch;
 
@@ -47,7 +40,7 @@ impl Receiver {
 
     pub async fn triggered(&mut self) {
         if let Err(Closed) = self.triggered_or_closed().await {
-            Pending.await;
+            pending::<()>().await;
         }
     }
 }
@@ -59,14 +52,3 @@ pub fn new() -> (Sender, Receiver) {
     let (tx, rx) = watch::channel(());
     (Sender { tx }, Receiver { rx })
 }
-
-// todo: we could also just import futures_util
-struct Pending;
-
-impl Future for Pending {
-    type Output = ();
-
-    fn poll(self: Pin<&mut Self>, _cx: &mut Context<'_>) -> Poll<Self::Output> {
-        Poll::Pending
-    }
-}