Alexander Galanin <a...@galanin.nnov.ru> wrote:

Hi,

Your patch makes 2 unrelated changes. Could you please clarify which is
the one that makes it work?

> @@ -24,10 +24,10 @@
>  # the sane-devel mailing list (sane-de...@lists.alioth.debian.org).
>  #
>  ACTION!="add", GOTO="libsane_rules_end"
> -ENV{DEVTYPE}=="usb_device", GOTO="libsane_create_usb_dev"
> +ENV{DEVTYPE}=="usb_interface", GOTO="libsane_create_usb_dev"
>  SUBSYSTEMS=="scsi", GOTO="libsane_scsi_rules_begin"
> -SUBSYSTEM=="usb_device", GOTO="libsane_usb_rules_begin"
> -SUBSYSTEM!="usb_device", GOTO="libsane_usb_rules_end"
> +SUBSYSTEM=="usb", GOTO="libsane_usb_rules_begin"
> +SUBSYSTEM!="usb", GOTO="libsane_usb_rules_end"
>  
>  # Kernel >= 2.6.22 jumps here
>  LABEL="libsane_create_usb_dev"

This part may actually be somewhat right, although the usb_interface
part seems wrong.

> @@ -1214,6 +1214,6 @@
>  KERNEL=="sg[0-9]*", ATTRS{type}=="3", ATTRS{vendor}=="HP", 
> ATTRS{model}=="C7670A", ENV{libsane_matched}="yes"
>  LABEL="libsane_scsi_rules_end"
>  
> -ENV{libsane_matched}=="yes", RUN+="/bin/setfacl -m g:scanner:rw 
> $env{DEVNAME}"
> +ENV{libsane_matched}=="yes", GROUP = "scanner"
>  
>  LABEL="libsane_rules_end"

And that is absolutely not right. We've moved away from using GROUP= and
use ACLs instead.

JB

-- 
 Julien BLACHE - Debian & GNU/Linux Developer - <jbla...@debian.org> 
 
 Public key available on <http://www.jblache.org> - KeyID: F5D6 5169 
 GPG Fingerprint : 935A 79F1 C8B3 3521 FD62 7CC7 CD61 4FD7 F5D6 5169 



-- 
To UNSUBSCRIBE, email to debian-bugs-rc-requ...@lists.debian.org
with a subject of "unsubscribe". Trouble? Contact listmas...@lists.debian.org

Reply via email to