[devel] Правила для udev/rules.d

[devel] Правила для udev/rules.d

[Dubrovskiy Viacheslav]

[-- Attachment #1: Type: text/plain, Size: 1109 bytes --]

Здравствуйте.


Озадачился тут упаковкой ploop http://wiki.openvz.org/Ploop
В оригинальном спеке есть такой триггер:

%triggerin -- udev
SCRIPT="/lib/udev/rules.d/60-persistent-storage.rules"
if [ -f $SCRIPT ]; then
        fgrep 'KERNEL=="ploop*", GOTO="persistent_storage_end"' $SCRIPT
> /dev/null 2>&1 ||
        sed -i -e '1 s/^/KERNEL=="ploop*",
GOTO="persistent_storage_end"\n/;' $SCRIPT
fi

Меня смущает, что в /lib/udev/rules.d/60-persistent-storage.rules
написано: - "do not edit this file, it will be overwritten on update"
Да и не очень это правильно править файлы от чужого пакета.
Вопрос: Как правильно сделать?

Думаю есть такие варианты:
1. Сделать отдельный файл с аналогичным содержанием в /lib/udev/rules.d/?
2. Сделать отдельный файл с аналогичным содержанием в /etc/udev/rules.d/?


-- 
WBR,
Dubrovskiy Viacheslav


[-- Attachment #2: Криптографическая подпись S/MIME --]
[-- Type: application/pkcs7-signature, Size: 4903 bytes --]

Re: [devel] Правила для udev/rules.d

[Michael Shigorin]

On Wed, Mar 14, 2012 at 11:07:35AM -0400, Dubrovskiy Viacheslav wrote:
> Думаю есть такие варианты:
> 1. Сделать отдельный файл с аналогичным содержанием в /lib/udev/rules.d/?
> 2. Сделать отдельный файл с аналогичным содержанием в /etc/udev/rules.d/?

Если правка этого файла не предполагается, то (1), иначе (2).
Насколько понимаю, (1).

А Кириллу надо рассказать, что /*/udev/rules.d/*.rules согласно
udev README не является API => не следует закладываться на их
существование или вид.  Не говоря уж о том, что такие хаки при
sysv-раскладке конфигурации выглядят просто дико.

-- 
 ---- WBR, Michael Shigorin <mike@altlinux.ru>
  ------ Linux.Kiev http://www.linux.kiev.ua/

Re: [devel] Правила для udev/rules.d

[Ildar Mulyukov]

[-- Attachment #1: Type: text/plain, Size: 1460 bytes --]

On 14.03.2012 21:34:18, Michael Shigorin wrote:
> On Wed, Mar 14, 2012 at 11:07:35AM -0400, Dubrovskiy Viacheslav wrote:
> > Думаю есть такие варианты:
> > 1. Сделать отдельный файл с аналогичным содержанием в  
> /lib/udev/rules.d/?
> > 2. Сделать отдельный файл с аналогичным содержанием в  
> /etc/udev/rules.d/?
> 
> Если правка этого файла не предполагается, то (1), иначе (2).
> Насколько понимаю, (1).
> 
> А Кириллу надо рассказать, что /*/udev/rules.d/*.rules согласно
> udev README не является API => не следует закладываться на их
> существование или вид.  Не говоря уж о том, что такие хаки при
> sysv-раскладке конфигурации выглядят просто дико.

Мнэээ, извините, я предложу третий вариант.
(3) По сути, этот триггер хочет сделать то же, что уже сделано в этом  
файле для других блочных устройств:
# skip rules for inappropriate block devices
KERNEL=="fd*|mtd*|nbd*|gnbd*|btibm*|dm-*|md*",  
GOTO="persistent_storage_end"

Следовательно, нужно, чтобы был приложен простой патч (см. вложение).  
Для того, чтобы патч был правильно приложен, надо просто открыть багу  
на пакет:
$ rpm -qf 60-persistent-storage.rules
udev-rules-172-alt2

Всего хорошего.
-- 
Ildar Mulyukov,
   free SW designer/programmer/packager
=========================================
email: ildar@altlinux.ru
Jabber: ildar.mulyukov@gmail.com
ICQ: 4334029
ALT Linux Sisyphus http://www.sisyphus.ru
=========================================

[-- Attachment #2: 60-persistent-storage.rules.patch --]
[-- Type: text/x-patch, Size: 517 bytes --]

--- 60-persistent-storage.rules-	2011-07-12 00:35:47.000000000 +0600
+++ 60-persistent-storage.rules	2012-03-15 15:27:27.353991577 +0600
@@ -14,7 +14,7 @@
 SUBSYSTEM!="block", GOTO="persistent_storage_end"
 
 # skip rules for inappropriate block devices
-KERNEL=="fd*|mtd*|nbd*|gnbd*|btibm*|dm-*|md*", GOTO="persistent_storage_end"
+KERNEL=="fd*|mtd*|nbd*|gnbd*|btibm*|dm-*|md*|ploop*", GOTO="persistent_storage_end"
 
 # ignore partitions that span the entire disk
 TEST=="whole_disk", GOTO="persistent_storage_end"

Re: [devel] Правила для udev/rules.d

[Dubrovskiy Viacheslav]

[-- Attachment #1: Type: text/plain, Size: 1942 bytes --]

15.03.2012 05:31, Ildar Mulyukov пишет:
> On 14.03.2012 21:34:18, Michael Shigorin wrote:
>> On Wed, Mar 14, 2012 at 11:07:35AM -0400, Dubrovskiy Viacheslav wrote:
>> > Думаю есть такие варианты:
>> > 1. Сделать отдельный файл с аналогичным содержанием в
>> /lib/udev/rules.d/?
>> > 2. Сделать отдельный файл с аналогичным содержанием в
>> /etc/udev/rules.d/?
>>
>> Если правка этого файла не предполагается, то (1), иначе (2).
>> Насколько понимаю, (1).
>>
>> А Кириллу надо рассказать, что /*/udev/rules.d/*.rules согласно
>> udev README не является API => не следует закладываться на их
>> существование или вид.  Не говоря уж о том, что такие хаки при
>> sysv-раскладке конфигурации выглядят просто дико.
>
> Мнэээ, извините, я предложу третий вариант.
> (3) По сути, этот триггер хочет сделать то же, что уже сделано в этом
> файле для других блочных устройств:
> # skip rules for inappropriate block devices
> KERNEL=="fd*|mtd*|nbd*|gnbd*|btibm*|dm-*|md*",
> GOTO="persistent_storage_end"
>
> Следовательно, нужно, чтобы был приложен простой патч (см. вложение).
> Для того, чтобы патч был правильно приложен, надо просто открыть багу
> на пакет:
> $ rpm -qf 60-persistent-storage.rules
> udev-rules-172-alt2
>
> Всего хорошего.
Спасибо за идею. Завел FR
https://bugzilla.altlinux.org/show_bug.cgi?id=27083

-- 
WBR,
Dubrovskiy Viacheslav


[-- Attachment #2: Криптографическая подпись S/MIME --]
[-- Type: application/pkcs7-signature, Size: 4903 bytes --]