libera/#eudev/ Wednesday, 2022-04-06

bb|hcband one more: https://github.com/eudev-project/eudev/pull/22000:30
bb|hcbthis is more complicated than #219, I have kept RUN{builtin}+="keyboard" instead of IMPORT{builtin}="keyboard" - need advise on that00:32
bb|hcbhttps://github.com/eudev-project/eudev/pull/221 - this one should be straight forward to review; I didn't proof read the copied man page though04:33
bb|hcbci fails because of a missing dep...04:34
bb|hcbIn the meantime I did directly push some trivial changes - typo fixes, config file comment header (from Debian) and the missing deps for ci05:05
bb|hcbI also think that this one is good to include: https://git.devuan.org/devuan/eudev/src/branch/master/debian/patches/Always-install-non-arch-specific-files-in-lib-udev.patch05:14
bb|hcband this: https://git.devuan.org/devuan/eudev/src/branch/master/debian/patches/Load-sg-module-from-80-drivers.rules.patch05:15
bb|hcband: https://git.devuan.org/devuan/eudev/src/branch/master/debian/patches/Set-default-polling-interval-on-removable-devices-as.patch05:17
Arsenbb|hcb: wrt 219, what changes did you make compared to upstream?15:02
ArsenRUN{} runs programs, IMPORT{} imports rules15:06
Arsenor rather properties15:07
Arsenhttps://github.com/systemd/systemd/blob/5efbd0bf897a990ebe43d7dc69141d87c404ac9a/rules.d/60-evdev.rules#L7-L8 vs https://github.com/systemd/systemd/blob/07c0e5eeaf3183963f4d680555d2289a33735d6e/rules.d/80-drivers.rules#L515:09
Arsenit's probably right to use import, as long as it runs (and I don't see why it wouldn't from a quick look around)15:12
bb|hcbArsen: Original eudev's rule was using RUN, while the corresponding one from systemd used IMPORT22:39
bb|hcbI have just merged the non-ambiguous parts, that one was unclear for me so I left it as-is22:40
bb|hcbMaybe I need to read the code to understand how RUN{builtin}+="keyboard" vs IMPORT{builtin}="keyboard" work22:59

Generated by irclog2html.py 2.17.0 by Marius Gedminas - find it at https://mg.pov.lt/irclog2html/!