udev.rules documentation improvement #326

2021-08-23T12:52:42-05:00

Youw commented

2021-08-23 12:52:42 -05:00

(Migrated from github.com)

As recently discovered (thanks @tresf), the TAG+="uaccess" has to be used
before 73-seat-late.rules and not before 70-uaccess.rules.

Technically it doesn't harm, to have a more restrictive recommenration,
but it was not technically currect.

Refference on uaccess TAG consumption:
1, 2.

Relates to: #276, systemd/systemd/issues/4288.

As recently discovered (thanks @tresf), the `TAG+="uaccess"` has to be used before `73-seat-late.rules` and not before `70-uaccess.rules`. Technically it doesn't harm, to have a more restrictive recommenration, but it was not technically currect. Refference on `uaccess` TAG consumption: [1](https://github.com/systemd/systemd/blob/3a1220eedfb65355c387c29584380742a21917aa/src/login/73-seat-late.rules.in), [2](https://www.apt-browse.org/browse/debian/wheezy/main/amd64/systemd/44-11+deb7u4/file/lib/udev/rules.d/73-seat-late.rules). Relates to: #276, systemd/systemd/issues/4288.

Be-ing (Migrated from github.com) reviewed

2021-08-24 18:48:34 -05:00

udev/69-hid.rules

					
				@ -21,2 +22,4 @@

				#  https://github.com/systemd/systemd/issues/4288#issuecomment-348166161)

				# for example, name the file /etc/udev/rules.d/70-my-application-hid.rules.

				# Then, replug your device or run:

				# sudo udevadm control --reload-rules && sudo udevadm trigger

# for example, name the file /etc/udev/rules.d/72-my-application-hid.rules.

```suggestion # for example, name the file /etc/udev/rules.d/72-my-application-hid.rules. ```

Be-ing commented

2021-08-24 18:49:01 -05:00

(Migrated from github.com)

How about renaming the file to 72-hid.rules?

How about renaming the file to `72-hid.rules`?

Be-ing commented

2021-08-24 18:49:28 -05:00

(Migrated from github.com)

I tested changing the priority of Mixxx's udev rule file and confirmed this is correct.

Youw (Migrated from github.com) reviewed

2021-08-25 05:37:37 -05:00

udev/69-hid.rules

					
				@ -21,2 +22,4 @@

				#  https://github.com/systemd/systemd/issues/4288#issuecomment-348166161)

				# for example, name the file /etc/udev/rules.d/70-my-application-hid.rules.

				# Then, replug your device or run:

				# sudo udevadm control --reload-rules && sudo udevadm trigger

How about renaming the file to 72-hid.rules?

So there is a standard set of rules that adds a uaccess TAG (70-uaccess.rules) with priority 70.
I'm thinking maybe having and example with the same priority would be nit.

> How about renaming the file to 72-hid.rules? So there is a standard set of rules that adds a `uaccess` TAG (`70-uaccess.rules`) with priority `70`. I'm thinking maybe having and example with the same priority would be nit.