Update hint to make new udev rule

New udev rule was updated in probe-rs/webpage#200,
see it for detail info.

Signed-off-by: Celeste Liu <[email protected]>

Author: CoelacanthusHex

changelog/added-linux-setup-hints.md

@@ -2,11 +2,9 @@ Added hints on Linux that warn the user if their setup might be incorrect.
 
 The hints cover the following cases:
 
-- The user has no udev rule file containing the string `probe-rs`.
-- The `plugdev` group is missing 
-- The `plugdev` group is not a system group (ie. the group has id >= 1000) and
-  uses a recent systemd version (ie >= 258).
-  See https://github.com/probe-rs/probe-rs/issues/3566 for more info.
-- The user does not belong to the `plugdev` group.
+- Udev rules was installed.
+- The systemd version is too low to support uaccess mechanism (since systemd v30).
+
+See https://github.com/probe-rs/webpage/pull/200 for more info.
 
 You can disable the hints if by setting the `PROBE_RS_DISABLE_SETUP_HINTS` variable.

probe-rs/src/probe/list.rs

@@ -139,8 +139,7 @@ impl AllProbesLister {
 mod linux {
     use std::process::Command;
 
-    const UDEV_GROUP: &str = "plugdev";
-    const SYSTEMD_STRICT_SYSTEM_GROUP_VERSION: usize = 258;
+    const SYSTEMD_SUPPORT_UACCESS_VERSION: usize = 30;
     const UDEV_RULES_PATH: &str = "/etc/udev/rules.d";
 
     /// Gives the user a hint if they are on Linux.
@@ -168,66 +167,13 @@ mod linux {
 
     /// Prints a helptext if udev user groups seem to be missing or wrong.
     fn help_systemd() {
-        let groups = user_groups();
         let systemd_version = systemd_version();
-        let udev_group_id = udev_group_id();
-
-        let systemd_version_requires_system_group =
-            systemd_version.unwrap_or_default() >= SYSTEMD_STRICT_SYSTEM_GROUP_VERSION;
-
-        match udev_group_id {
-            None => {
-                tracing::warn!("The '{UDEV_GROUP}' group does not exist.");
-                tracing::warn!(
-                    "On how to create it, read more under https://probe.rs/docs/getting-started/probe-setup/"
-                );
-                return;
-            }
-            Some(id) if id >= 1000 && systemd_version_requires_system_group => {
-                tracing::warn!("The '{UDEV_GROUP}' group is not a system group.");
-                tracing::warn!(
-                    "Read more under https://probe.rs/docs/getting-started/probe-setup/"
-                );
-
-                return;
-            }
-            _ => {
-                // We do not have to print a message as the group exists and is a system group (id < 1000).
-            }
-        }
-
-        // Warn if the user does not belong to the right udev group.
-        if !groups.iter().any(|g| g == UDEV_GROUP) {
-            tracing::warn!("The user does not belong to the group '{UDEV_GROUP}'.");
-            tracing::warn!("Read more under https://probe.rs/docs/getting-started/probe-setup/");
-        } else {
-            tracing::warn!("Make sure you have reloaded udev rules after setting everything up");
-            tracing::warn!("Read more under https://probe.rs/docs/getting-started/probe-setup/");
-        }
-    }
 
-    /// Returns the groups assigned to the current user.
-    fn user_groups() -> Vec<String> {
-        let output = match Command::new("id").arg("-Gn").output() {
-            Err(error) => {
-                tracing::debug!("Gathering information about relevant user groups failed: {error}");
-                return Vec::new();
-            }
-            Ok(child) => child,
-        };
-        if !output.status.success() {
-            tracing::debug!(
-                "Gathering information about relevant user groups failed: {:?}",
-                output.status.code()
+        if systemd_version.unwrap_or_default() < SYSTEMD_SUPPORT_UACCESS_VERSION {
+            tracing::warn!(
+                "The systemd on your Linux is older than v30, which doesn't support uaccess mechanism"
             );
-            return Vec::new();
         }
-
-        let stdout = String::from_utf8_lossy(&output.stdout);
-        stdout
-            .split_ascii_whitespace()
-            .map(|g| g.to_owned())
-            .collect()
     }
 
     /// Returns the systemd version of the current system.
@@ -255,28 +201,6 @@ mod linux {
             .and_then(|version| version.parse().ok())
     }
 
-    /// Returns the group id of the group required to list udev devices.
-    fn udev_group_id() -> Option<usize> {
-        let output = match Command::new("getent").arg("group").arg(UDEV_GROUP).output() {
-            Err(error) => {
-                tracing::debug!("Finding the entry for the {UDEV_GROUP} failed: {error}");
-                return None;
-            }
-            Ok(child) => child,
-        };
-
-        if !output.status.success() {
-            tracing::debug!(
-                "Finding the entry for the {UDEV_GROUP} failed: {:?}",
-                output.status.code()
-            );
-            return None;
-        }
-
-        let stdout = String::from_utf8_lossy(&output.stdout);
-        stdout.split(':').nth(2).and_then(|id| id.parse().ok())
-    }
-
     /// Returns true if there is a probe-rs resembling udev rule file.
     fn udev_rule_present() -> bool {
         let mut files = match std::fs::read_dir(UDEV_RULES_PATH) {