FS#77815 - Systemd 253.1 udev reload unnecessary triggers udev rules

Attached to Project: Arch Linux
Opened by Chris Werner Rau (cwrau) - Saturday, 11 March 2023, 14:58 GMT
Last edited by Toolybird (Toolybird) - Saturday, 11 March 2023, 20:58 GMT
Task Type Bug Report
Category Packages: Core
Status Closed
Assigned To No-one
Architecture x86_64
Severity Medium
Priority Normal
Reported Version
Due in Version Undecided
Due Date Undecided
Percent Complete 100%
Votes 1
Private No

Details

Description:

The following change introduced to the file repos/core-x86_64/systemd-hook in commit 5f99b3117ac9f7a4b32eab950bb002e9d4a9978d;

```diff
diff --git a/repos/core-x86_64/systemd-hook b/repos/core-x86_64/systemd-hook
index 0a4754b2..e8fe0333 100644
--- a/repos/core-x86_64/systemd-hook
+++ b/repos/core-x86_64/systemd-hook
@@ -33,7 +33,7 @@ case "$op" in
tmpfiles) /usr/bin/systemd-tmpfiles --create ;;

daemon-reload) systemd_live; /usr/bin/systemctl daemon-reload ;;
- udev-reload) udevd_live; /usr/bin/udevadm control --reload ;;
+ udev-reload) udevd_live; /usr/bin/udevadm control --reload; /usr/bin/udevadm trigger ;;
binfmt) systemd_live; /usr/lib/systemd/systemd-binfmt ;;
sysctl) systemd_live; /usr/lib/systemd/systemd-sysctl ;;
```

triggers udev rules to execute unnecessarily if any udev rule definition changes.

This leads to _lots_ of weird stuff happening, like my whole desktop basically reloading because `autorandr` restarts stuff and changes display settings, pipewire/pulseaudio resetting devices, ...

Additional info:
* package version(s): 253.1-3
* config and/or log files etc.
* link to upstream bug report, if any

Steps to reproduce:

Update, or reinstall, a package that has udev rules included, like NetworkManager, see your various udev triggered applications run crazy
This task depends upon

Closed by  Toolybird (Toolybird)
Saturday, 11 March 2023, 20:58 GMT
Reason for closing:  Duplicate
Additional comments about closing:   FS#77789 
Comment by loqs (loqs) - Saturday, 11 March 2023, 17:33 GMT
https://bugs.archlinux.org/task/77789#comment216072 explains why the change was made.

Loading...