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.
 
> If you want sigrok to work consistently and flawlessly with all udev
> versions, you have to use the patch that I posted!

As you don't provide any details nobody can tell to which udev "replacement" 
you are referring to.

The majority of distributions use the regular udev maintained in the systemd 
repository. Your patch breaks permissions for anyone relying on the uaccess 
scheme.
 
> Least but not last the GROUP setting is too distribution-specific, for
> example, "plugdev" I think exists only in Debian, so there is no point in
> distributing such file.

It is distributed in the sources. Distributors can pick the rules file they 
consider appropriate. They can also patch a different group into the file.

It also serves as an example for anyone who wants to grant access permissions 
based on group membership.

So full NAK for this patch.

Regards,

Stefan




_______________________________________________
sigrok-devel mailing list
sigrok-devel@lists.sourceforge.net
https://lists.sourceforge.net/lists/listinfo/sigrok-devel

Reply via email to