* [devel] Правила для udev/rules.d
@ 2012-03-14 15:07 Dubrovskiy Viacheslav
2012-03-14 15:34 ` Michael Shigorin
0 siblings, 1 reply; 4+ messages in thread
From: Dubrovskiy Viacheslav @ 2012-03-14 15:07 UTC (permalink / raw)
To: Devel
[-- 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 --]
^ permalink raw reply [flat|nested] 4+ messages in thread
* Re: [devel] Правила для udev/rules.d 2012-03-14 15:07 [devel] Правила для udev/rules.d Dubrovskiy Viacheslav @ 2012-03-14 15:34 ` Michael Shigorin 2012-03-15 9:31 ` Ildar Mulyukov 0 siblings, 1 reply; 4+ messages in thread From: Michael Shigorin @ 2012-03-14 15:34 UTC (permalink / raw) To: Devel 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/ ^ permalink raw reply [flat|nested] 4+ messages in thread
* Re: [devel] Правила для udev/rules.d 2012-03-14 15:34 ` Michael Shigorin @ 2012-03-15 9:31 ` Ildar Mulyukov 2012-03-16 2:37 ` Dubrovskiy Viacheslav 0 siblings, 1 reply; 4+ messages in thread From: Ildar Mulyukov @ 2012-03-15 9:31 UTC (permalink / raw) To: devel [-- 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" ^ permalink raw reply [flat|nested] 4+ messages in thread
* Re: [devel] Правила для udev/rules.d 2012-03-15 9:31 ` Ildar Mulyukov @ 2012-03-16 2:37 ` Dubrovskiy Viacheslav 0 siblings, 0 replies; 4+ messages in thread From: Dubrovskiy Viacheslav @ 2012-03-16 2:37 UTC (permalink / raw) To: ALT Linux Team development discussions [-- 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 --] ^ permalink raw reply [flat|nested] 4+ messages in thread
end of thread, other threads:[~2012-03-16 2:37 UTC | newest] Thread overview: 4+ messages (download: mbox.gz / follow: Atom feed) -- links below jump to the message on this page -- 2012-03-14 15:07 [devel] Правила для udev/rules.d Dubrovskiy Viacheslav 2012-03-14 15:34 ` Michael Shigorin 2012-03-15 9:31 ` Ildar Mulyukov 2012-03-16 2:37 ` Dubrovskiy Viacheslav
ALT Linux Team development discussions This inbox may be cloned and mirrored by anyone: git clone --mirror http://lore.altlinux.org/devel/0 devel/git/0.git # If you have public-inbox 1.1+ installed, you may # initialize and index your mirror using the following commands: public-inbox-init -V2 devel devel/ http://lore.altlinux.org/devel \ devel@altlinux.org devel@altlinux.ru devel@lists.altlinux.org devel@lists.altlinux.ru devel@linux.iplabs.ru mandrake-russian@linuxteam.iplabs.ru sisyphus@linuxteam.iplabs.ru public-inbox-index devel Example config snippet for mirrors. Newsgroup available over NNTP: nntp://lore.altlinux.org/org.altlinux.lists.devel AGPL code for this site: git clone https://public-inbox.org/public-inbox.git