On Donnerstag, 8. November 2018 22:15:27 CET Guido Trentalancia wrote: > On Thu, 08/11/2018 at 21.01 +0000, Brüns, Stefan wrote: > > On Donnerstag, 8. November 2018 18:45:22 CET Guido Trentalancia > > > > wrote: > > > On Thu, 08/11/2018 at 17.11 +0000, Brüns, Stefan wrote: > > > > On Sonntag, 21. Oktober 2018 20:26:20 CET Guido Trentalancia > > > > > > > > wrote: > > > > > As already explained, the TAG method does not work with all > > > > > udev > > > > > versions!! > > > > > > > > The very first udev version imported into the systemd repository > > > > already > > > > supports TAG - that happened 2012-04-04. > > > > > > The udev project started here: > > > > > > https://www.kernel.org/pub/linux/utils/kernel/hotplug/ > > > > > > not on 2012-04-04 with systemd as you say. > > > > Try reading again what I wrote. > > > > Udev was imported into the systemd repository on 2012-04-04 - 'the > > very first > > version imported into the repository', not 'the very first version > > ever > > released'. > > > > Nobody is interested in supporting ancient versions nobody uses only > > for the > > sake of it. > > You are wrong ! > > How do you know nobody uses udev <= 182 ?? How can you know something > like that, since it is an official release appeared only 6 years ago ??
I only claimed even a 6 year old version supports that. Actually, its at least 8 year, see below. > I do not use systemd, therefore I am using udev released from > kernel.org (<= 182). That certainly proves that you are wrong ! > > And there are probably several other users in the same situation... TAG+= is supported at least in version 154, see https://git.kernel.org/pub/scm/linux/hotplug/udev.git/commit/? h=154&id=1dbfbfbea6d80971fba6bb57469b5c437a8980d6 Is that old enough? You can not even compile a recent libsigrok on systems that old, without heavily modifying it. > > > > > If you want sigrok to work consistently and flawlessly with all > > > > > udev > > > > > versions, you have to use the patch that I posted! > > > > It breaks uaccess support. EOD. > > I don't care, it is the least important patch in the set. > > I had created the patch for my own system and I thought it could be > useful to other users, as simple as that. How to add local rules is documented in libsigroks README.devices. > Also the patch has been provided in the belief that backwards > compatibility is much more important than a minor functionality of very > limited use such as uaccess that you mention. Breaking all systems relying on sane permissions, set via uaccess or based on group membership, is not backwards compatibile. Your patch makes all listed devices world writable. If you need a different permission model, just *add* a sufficient udev rules file to /etc/udev/rules.d/ Stefan _______________________________________________ sigrok-devel mailing list sigrok-devel@lists.sourceforge.net https://lists.sourceforge.net/lists/listinfo/sigrok-devel
