Update notify-rust to 4.7 (#114)

This included the `FromStr` for `Timeout` implementation
(hoodie/notify-rust#169) so we can get rid of
our custom implementation.

Cargo.toml

@@ -23,7 +23,7 @@ edition = "2018"
 [dependencies]
 alpm = "2.1.0"
 clap = { version = "4.1.1", features = ["help", "usage", "error-context", "std", "derive"], default-features = false }
-notify-rust = "4.5"
+notify-rust = "4.7"
 utmp-rs = "0.3.0"
 time = "0.3.14"
 log = "0.4.17"

src/main.rs

@@ -1,5 +1,3 @@
-use std::{num::ParseIntError, str::FromStr};
-
 use clap::Parser;
 use log::error;
 use notify_rust::{Notification, Timeout};
@@ -15,14 +13,6 @@ mod critical_packages_check;
 use critical_packages_check::CriticalPackagesCheck;
 mod session;
 
-fn timeout_from_str(s: &str) -> Result<Timeout, ParseIntError> {
-    match s {
-        "default" => Ok(Timeout::Default),
-        "never" => Ok(Timeout::Never),
-        milliseconds => Ok(Timeout::Milliseconds(u32::from_str(milliseconds)?)),
-    }
-}
-
 #[derive(Debug, Parser)]
 #[clap(
     version,
@@ -40,7 +30,7 @@ struct Args {
     /// * "never" or "0" will cause the notification never to expire.
     ///
     /// * Any other number will be interpreted as the timeout in milliseconds.
-    #[clap(long, value_parser(timeout_from_str), default_value = "default")]
+    #[clap(long, default_value = "default")]
     notification_timeout: Timeout,
 
     /// Comma separated list of packages were we should reboot after an upgrade.