Bug 29425

Summary:	и опять права на устройства
Product:	Sisyphus	Reporter:	Michael Shigorin <mike>
Component:	sane	Assignee:	Vitaly Lipatov <lav>
Status:	CLOSED FIXED	QA Contact:	qa-sisyphus
Severity:	normal
Priority:	P3	CC:	cas, lav, led, mike, nickel
Version:	unstable
Hardware:	all
OS:	Linux

Description Michael Shigorin 2013-10-03 19:17:29 MSK

Как уже обсуждалось в bug #9638 и в рассылках, есть проблемы с правами на файлы устройств, наблюдаю на sane-1.0.24-alt1, udev-208-alt1 и Canon LiDE 25.

Вот проверенный патчик:

--- /etc/udev/rules.d/25-libsane.rules~ 2013-10-02 23:54:17.000000000 +0400
+++ /etc/udev/rules.d/25-libsane.rules  2013-10-03 18:56:03.887212677 +0400
@@ -1981,6 +1981,6 @@
 KERNEL=="sg[0-9]*", ATTRS{type}=="3", ATTRS{vendor}=="HP", ATTRS{model}=="C7670A", MODE="0664", ENV{libsane_matched}="yes"
 LABEL="libsane_scsi_rules_end"
 
-ENV{libsane_matched}=="yes", MODE="664"
+ENV{libsane_matched}=="yes", MODE="664", GROUP="scanner"
 
 LABEL="libsane_rules_end"

По-хорошему, касается и сборки в p7/branch.

Comment 1 Vitaly Lipatov 2013-10-04 13:53:30 MSK

По результатам bug #21808 почему-то был приложен патч к sane, удаляющий GROUP. Убрал исправление, теперь GROUP будет. Отправляю 1.0.24-alt2.
В p7 надо будет портировать после подтверждения снятия проблемы.
Я понял, откуда проблема. У меня сканер через saned работает, который запускается под рутом.

Comment 2 Repository Robot 2013-10-04 15:47:14 MSK

sane-1.0.24-alt2 -> sisyphus:

* Fri Oct 04 2013 Vitaly Lipatov <lav@altlinux> 1.0.24-alt2
- return GROUP=scanner (ALT bug #29425)
- remove obsoleted patches
- add user _saned and run saned under _saned:scanner

Comment 3 Andrey Cherepanov 2013-10-04 15:59:36 MSK

(В ответ на комментарий №1)
> По результатам bug #21808 почему-то был приложен патч к sane, удаляющий GROUP.
Потому что авторизация велась через PolicyKit (а ныне вообще logind) для любых пользователей с активными сеансами. Причём через attr. Как теперь ты будешь печатать на МФУ с таким sane - пока не знаю, хватит ли прав.

Comment 4 Vitaly Lipatov 2013-10-04 16:31:46 MSK

(В ответ на комментарий №3)
> (В ответ на комментарий №1)
> > По результатам bug #21808 почему-то был приложен патч к sane, удаляющий GROUP.
> Потому что авторизация велась через PolicyKit (а ныне вообще logind) для любых
Не знаю ни первого, ни второго.

> пользователей с активными сеансами. Причём через attr. Как теперь ты будешь
Подход с группами всегда был более привычен.

> печатать на МФУ с таким sane - пока не знаю, хватит ли прав.
Печатать, видимо, будет сложновато :)

Comment 5 Andrey Cherepanov 2013-10-04 17:27:42 MSK

(В ответ на комментарий №4)
> > пользователей с активными сеансами. Причём через attr. Как теперь ты будешь
> Подход с группами всегда был более привычен.
Ага. До тех пор, пока понадобиться для устройства назначить более одной группы. Ну не зря же так сделали!
 
> > печатать на МФУ с таким sane - пока не знаю, хватит ли прав.
> Печатать, видимо, будет сложновато :)
Поэтому нужно проверять. Переоткрываю баг.

Comment 6 Michael Shigorin 2013-10-04 18:38:09 MSK

(В ответ на комментарий №5)
> > > пользователей с активными сеансами. Причём через attr.
> > Подход с группами всегда был более привычен.
> Ага. До тех пор, пока понадобиться для устройства назначить более одной группы.
> Ну не зря же так сделали!
А группа мешала?

Comment 7 Andrey Cherepanov 2013-10-04 19:13:43 MSK

(В ответ на комментарий №6)
> А группа мешала?
Сам подумай почему.

Comment 8 Vitaly Lipatov 2013-10-06 20:42:56 MSK

(В ответ на комментарий №5)
...
> Поэтому нужно проверять. Переоткрываю баг.
Только надо перевесить на того, кто будет это делать.

Comment 9 Repository Robot 2014-01-05 17:34:40 MSK

sane-1.0.24-alt2.2 -> p7:

* Sun Jan 05 2014 Andrey Cherepanov <cas@altlinux> 1.0.24-alt2.2
- Add all provides for sane-backends-* for Red Hat compatibility

* Sat Jan 04 2014 Andrey Cherepanov <cas@altlinux> 1.0.24-alt2.1
- Add sane-backends-devel provides for Red Hat compatibility

* Fri Oct 04 2013 Vitaly Lipatov <lav@altlinux> 1.0.24-alt2
- return GROUP=scanner (ALT bug #29425)
- remove obsoleted patches
- add user _saned and run saned under _saned:scanner