ALT Linux Team development discussions
 help / color / mirror / Atom feed
* Re: [devel] [#317085] TESTED sisyphus_check.git=0.8.67-alt1
  @ 2023-03-21  8:53 ` Dmitry V. Levin
  2023-03-21 10:56   ` Alexey Gladkov
                     ` (2 more replies)
  0 siblings, 3 replies; 31+ messages in thread
From: Dmitry V. Levin @ 2023-03-21  8:53 UTC (permalink / raw)
  To: ALT Devel discussion list

On Tue, Mar 21, 2023 at 12:46:47AM +0000, Girar awaiter (ldv) wrote:
> https://git.altlinux.org/tasks/317085/logs/events.1.1.log
[...]
> #100 sisyphus_check 0.8.66-alt1 -> 0.8.67-alt1
>  Sat Mar 18 2023 Dmitry V. Levin <ldv@altlinux> 0.8.67-alt1
>  - 185-check-udev: new check, disallows packages containing
>    system udev rules files outside /lib/udev/rules.d/.

$ cat Sisyphus/{noarch,x86_64}/base/contents_index |\
  grep '/udev/rules\.d/.*\.rules[[:space:]]' |\
  grep -Ev '^/(lib|usr/share/.*)/udev/rules\.d/' |\
  sort -k2,2 |\
  join -t$'\t' -12 -21 -o1.1,1.2,2.2 - acl/list.packages.sisyphus |\
  sort  
/etc/udev/rules.d/10-wacom.rules	xorg-drv-wacom	shrek
/etc/udev/rules.d/20-radio.rules	gkrellm-radio	@nobody
/etc/udev/rules.d/50-voiceman-media-ru.rules	voiceman-media-ru	msp @everybody
/etc/udev/rules.d/50-voiceman-media.rules	voiceman-media	msp @qa @everybody
/etc/udev/rules.d/55-ovirt-guest-agent.rules	ovirt-guest-agent	cas @everybody
/etc/udev/rules.d/70-libfprint-2.rules	libfprint2	rider @everybody
/etc/udev/rules.d/70-persistent-ipoib.rules	rdma-core	shaba @everybody
/etc/udev/rules.d/70-touchscreen-egalax.rules	twofing	@nobody
/etc/udev/rules.d/74-hp4600-scan.rules	hp4600-scan	majioa @everybody
/etc/udev/rules.d/80-docker.rules	docker-engine	cow @everybody
/etc/udev/rules.d/88-ltsp.rules	lbuscd	mike @qa
/etc/udev/rules.d/90-tun.rules	tunctl	@nobody
/etc/udev/rules.d/98-gpib-generic.rules	linux-gpib	slazav @everybody
/etc/udev/rules.d/99-agilent_82357a.rules	linux-gpib	slazav @everybody
/etc/udev/rules.d/99-alt-udisks2-media-mount-point.rules	udisks2	aris
/etc/udev/rules.d/99-lpvo_usb_gpib.rules	linux-gpib	slazav @everybody
/etc/udev/rules.d/99-ni_usb_gpib.rules	linux-gpib	slazav @everybody
/etc/udev/rules.d/99-nst.rules	nspec	alexvm @everybody
/etc/udev/rules.d/99-perfctr.rules	perfctr	@nobody
/etc/udev/rules.d/99-qdmr.rules	qdmr	asheplyakov @everybody
/etc/udev/rules.d/99-uuu.rules	mfgtools	zorg @everybody
/etc/udev/rules.d/isight.rules	isight-firmware-tools	@nobody
/etc/udev/rules.d/kvm-on-demand.rules	kvm-on-demand	led @everybody
/etc/udev/rules.d/z65-anyma-udmx.rules	qlcplus	grenka @everybody
/etc/udev/rules.d/z65-dmxusb.rules	qlcplus	grenka @everybody
/etc/udev/rules.d/z65-fx5-hid.rules	qlcplus	grenka @everybody
/etc/udev/rules.d/z65-peperoni.rules	qlcplus	grenka @everybody
/etc/udev/rules.d/z65-spi.rules	qlcplus	grenka @everybody
/usr/lib/udev/rules.d/42-logitech-unify-permissions.rules	solaar	cas @everybody
/usr/lib64/udev/rules.d/99-neard.rules	neard	mike @everybody


-- 
ldv


^ permalink raw reply	[flat|nested] 31+ messages in thread

* Re: [devel] [#317085] TESTED sisyphus_check.git=0.8.67-alt1
  2023-03-21  8:53 ` [devel] [#317085] TESTED sisyphus_check.git=0.8.67-alt1 Dmitry V. Levin
@ 2023-03-21 10:56   ` Alexey Gladkov
  2023-03-21 11:01     ` [devel] bugs in udev rules Dmitry V. Levin
  2023-04-27  9:12   ` [devel] [#317085] TESTED sisyphus_check.git=0.8.67-alt1 Sergey Afonin
  2023-07-04 17:53   ` [devel] Запрет и других конфигов в /etc " Alexey Shabalin
  2 siblings, 1 reply; 31+ messages in thread
From: Alexey Gladkov @ 2023-03-21 10:56 UTC (permalink / raw)
  To: ALT Linux Team development discussions

On Tue, Mar 21, 2023 at 10:53:50AM +0200, Dmitry V. Levin wrote:
> On Tue, Mar 21, 2023 at 12:46:47AM +0000, Girar awaiter (ldv) wrote:
> > https://git.altlinux.org/tasks/317085/logs/events.1.1.log
> [...]
> > #100 sisyphus_check 0.8.66-alt1 -> 0.8.67-alt1
> >  Sat Mar 18 2023 Dmitry V. Levin <ldv@altlinux> 0.8.67-alt1
> >  - 185-check-udev: new check, disallows packages containing
> >    system udev rules files outside /lib/udev/rules.d/.
> 
> $ cat Sisyphus/{noarch,x86_64}/base/contents_index |\
>   grep '/udev/rules\.d/.*\.rules[[:space:]]' |\
>   grep -Ev '^/(lib|usr/share/.*)/udev/rules\.d/' |\
>   sort -k2,2 |\
>   join -t$'\t' -12 -21 -o1.1,1.2,2.2 - acl/list.packages.sisyphus |\
>   sort  

Может в правилах ещё и ошибки есть ? :)

> /etc/udev/rules.d/10-wacom.rules	xorg-drv-wacom	shrek
> /etc/udev/rules.d/20-radio.rules	gkrellm-radio	@nobody
> /etc/udev/rules.d/50-voiceman-media-ru.rules	voiceman-media-ru	msp @everybody
> /etc/udev/rules.d/50-voiceman-media.rules	voiceman-media	msp @qa @everybody
> /etc/udev/rules.d/55-ovirt-guest-agent.rules	ovirt-guest-agent	cas @everybody
> /etc/udev/rules.d/70-libfprint-2.rules	libfprint2	rider @everybody
> /etc/udev/rules.d/70-persistent-ipoib.rules	rdma-core	shaba @everybody
> /etc/udev/rules.d/70-touchscreen-egalax.rules	twofing	@nobody
> /etc/udev/rules.d/74-hp4600-scan.rules	hp4600-scan	majioa @everybody
> /etc/udev/rules.d/80-docker.rules	docker-engine	cow @everybody
> /etc/udev/rules.d/88-ltsp.rules	lbuscd	mike @qa
> /etc/udev/rules.d/90-tun.rules	tunctl	@nobody
> /etc/udev/rules.d/98-gpib-generic.rules	linux-gpib	slazav @everybody
> /etc/udev/rules.d/99-agilent_82357a.rules	linux-gpib	slazav @everybody
> /etc/udev/rules.d/99-alt-udisks2-media-mount-point.rules	udisks2	aris
> /etc/udev/rules.d/99-lpvo_usb_gpib.rules	linux-gpib	slazav @everybody
> /etc/udev/rules.d/99-ni_usb_gpib.rules	linux-gpib	slazav @everybody
> /etc/udev/rules.d/99-nst.rules	nspec	alexvm @everybody
> /etc/udev/rules.d/99-perfctr.rules	perfctr	@nobody
> /etc/udev/rules.d/99-qdmr.rules	qdmr	asheplyakov @everybody
> /etc/udev/rules.d/99-uuu.rules	mfgtools	zorg @everybody
> /etc/udev/rules.d/isight.rules	isight-firmware-tools	@nobody
> /etc/udev/rules.d/kvm-on-demand.rules	kvm-on-demand	led @everybody
> /etc/udev/rules.d/z65-anyma-udmx.rules	qlcplus	grenka @everybody
> /etc/udev/rules.d/z65-dmxusb.rules	qlcplus	grenka @everybody
> /etc/udev/rules.d/z65-fx5-hid.rules	qlcplus	grenka @everybody
> /etc/udev/rules.d/z65-peperoni.rules	qlcplus	grenka @everybody
> /etc/udev/rules.d/z65-spi.rules	qlcplus	grenka @everybody
> /usr/lib/udev/rules.d/42-logitech-unify-permissions.rules	solaar	cas @everybody
> /usr/lib64/udev/rules.d/99-neard.rules	neard	mike @everybody
> 
> 
> -- 
> ldv
> _______________________________________________
> Devel mailing list
> Devel@lists.altlinux.org
> https://lists.altlinux.org/mailman/listinfo/devel

-- 
Rgrds, legion



^ permalink raw reply	[flat|nested] 31+ messages in thread

* Re: [devel] bugs in udev rules
  2023-03-21 10:56   ` Alexey Gladkov
@ 2023-03-21 11:01     ` Dmitry V. Levin
  0 siblings, 0 replies; 31+ messages in thread
From: Dmitry V. Levin @ 2023-03-21 11:01 UTC (permalink / raw)
  To: ALT Devel discussion list

On Tue, Mar 21, 2023 at 11:56:27AM +0100, Alexey Gladkov wrote:
> On Tue, Mar 21, 2023 at 10:53:50AM +0200, Dmitry V. Levin wrote:
> > On Tue, Mar 21, 2023 at 12:46:47AM +0000, Girar awaiter (ldv) wrote:
> > > https://git.altlinux.org/tasks/317085/logs/events.1.1.log
> > [...]
> > > #100 sisyphus_check 0.8.66-alt1 -> 0.8.67-alt1
> > >  Sat Mar 18 2023 Dmitry V. Levin <ldv@altlinux> 0.8.67-alt1
> > >  - 185-check-udev: new check, disallows packages containing
> > >    system udev rules files outside /lib/udev/rules.d/.
> > 
> > $ cat Sisyphus/{noarch,x86_64}/base/contents_index |\
> >   grep '/udev/rules\.d/.*\.rules[[:space:]]' |\
> >   grep -Ev '^/(lib|usr/share/.*)/udev/rules\.d/' |\
> >   sort -k2,2 |\
> >   join -t$'\t' -12 -21 -o1.1,1.2,2.2 - acl/list.packages.sisyphus |\
> >   sort  
> 
> Может в правилах ещё и ошибки есть ? :)

Увы, в самих правилах тоже есть ошибки:

$ udevadm verify -N never $(find -type f -path '*/udev/rules.d/*.rules')
./etc/udev/rules.d/z65-peperoni.rules:7 Invalid key 'SYSFS'
./etc/udev/rules.d/z65-peperoni.rules: udev rules check failed
./etc/udev/rules.d/z65-anyma-udmx.rules:7 Invalid key 'SYSFS'
./etc/udev/rules.d/z65-anyma-udmx.rules:10 Invalid key 'SYSFS'
./etc/udev/rules.d/z65-anyma-udmx.rules: udev rules check failed
./etc/udev/rules.d/utsushi-esci.rules:25 LABEL="utsushi_esci_rules_begin" is unused.
./etc/udev/rules.d/utsushi-esci.rules: udev rules check failed
./lib/udev/rules.d/60-persistent-storage.rules:139 duplicate expressions
./lib/udev/rules.d/60-persistent-storage.rules: udev rules check failed
./lib/udev/rules.d/quasarm.rules:2 Ignoring NAME="%k", as it will take no effect.
./lib/udev/rules.d/quasarm.rules: udev rules check failed
./lib/udev/rules.d/quasar.rules:2 Ignoring NAME="%k", as it will take no effect.
./lib/udev/rules.d/quasar.rules: udev rules check failed
Configuration file ./lib/udev/rules.d/60-openct.rules is marked executable. Please remove executable permission bits. Proceeding anyway.
Configuration file ./lib/udev/rules.d/41-odvr.rules is marked executable. Please remove executable permission bits. Proceeding anyway.
./lib/udev/rules.d/41-odvr.rules:1 Invalid key 'SYSFS'
./lib/udev/rules.d/41-odvr.rules: udev rules check failed
./lib/udev/rules.d/40-nitrokey.rules:9 GROUP key takes '=' or ':=' operator, assuming '='.
./lib/udev/rules.d/40-nitrokey.rules:11 GROUP key takes '=' or ':=' operator, assuming '='.
./lib/udev/rules.d/40-nitrokey.rules:13 GROUP key takes '=' or ':=' operator, assuming '='.
./lib/udev/rules.d/40-nitrokey.rules:15 GROUP key takes '=' or ':=' operator, assuming '='.
./lib/udev/rules.d/40-nitrokey.rules:17 GROUP key takes '=' or ':=' operator, assuming '='.
./lib/udev/rules.d/40-nitrokey.rules: udev rules check failed
Configuration file ./lib/udev/rules.d/00-misdn.rules is marked executable. Please remove executable permission bits. Proceeding anyway.
./lib/udev/rules.d/40-lomoco.rules:3 Invalid key 'SYSFS'
./lib/udev/rules.d/40-lomoco.rules:6 Invalid key 'SYSFS'
./lib/udev/rules.d/40-lomoco.rules:8 Invalid key 'SYSFS'
./lib/udev/rules.d/40-lomoco.rules:10 Invalid key 'SYSFS'
./lib/udev/rules.d/40-lomoco.rules:12 Invalid key 'SYSFS'
./lib/udev/rules.d/40-lomoco.rules:14 Invalid key 'SYSFS'
./lib/udev/rules.d/40-lomoco.rules:16 Invalid key 'SYSFS'
./lib/udev/rules.d/40-lomoco.rules:18 Invalid key 'SYSFS'
./lib/udev/rules.d/40-lomoco.rules:20 Invalid key 'SYSFS'
./lib/udev/rules.d/40-lomoco.rules:22 Invalid key 'SYSFS'
./lib/udev/rules.d/40-lomoco.rules:24 Invalid key 'SYSFS'
./lib/udev/rules.d/40-lomoco.rules:26 Invalid key 'SYSFS'
./lib/udev/rules.d/40-lomoco.rules:28 Invalid key 'SYSFS'
./lib/udev/rules.d/40-lomoco.rules:30 Invalid key 'SYSFS'
./lib/udev/rules.d/40-lomoco.rules:32 Invalid key 'SYSFS'
./lib/udev/rules.d/40-lomoco.rules:34 Invalid key 'SYSFS'
./lib/udev/rules.d/40-lomoco.rules:36 Invalid key 'SYSFS'
./lib/udev/rules.d/40-lomoco.rules:38 Invalid key 'SYSFS'
./lib/udev/rules.d/40-lomoco.rules:40 Invalid key 'SYSFS'
./lib/udev/rules.d/40-lomoco.rules:42 Invalid key 'SYSFS'
./lib/udev/rules.d/40-lomoco.rules:44 Invalid key 'SYSFS'
./lib/udev/rules.d/40-lomoco.rules:46 Invalid key 'SYSFS'
./lib/udev/rules.d/40-lomoco.rules: udev rules check failed
./lib/udev/rules.d/99-indi_auxiliary.rules:5 Invalid value "/bin/sh -c 'test -f /sys/module/usbcore/parameters/usbfs_memory_mb && test $(cat /sys/module/usbcore/parameters/usbfs_memory_mb) -lt 256 && echo 256 > /sys/module/usbcore/parameters/usbfs_memory_mb'" for RUN (char 76: invalid substitution type), ignoring.
./lib/udev/rules.d/99-indi_auxiliary.rules: udev rules check failed
./lib/udev/rules.d/60-epsonscan2.rules:23 LABEL="epsonscan2_esci_rules_begin" is unused.
./lib/udev/rules.d/60-epsonscan2.rules: udev rules check failed
./lib/udev/rules.d/cmotech.rules:4 Invalid key 'SYSFS'
./lib/udev/rules.d/cmotech.rules: udev rules check failed
./lib/udev/rules.d/60-aoe.rules:14 Ignoring NAME="%k", as it will take no effect.
./lib/udev/rules.d/60-aoe.rules: udev rules check failed
./lib/udev/rules.d/51-android.rules:28 LABEL="android_usb_rules_begin" is unused.
./lib/udev/rules.d/51-android.rules: udev rules check failed
./lib/udev/rules.d/61-gdm.rules:96 LABEL="gdm_virt_passthrough_check" is unused.
./lib/udev/rules.d/61-gdm.rules:104 LABEL="gdm_virt_multi_gpu_check" is unused.
./lib/udev/rules.d/61-gdm.rules: udev rules check failed

344 udev rules files have been checked.
  Success: 329
  Fail:    15


-- 
ldv


^ permalink raw reply	[flat|nested] 31+ messages in thread

* Re: [devel] [#317085] TESTED sisyphus_check.git=0.8.67-alt1
  2023-03-21  8:53 ` [devel] [#317085] TESTED sisyphus_check.git=0.8.67-alt1 Dmitry V. Levin
  2023-03-21 10:56   ` Alexey Gladkov
@ 2023-04-27  9:12   ` Sergey Afonin
  2023-04-27  9:14     ` Sergey Afonin
  2023-07-04 17:53   ` [devel] Запрет и других конфигов в /etc " Alexey Shabalin
  2 siblings, 1 reply; 31+ messages in thread
From: Sergey Afonin @ 2023-04-27  9:12 UTC (permalink / raw)
  To: ALT Linux Team development discussions

On Tuesday 21 March 2023, Dmitry V. Levin wrote:

> > #100 sisyphus_check 0.8.66-alt1 -> 0.8.67-alt1

> >  Sat Mar 18 2023 Dmitry V. Levin <ldv@altlinux> 0.8.67-alt1
> >  - 185-check-udev: new check, disallows packages containing
> >    system udev rules files outside /lib/udev/rules.d/.
 
И теперь надо руками лазить в /lib/udev/rules.d/ вместо /etc ?
Это хорошо, если правило предполагается неисправляемым, а если нет?

-- 
С уважением, Сергей Афонин.


^ permalink raw reply	[flat|nested] 31+ messages in thread

* Re: [devel] [#317085] TESTED sisyphus_check.git=0.8.67-alt1
  2023-04-27  9:12   ` [devel] [#317085] TESTED sisyphus_check.git=0.8.67-alt1 Sergey Afonin
@ 2023-04-27  9:14     ` Sergey Afonin
  2023-04-27  9:31       ` Anton Farygin
  0 siblings, 1 reply; 31+ messages in thread
From: Sergey Afonin @ 2023-04-27  9:14 UTC (permalink / raw)
  To: ALT Linux Team development discussions

On Thursday 27 April 2023, Sergey Afonin wrote:

> > >  Sat Mar 18 2023 Dmitry V. Levin <ldv@altlinux> 0.8.67-alt1
> > >  - 185-check-udev: new check, disallows packages containing
> > >    system udev rules files outside /lib/udev/rules.d/.
>  
> И теперь надо руками лазить в /lib/udev/rules.d/ вместо /etc ?
> Это хорошо, если правило предполагается неисправляемым, а если нет?
 
Или это в спеке отключается?

-- 
С уважением, Сергей Афонин.


^ permalink raw reply	[flat|nested] 31+ messages in thread

* Re: [devel] [#317085] TESTED sisyphus_check.git=0.8.67-alt1
  2023-04-27  9:14     ` Sergey Afonin
@ 2023-04-27  9:31       ` Anton Farygin
  2023-04-27  9:43         ` Dmitry V. Levin
  0 siblings, 1 reply; 31+ messages in thread
From: Anton Farygin @ 2023-04-27  9:31 UTC (permalink / raw)
  To: devel

On 27.04.2023 12:14, Sergey Afonin wrote:
> On Thursday 27 April 2023, Sergey Afonin wrote:
>
>>>>   Sat Mar 18 2023 Dmitry V. Levin <ldv@altlinux> 0.8.67-alt1
>>>>   - 185-check-udev: new check, disallows packages containing
>>>>     system udev rules files outside /lib/udev/rules.d/.
>>   
>> И теперь надо руками лазить в /lib/udev/rules.d/ вместо /etc ?
>> Это хорошо, если правило предполагается неисправляемым, а если нет?
>   
> Или это в спеке отключается?
>
кстати, хороший вопрос.

почему нельзя упаковать конфиги ?



^ permalink raw reply	[flat|nested] 31+ messages in thread

* Re: [devel] [#317085] TESTED sisyphus_check.git=0.8.67-alt1
  2023-04-27  9:31       ` Anton Farygin
@ 2023-04-27  9:43         ` Dmitry V. Levin
  2023-04-27 10:12           ` Anton Farygin
  0 siblings, 1 reply; 31+ messages in thread
From: Dmitry V. Levin @ 2023-04-27  9:43 UTC (permalink / raw)
  To: ALT Devel discussion list

On Thu, Apr 27, 2023 at 12:31:30PM +0300, Anton Farygin wrote:
> On 27.04.2023 12:14, Sergey Afonin wrote:
> > On Thursday 27 April 2023, Sergey Afonin wrote:
> >
> >>>>   Sat Mar 18 2023 Dmitry V. Levin <ldv@altlinux> 0.8.67-alt1
> >>>>   - 185-check-udev: new check, disallows packages containing
> >>>>     system udev rules files outside /lib/udev/rules.d/.
> >>   
> >> И теперь надо руками лазить в /lib/udev/rules.d/ вместо /etc ?
> >> Это хорошо, если правило предполагается неисправляемым, а если нет?
> >   
> > Или это в спеке отключается?
> >
> кстати, хороший вопрос.
> 
> почему нельзя упаковать конфиги ?

На мой взгляд, man udev, в разделе "RULES FILES", есть ответы на все эти вопросы.


-- 
ldv


^ permalink raw reply	[flat|nested] 31+ messages in thread

* Re: [devel] [#317085] TESTED sisyphus_check.git=0.8.67-alt1
  2023-04-27  9:43         ` Dmitry V. Levin
@ 2023-04-27 10:12           ` Anton Farygin
  2023-04-27 10:17             ` Dmitry V. Levin
  0 siblings, 1 reply; 31+ messages in thread
From: Anton Farygin @ 2023-04-27 10:12 UTC (permalink / raw)
  To: devel

On 27.04.2023 12:43, Dmitry V. Levin wrote:
> On Thu, Apr 27, 2023 at 12:31:30PM +0300, Anton Farygin wrote:
>> On 27.04.2023 12:14, Sergey Afonin wrote:
>>> On Thursday 27 April 2023, Sergey Afonin wrote:
>>>
>>>>>>    Sat Mar 18 2023 Dmitry V. Levin <ldv@altlinux> 0.8.67-alt1
>>>>>>    - 185-check-udev: new check, disallows packages containing
>>>>>>      system udev rules files outside /lib/udev/rules.d/.
>>>>    
>>>> И теперь надо руками лазить в /lib/udev/rules.d/ вместо /etc ?
>>>> Это хорошо, если правило предполагается неисправляемым, а если нет?
>>>    
>>> Или это в спеке отключается?
>>>
>> кстати, хороший вопрос.
>>
>> почему нельзя упаковать конфиги ?
> На мой взгляд, man udev, в разделе "RULES FILES", есть ответы на все эти вопросы.
>
>
Там нет ответа на вопрос "почему нельзя запаковать конфигурационный файл 
для дальнейшего редактирования"

Я понимаю что можно скопировать из /lib/udev/rules.d файл в /etc, но это 
будет неравнозначно редактированию упакеченного файла, т.к. %config в 
rpm всё-таки добавляет дополнительные возможности по обработке изменений 
в файлах.




^ permalink raw reply	[flat|nested] 31+ messages in thread

* Re: [devel] [#317085] TESTED sisyphus_check.git=0.8.67-alt1
  2023-04-27 10:12           ` Anton Farygin
@ 2023-04-27 10:17             ` Dmitry V. Levin
  2023-04-27 10:24               ` Sergey Afonin
                                 ` (2 more replies)
  0 siblings, 3 replies; 31+ messages in thread
From: Dmitry V. Levin @ 2023-04-27 10:17 UTC (permalink / raw)
  To: ALT Devel discussion list

On Thu, Apr 27, 2023 at 01:12:03PM +0300, Anton Farygin wrote:
> On 27.04.2023 12:43, Dmitry V. Levin wrote:
> > On Thu, Apr 27, 2023 at 12:31:30PM +0300, Anton Farygin wrote:
> >> On 27.04.2023 12:14, Sergey Afonin wrote:
> >>> On Thursday 27 April 2023, Sergey Afonin wrote:
> >>>
> >>>>>>    Sat Mar 18 2023 Dmitry V. Levin <ldv@altlinux> 0.8.67-alt1
> >>>>>>    - 185-check-udev: new check, disallows packages containing
> >>>>>>      system udev rules files outside /lib/udev/rules.d/.
> >>>>    
> >>>> И теперь надо руками лазить в /lib/udev/rules.d/ вместо /etc ?
> >>>> Это хорошо, если правило предполагается неисправляемым, а если нет?
> >>>    
> >>> Или это в спеке отключается?
> >>>
> >> кстати, хороший вопрос.
> >>
> >> почему нельзя упаковать конфиги ?
> > На мой взгляд, man udev, в разделе "RULES FILES", есть ответы на все эти вопросы.
> >
> Там нет ответа на вопрос "почему нельзя запаковать конфигурационный файл 
> для дальнейшего редактирования"

Потому что упаковывать полагается в /lib/udev/rules.d/, а редактировать -
уже в /etc/rules.d/, такая там концепция.

> Я понимаю что можно скопировать из /lib/udev/rules.d файл в /etc, но это 
> будет неравнозначно редактированию упакеченного файла, т.к. %config в 
> rpm всё-таки добавляет дополнительные возможности по обработке изменений 
> в файлах.

Во время обновления пакета сохранить предыдущую или последующую редакцию
файла под другим именем?  Это помогает в случае с udev rules?


-- 
ldv


^ permalink raw reply	[flat|nested] 31+ messages in thread

* Re: [devel] [#317085] TESTED sisyphus_check.git=0.8.67-alt1
  2023-04-27 10:17             ` Dmitry V. Levin
@ 2023-04-27 10:24               ` Sergey Afonin
  2023-05-10 11:55                 ` Sergey Afonin
  2023-04-27 10:26               ` Anton Farygin
  2023-04-27 11:52               ` Alexey Gladkov
  2 siblings, 1 reply; 31+ messages in thread
From: Sergey Afonin @ 2023-04-27 10:24 UTC (permalink / raw)
  To: ALT Linux Team development discussions

On Thursday 27 April 2023, Dmitry V. Levin wrote:

> > Там нет ответа на вопрос "почему нельзя запаковать конфигурационный файл 
> > для дальнейшего редактирования"
> 
> Потому что упаковывать полагается в /lib/udev/rules.d/, а редактировать -
> уже в /etc/rules.d/, такая там концепция.

А что делать с такой концепцией?

%ghost %_sysconfdir/udev/rules.d/70-persistent-cd.rules
%ghost %_sysconfdir/udev/rules.d/70-persistent-net.rules

-- 
С уважением, Сергей Афонин.


^ permalink raw reply	[flat|nested] 31+ messages in thread

* Re: [devel] [#317085] TESTED sisyphus_check.git=0.8.67-alt1
  2023-04-27 10:17             ` Dmitry V. Levin
  2023-04-27 10:24               ` Sergey Afonin
@ 2023-04-27 10:26               ` Anton Farygin
  2023-04-27 10:42                 ` Dmitry V. Levin
  2023-04-27 11:52               ` Alexey Gladkov
  2 siblings, 1 reply; 31+ messages in thread
From: Anton Farygin @ 2023-04-27 10:26 UTC (permalink / raw)
  To: devel

On 27.04.2023 13:17, Dmitry V. Levin wrote:
> On Thu, Apr 27, 2023 at 01:12:03PM +0300, Anton Farygin wrote:
>> On 27.04.2023 12:43, Dmitry V. Levin wrote:
>>> On Thu, Apr 27, 2023 at 12:31:30PM +0300, Anton Farygin wrote:
>>>> On 27.04.2023 12:14, Sergey Afonin wrote:
>>>>> On Thursday 27 April 2023, Sergey Afonin wrote:
>>>>>
>>>>>>>>     Sat Mar 18 2023 Dmitry V. Levin <ldv@altlinux> 0.8.67-alt1
>>>>>>>>     - 185-check-udev: new check, disallows packages containing
>>>>>>>>       system udev rules files outside /lib/udev/rules.d/.
>>>>>>     
>>>>>> И теперь надо руками лазить в /lib/udev/rules.d/ вместо /etc ?
>>>>>> Это хорошо, если правило предполагается неисправляемым, а если нет?
>>>>>     
>>>>> Или это в спеке отключается?
>>>>>
>>>> кстати, хороший вопрос.
>>>>
>>>> почему нельзя упаковать конфиги ?
>>> На мой взгляд, man udev, в разделе "RULES FILES", есть ответы на все эти вопросы.
>>>
>> Там нет ответа на вопрос "почему нельзя запаковать конфигурационный файл
>> для дальнейшего редактирования"
> Потому что упаковывать полагается в /lib/udev/rules.d/, а редактировать -
> уже в /etc/rules.d/, такая там концепция.

да, это похоже на systemd, но в том случае есть отличная команда 
systemctl edit <имя сервиса>, которая сильно облегчает поиск и 
редактирование.

В данном случае, если я уверен что хочу предоставить пользователю право 
редактирования конфигурационного файла, то его проще положить сразу в 
/etc, а не выкладывать в /lib


Кстати, конфиги в /lib это же тоже ошибка ?

или тоже концепция ?


>
>> Я понимаю что можно скопировать из /lib/udev/rules.d файл в /etc, но это
>> будет неравнозначно редактированию упакеченного файла, т.к. %config в
>> rpm всё-таки добавляет дополнительные возможности по обработке изменений
>> в файлах.
> Во время обновления пакета сохранить предыдущую или последующую редакцию
> файла под другим именем?  Это помогает в случае с udev rules?
>
>
Да, мне это помогало.




^ permalink raw reply	[flat|nested] 31+ messages in thread

* Re: [devel] [#317085] TESTED sisyphus_check.git=0.8.67-alt1
  2023-04-27 10:26               ` Anton Farygin
@ 2023-04-27 10:42                 ` Dmitry V. Levin
  2023-04-27 11:03                   ` Ruslandh
  2023-04-27 11:28                   ` Anton Farygin
  0 siblings, 2 replies; 31+ messages in thread
From: Dmitry V. Levin @ 2023-04-27 10:42 UTC (permalink / raw)
  To: devel

On Thu, Apr 27, 2023 at 01:26:35PM +0300, Anton Farygin wrote:
> On 27.04.2023 13:17, Dmitry V. Levin wrote:
> > On Thu, Apr 27, 2023 at 01:12:03PM +0300, Anton Farygin wrote:
> >> On 27.04.2023 12:43, Dmitry V. Levin wrote:
> >>> On Thu, Apr 27, 2023 at 12:31:30PM +0300, Anton Farygin wrote:
> >>>> On 27.04.2023 12:14, Sergey Afonin wrote:
> >>>>> On Thursday 27 April 2023, Sergey Afonin wrote:
> >>>>>
> >>>>>>>>     Sat Mar 18 2023 Dmitry V. Levin <ldv@altlinux> 0.8.67-alt1
> >>>>>>>>     - 185-check-udev: new check, disallows packages containing
> >>>>>>>>       system udev rules files outside /lib/udev/rules.d/.
> >>>>>>     
> >>>>>> И теперь надо руками лазить в /lib/udev/rules.d/ вместо /etc ?
> >>>>>> Это хорошо, если правило предполагается неисправляемым, а если нет?
> >>>>>     
> >>>>> Или это в спеке отключается?
> >>>>>
> >>>> кстати, хороший вопрос.
> >>>>
> >>>> почему нельзя упаковать конфиги ?
> >>> На мой взгляд, man udev, в разделе "RULES FILES", есть ответы на все эти вопросы.
> >>>
> >> Там нет ответа на вопрос "почему нельзя запаковать конфигурационный файл
> >> для дальнейшего редактирования"
> > Потому что упаковывать полагается в /lib/udev/rules.d/, а редактировать -
> > уже в /etc/rules.d/, такая там концепция.
> 
> да, это похоже на systemd, но в том случае есть отличная команда 
> systemctl edit <имя сервиса>, которая сильно облегчает поиск и 
> редактирование.

Это уже немного другая тема, но можно попросить у них какой-нибудь udevadm edit.

> В данном случае, если я уверен что хочу предоставить пользователю право 
> редактирования конфигурационного файла, то его проще положить сразу в 
> /etc, а не выкладывать в /lib

Мне кажется, что они рассматривают эти файлы не как конфигурационные,
а скорее как скрипты.

> Кстати, конфиги в /lib это же тоже ошибка ?
> 
> или тоже концепция ?

Это выглядит как продолжение той же концепции: в lib прошивка, в etc настройки.

> >> Я понимаю что можно скопировать из /lib/udev/rules.d файл в /etc, но это
> >> будет неравнозначно редактированию упакеченного файла, т.к. %config в
> >> rpm всё-таки добавляет дополнительные возможности по обработке изменений
> >> в файлах.
> > Во время обновления пакета сохранить предыдущую или последующую редакцию
> > файла под другим именем?  Это помогает в случае с udev rules?
> >
> Да, мне это помогало.

Каким образом?


-- 
ldv


^ permalink raw reply	[flat|nested] 31+ messages in thread

* Re: [devel] [#317085] TESTED sisyphus_check.git=0.8.67-alt1
  2023-04-27 10:42                 ` Dmitry V. Levin
@ 2023-04-27 11:03                   ` Ruslandh
  2023-04-27 11:29                     ` Anton Farygin
  2023-04-27 11:28                   ` Anton Farygin
  1 sibling, 1 reply; 31+ messages in thread
From: Ruslandh @ 2023-04-27 11:03 UTC (permalink / raw)
  To: devel


[-- Attachment #1.1.1: Type: text/plain, Size: 662 bytes --]

27.04.2023 13:42, Dmitry V. Levin пишет:
> Это выглядит как продолжение той же концепции: в lib прошивка, в etc настройки.

Правильная концепция, которая позволяет админу вносить свои добавления, 
не боясь, что при обновлении "его" конфиги слетят, как впрочем и 
продолжение её, что конфиги от программы, созданные программой в  /run 
создают конфиги "времяни выполнения программы" ;-)

[-- Attachment #1.1.2: OpenPGP public key --]
[-- Type: application/pgp-keys, Size: 657 bytes --]

[-- Attachment #2: OpenPGP digital signature --]
[-- Type: application/pgp-signature, Size: 236 bytes --]

^ permalink raw reply	[flat|nested] 31+ messages in thread

* Re: [devel] [#317085] TESTED sisyphus_check.git=0.8.67-alt1
  2023-04-27 10:42                 ` Dmitry V. Levin
  2023-04-27 11:03                   ` Ruslandh
@ 2023-04-27 11:28                   ` Anton Farygin
  1 sibling, 0 replies; 31+ messages in thread
From: Anton Farygin @ 2023-04-27 11:28 UTC (permalink / raw)
  To: devel

On 27.04.2023 13:42, Dmitry V. Levin wrote:
> On Thu, Apr 27, 2023 at 01:26:35PM +0300, Anton Farygin wrote:
>> On 27.04.2023 13:17, Dmitry V. Levin wrote:
>>> On Thu, Apr 27, 2023 at 01:12:03PM +0300, Anton Farygin wrote:
>>>> On 27.04.2023 12:43, Dmitry V. Levin wrote:
>>>>> On Thu, Apr 27, 2023 at 12:31:30PM +0300, Anton Farygin wrote:
>>>>>> On 27.04.2023 12:14, Sergey Afonin wrote:
>>>>>>> On Thursday 27 April 2023, Sergey Afonin wrote:
>>>>>>>
>>>>>>>>>>      Sat Mar 18 2023 Dmitry V. Levin <ldv@altlinux> 0.8.67-alt1
>>>>>>>>>>      - 185-check-udev: new check, disallows packages containing
>>>>>>>>>>        system udev rules files outside /lib/udev/rules.d/.
>>>>>>>>      
>>>>>>>> И теперь надо руками лазить в /lib/udev/rules.d/ вместо /etc ?
>>>>>>>> Это хорошо, если правило предполагается неисправляемым, а если нет?
>>>>>>>      
>>>>>>> Или это в спеке отключается?
>>>>>>>
>>>>>> кстати, хороший вопрос.
>>>>>>
>>>>>> почему нельзя упаковать конфиги ?
>>>>> На мой взгляд, man udev, в разделе "RULES FILES", есть ответы на все эти вопросы.
>>>>>
>>>> Там нет ответа на вопрос "почему нельзя запаковать конфигурационный файл
>>>> для дальнейшего редактирования"
>>> Потому что упаковывать полагается в /lib/udev/rules.d/, а редактировать -
>>> уже в /etc/rules.d/, такая там концепция.
>> да, это похоже на systemd, но в том случае есть отличная команда
>> systemctl edit <имя сервиса>, которая сильно облегчает поиск и
>> редактирование.
> Это уже немного другая тема, но можно попросить у них какой-нибудь udevadm edit.

а в случае с udev всё ещё усложняется тем, что непонятно что "edit", в 
отличии от systemd, в котором всё довольно очевидно.

а ещё есть hwdb, который тоже состоит из двух частей - одна в /lib, 
вторая в /etc


>
>> В данном случае, если я уверен что хочу предоставить пользователю право
>> редактирования конфигурационного файла, то его проще положить сразу в
>> /etc, а не выкладывать в /lib
> Мне кажется, что они рассматривают эти файлы не как конфигурационные,
> а скорее как скрипты.
да, но это как раз зависит от содержимого - может быть как скрипт, так и 
простой конфиг.
>
>> Кстати, конфиги в /lib это же тоже ошибка ?
>>
>> или тоже концепция ?
> Это выглядит как продолжение той же концепции: в lib прошивка, в etc настройки.

Я про файлы-конфиги.

find /lib -type f -name '*.conf'

>
>>>> Я понимаю что можно скопировать из /lib/udev/rules.d файл в /etc, но это
>>>> будет неравнозначно редактированию упакеченного файла, т.к. %config в
>>>> rpm всё-таки добавляет дополнительные возможности по обработке изменений
>>>> в файлах.
>>> Во время обновления пакета сохранить предыдущую или последующую редакцию
>>> файла под другим именем?  Это помогает в случае с udev rules?
>>>
>> Да, мне это помогало.
> Каким образом?
>
>
ровно таким - сохранение предыдущей и новой редакции в случае ручного 
редактирования.

В общем я хотел бы сказать, что иногда очень нужно упаковать именно 
конфиг и именно в /etc - держать и распространять конфиги в пакетах 
бывает удобно. И хотелось бы иметь возможность снимать это ограничение 
сборочницы.




^ permalink raw reply	[flat|nested] 31+ messages in thread

* Re: [devel] [#317085] TESTED sisyphus_check.git=0.8.67-alt1
  2023-04-27 11:03                   ` Ruslandh
@ 2023-04-27 11:29                     ` Anton Farygin
  2023-04-27 11:32                       ` Ruslandh
  0 siblings, 1 reply; 31+ messages in thread
From: Anton Farygin @ 2023-04-27 11:29 UTC (permalink / raw)
  To: devel

On 27.04.2023 14:03, Ruslandh wrote:
> 27.04.2023 13:42, Dmitry V. Levin пишет:
>> Это выглядит как продолжение той же концепции: в lib прошивка, в etc 
>> настройки.
>
> Правильная концепция, которая позволяет админу вносить свои 
> добавления, не боясь, что при обновлении "его" конфиги слетят, как 
> впрочем и продолжение её, что конфиги от программы, созданные 
> программой в  /run создают конфиги "времяни выполнения программы" ;-) 

Так они слетят в любом случае, т.к. админ не отслеживает изменений в 
системных /lib конфигах.

Что-то приехало несовместимое с тем, что понаписал админ в /etc и всё, 
приплыли.



^ permalink raw reply	[flat|nested] 31+ messages in thread

* Re: [devel] [#317085] TESTED sisyphus_check.git=0.8.67-alt1
  2023-04-27 11:29                     ` Anton Farygin
@ 2023-04-27 11:32                       ` Ruslandh
  2023-04-27 13:52                         ` Anton Farygin
  0 siblings, 1 reply; 31+ messages in thread
From: Ruslandh @ 2023-04-27 11:32 UTC (permalink / raw)
  To: devel


[-- Attachment #1.1.1: Type: text/plain, Size: 500 bytes --]

27.04.2023 14:29, Anton Farygin пишет:
> Так они слетят в любом случае, т.к. админ не отслеживает изменений в 
> системных /lib конфигах.
> 
> Что-то приехало несовместимое с тем, что понаписал админ в /etc и всё, 
> приплыли.

Не слетит - значения в /etc более приоритетны, чем в /lib.
Хотя риск есть ;-)

[-- Attachment #1.1.2: OpenPGP public key --]
[-- Type: application/pgp-keys, Size: 657 bytes --]

[-- Attachment #2: OpenPGP digital signature --]
[-- Type: application/pgp-signature, Size: 236 bytes --]

^ permalink raw reply	[flat|nested] 31+ messages in thread

* Re: [devel] [#317085] TESTED sisyphus_check.git=0.8.67-alt1
  2023-04-27 10:17             ` Dmitry V. Levin
  2023-04-27 10:24               ` Sergey Afonin
  2023-04-27 10:26               ` Anton Farygin
@ 2023-04-27 11:52               ` Alexey Gladkov
  2 siblings, 0 replies; 31+ messages in thread
From: Alexey Gladkov @ 2023-04-27 11:52 UTC (permalink / raw)
  To: ALT Linux Team development discussions

On Thu, Apr 27, 2023 at 01:17:34PM +0300, Dmitry V. Levin wrote:
> On Thu, Apr 27, 2023 at 01:12:03PM +0300, Anton Farygin wrote:
> > On 27.04.2023 12:43, Dmitry V. Levin wrote:
> > > On Thu, Apr 27, 2023 at 12:31:30PM +0300, Anton Farygin wrote:
> > >> On 27.04.2023 12:14, Sergey Afonin wrote:
> > >>> On Thursday 27 April 2023, Sergey Afonin wrote:
> > >>>
> > >>>>>>    Sat Mar 18 2023 Dmitry V. Levin <ldv@altlinux> 0.8.67-alt1
> > >>>>>>    - 185-check-udev: new check, disallows packages containing
> > >>>>>>      system udev rules files outside /lib/udev/rules.d/.
> > >>>>    
> > >>>> И теперь надо руками лазить в /lib/udev/rules.d/ вместо /etc ?
> > >>>> Это хорошо, если правило предполагается неисправляемым, а если нет?
> > >>>    
> > >>> Или это в спеке отключается?
> > >>>
> > >> кстати, хороший вопрос.
> > >>
> > >> почему нельзя упаковать конфиги ?
> > > На мой взгляд, man udev, в разделе "RULES FILES", есть ответы на все эти вопросы.
> > >
> > Там нет ответа на вопрос "почему нельзя запаковать конфигурационный файл 
> > для дальнейшего редактирования"
> 
> Потому что упаковывать полагается в /lib/udev/rules.d/, а редактировать -
> уже в /etc/rules.d/, такая там концепция.

Правила лучше держать в /lib. В остальных дистрибутивах /lib это симлинк
на usr/lib. В этом смысле /usr это общее read-only пространство, а в /etc
конфигурация конкретной машины. В udev правила рассматриваются как
скрипты, а не конфигурационные файлы.

Запрещать нахождение правил в /etc мне кажется неправильно т.к. под
конкретную машину правила могут быть сгенерированы и у нас есть примеры
такого. Если /usr будет RO, то RW место для правил просто должно быть. И
udev его уже предоставляет.

Но в общем случае правила должны быть в /lib, иначе нам будет сложнее
сделать UsrMerge (планы такие были).

-- 
Rgrds, legion



^ permalink raw reply	[flat|nested] 31+ messages in thread

* Re: [devel] [#317085] TESTED sisyphus_check.git=0.8.67-alt1
  2023-04-27 11:32                       ` Ruslandh
@ 2023-04-27 13:52                         ` Anton Farygin
  2023-04-27 14:13                           ` Vladimir D. Seleznev
  0 siblings, 1 reply; 31+ messages in thread
From: Anton Farygin @ 2023-04-27 13:52 UTC (permalink / raw)
  To: devel

On 27.04.2023 14:32, Ruslandh wrote:
> 27.04.2023 14:29, Anton Farygin пишет:
>> Так они слетят в любом случае, т.к. админ не отслеживает изменений в 
>> системных /lib конфигах.
>>
>> Что-то приехало несовместимое с тем, что понаписал админ в /etc и 
>> всё, приплыли.
>
> Не слетит - значения в /etc более приоритетны, чем в /lib.
> Хотя риск есть ;-) 

Это же не в приоритете вопрос а в конфликте настроек.

Даже если файл переименуется, то тоже получим проблему



^ permalink raw reply	[flat|nested] 31+ messages in thread

* Re: [devel] [#317085] TESTED sisyphus_check.git=0.8.67-alt1
  2023-04-27 13:52                         ` Anton Farygin
@ 2023-04-27 14:13                           ` Vladimir D. Seleznev
  0 siblings, 0 replies; 31+ messages in thread
From: Vladimir D. Seleznev @ 2023-04-27 14:13 UTC (permalink / raw)
  To: ALT Linux Team development discussions

On Thu, Apr 27, 2023 at 04:52:31PM +0300, Anton Farygin wrote:
> On 27.04.2023 14:32, Ruslandh wrote:
> > 27.04.2023 14:29, Anton Farygin пишет:
> >> Так они слетят в любом случае, т.к. админ не отслеживает изменений в 
> >> системных /lib конфигах.
> >>
> >> Что-то приехало несовместимое с тем, что понаписал админ в /etc и 
> >> всё, приплыли.
> >
> > Не слетит - значения в /etc более приоритетны, чем в /lib.
> > Хотя риск есть ;-) 
> 
> Это же не в приоритете вопрос а в конфликте настроек.
> 
> Даже если файл переименуется, то тоже получим проблему

TBF if the config file has been renamed in the package, there is still a
problem.

-- 
   WBR,
   Vladimir D. Seleznev


^ permalink raw reply	[flat|nested] 31+ messages in thread

* Re: [devel] [#317085] TESTED sisyphus_check.git=0.8.67-alt1
  2023-04-27 10:24               ` Sergey Afonin
@ 2023-05-10 11:55                 ` Sergey Afonin
  2023-05-21 18:49                   ` Sergey Y. Afonin
  2023-08-14  8:38                   ` Ivan A. Melnikov
  0 siblings, 2 replies; 31+ messages in thread
From: Sergey Afonin @ 2023-05-10 11:55 UTC (permalink / raw)
  To: devel

On Thursday 27 April 2023, Sergey Afonin wrote:

> > Потому что упаковывать полагается в /lib/udev/rules.d/, а редактировать -
> > уже в /etc/rules.d/, такая там концепция.
> 
> А что делать с такой концепцией?
> 
> %ghost %_sysconfdir/udev/rules.d/70-persistent-cd.rules
> %ghost %_sysconfdir/udev/rules.d/70-persistent-net.rules

Так что, оставлять эти файлы бесхозными после генерации?
В /lib/udev/rules.d/ им делать точно нечего, так как они,
иногда, очень не постоянны. Как минимум net.rules.

-- 
С уважением, Сергей Афонин.


^ permalink raw reply	[flat|nested] 31+ messages in thread

* Re: [devel] [#317085] TESTED sisyphus_check.git=0.8.67-alt1
  2023-05-10 11:55                 ` Sergey Afonin
@ 2023-05-21 18:49                   ` Sergey Y. Afonin
  2023-08-14  8:38                   ` Ivan A. Melnikov
  1 sibling, 0 replies; 31+ messages in thread
From: Sergey Y. Afonin @ 2023-05-21 18:49 UTC (permalink / raw)
  To: devel

On Wednesday 10 May 2023, Sergey Afonin wrote:

> > > Потому что упаковывать полагается в /lib/udev/rules.d/, а редактировать -
> > > уже в /etc/rules.d/, такая там концепция.
> > 
> > А что делать с такой концепцией?
> > 
> > %ghost %_sysconfdir/udev/rules.d/70-persistent-cd.rules
> > %ghost %_sysconfdir/udev/rules.d/70-persistent-net.rules
> 
> Так что, оставлять эти файлы бесхозными после генерации?
> В /lib/udev/rules.d/ им делать точно нечего, так как они,
> иногда, очень не постоянны. Как минимум net.rules.

Так что же делать с этим безобразием в sisyphus_check, которое
мешает указать принадлежность файлов пакету?

-- 
С уважением, Сергей Афонин


^ permalink raw reply	[flat|nested] 31+ messages in thread

* [devel] Запрет и других конфигов в /etc Re:  [#317085] TESTED sisyphus_check.git=0.8.67-alt1
  2023-03-21  8:53 ` [devel] [#317085] TESTED sisyphus_check.git=0.8.67-alt1 Dmitry V. Levin
  2023-03-21 10:56   ` Alexey Gladkov
  2023-04-27  9:12   ` [devel] [#317085] TESTED sisyphus_check.git=0.8.67-alt1 Sergey Afonin
@ 2023-07-04 17:53   ` Alexey Shabalin
  2023-07-04 18:06     ` Alexey Shabalin
  2 siblings, 1 reply; 31+ messages in thread
From: Alexey Shabalin @ 2023-07-04 17:53 UTC (permalink / raw)
  To: ALT Linux Team development discussions, Dmitry V. Levin

вт, 21 мар. 2023 г. в 11:53, Dmitry V. Levin <ldv@altlinux.org>:
>
> On Tue, Mar 21, 2023 at 12:46:47AM +0000, Girar awaiter (ldv) wrote:
> > https://git.altlinux.org/tasks/317085/logs/events.1.1.log
> [...]
> > #100 sisyphus_check 0.8.66-alt1 -> 0.8.67-alt1
> >  Sat Mar 18 2023 Dmitry V. Levin <ldv@altlinux> 0.8.67-alt1
> >  - 185-check-udev: new check, disallows packages containing
> >    system udev rules files outside /lib/udev/rules.d/.

Прошу сделать аналогичный запрет в /etc/ для:
- systemd/system/*.{service,socket}
- tmpfiles.d/*.conf
- sysctl.d/*.conf
- modprobe.d/*.conf
- modules-load.d/*.conf (тут возможно придется сделать исключение для
systemd-modules-common)

-- 
Alexey Shabalin

^ permalink raw reply	[flat|nested] 31+ messages in thread

* Re: [devel] Запрет и других конфигов в /etc Re: [#317085] TESTED sisyphus_check.git=0.8.67-alt1
  2023-07-04 17:53   ` [devel] Запрет и других конфигов в /etc " Alexey Shabalin
@ 2023-07-04 18:06     ` Alexey Shabalin
  0 siblings, 0 replies; 31+ messages in thread
From: Alexey Shabalin @ 2023-07-04 18:06 UTC (permalink / raw)
  To: ALT Linux Team development discussions, Dmitry V. Levin

вт, 4 июл. 2023 г. в 20:54, Alexey Shabalin <a.shabalin@gmail.com>:
>
> вт, 21 мар. 2023 г. в 11:53, Dmitry V. Levin <ldv@altlinux.org>:
> >
> > On Tue, Mar 21, 2023 at 12:46:47AM +0000, Girar awaiter (ldv) wrote:
> > > https://git.altlinux.org/tasks/317085/logs/events.1.1.log
> > [...]
> > > #100 sisyphus_check 0.8.66-alt1 -> 0.8.67-alt1
> > >  Sat Mar 18 2023 Dmitry V. Levin <ldv@altlinux> 0.8.67-alt1
> > >  - 185-check-udev: new check, disallows packages containing
> > >    system udev rules files outside /lib/udev/rules.d/.
>
> Прошу сделать аналогичный запрет в /etc/ для:
> - systemd/system/*.{service,socket}
> - tmpfiles.d/*.conf
> - sysctl.d/*.conf
> - modprobe.d/*.conf

Поправка, modprobe.d надо исключить из этого списка. /etc не умеет
перекрывать значения в /lib.

> - modules-load.d/*.conf (тут возможно придется сделать исключение для
> systemd-modules-common)
>

-- 
Alexey Shabalin

^ permalink raw reply	[flat|nested] 31+ messages in thread

* Re: [devel] [#317085] TESTED sisyphus_check.git=0.8.67-alt1
  2023-05-10 11:55                 ` Sergey Afonin
  2023-05-21 18:49                   ` Sergey Y. Afonin
@ 2023-08-14  8:38                   ` Ivan A. Melnikov
  2023-08-16 11:25                     ` Sergey Afonin
  2023-08-23  8:25                     ` Dmitry V. Levin
  1 sibling, 2 replies; 31+ messages in thread
From: Ivan A. Melnikov @ 2023-08-14  8:38 UTC (permalink / raw)
  To: ALT Linux Team development discussions

On Wed, May 10, 2023 at 03:55:10PM +0400, Sergey Afonin wrote:
> On Thursday 27 April 2023, Sergey Afonin wrote:
> 
> > > Потому что упаковывать полагается в /lib/udev/rules.d/, а редактировать -
> > > уже в /etc/rules.d/, такая там концепция.
> > 
> > А что делать с такой концепцией?
> > 
> > %ghost %_sysconfdir/udev/rules.d/70-persistent-cd.rules
> > %ghost %_sysconfdir/udev/rules.d/70-persistent-net.rules
> 
> Так что, оставлять эти файлы бесхозными после генерации?
> В /lib/udev/rules.d/ им делать точно нечего, так как они,
> иногда, очень не постоянны. Как минимум net.rules.

Присоединяюсь к вопросу и напоминаю о нём. Как правильно
поступить в данном случае?

-- 
  wbr,
    iv m.


^ permalink raw reply	[flat|nested] 31+ messages in thread

* Re: [devel] [#317085] TESTED sisyphus_check.git=0.8.67-alt1
  2023-08-14  8:38                   ` Ivan A. Melnikov
@ 2023-08-16 11:25                     ` Sergey Afonin
  2023-08-23  8:25                     ` Dmitry V. Levin
  1 sibling, 0 replies; 31+ messages in thread
From: Sergey Afonin @ 2023-08-16 11:25 UTC (permalink / raw)
  To: ALT Linux Team development discussions

On Monday 14 August 2023, Ivan A. Melnikov wrote:

> > > %ghost %_sysconfdir/udev/rules.d/70-persistent-cd.rules
> > > %ghost %_sysconfdir/udev/rules.d/70-persistent-net.rules
> > 
> > Так что, оставлять эти файлы бесхозными после генерации?
> > В /lib/udev/rules.d/ им делать точно нечего, так как они,
> > иногда, очень не постоянны. Как минимум net.rules.
> 
> Присоединяюсь к вопросу и напоминаю о нём. Как правильно
> поступить в данном случае?
 
https://bugzilla.altlinux.org/47262

-- 
С уважением, Сергей Афонин.


^ permalink raw reply	[flat|nested] 31+ messages in thread

* Re: [devel] [#317085] TESTED sisyphus_check.git=0.8.67-alt1
  2023-08-14  8:38                   ` Ivan A. Melnikov
  2023-08-16 11:25                     ` Sergey Afonin
@ 2023-08-23  8:25                     ` Dmitry V. Levin
  2023-08-23  9:04                       ` Sergey Afonin
  1 sibling, 1 reply; 31+ messages in thread
From: Dmitry V. Levin @ 2023-08-23  8:25 UTC (permalink / raw)
  To: ALT Devel discussion list

On Mon, Aug 14, 2023 at 12:38:45PM +0400, Ivan A. Melnikov wrote:
> On Wed, May 10, 2023 at 03:55:10PM +0400, Sergey Afonin wrote:
> > On Thursday 27 April 2023, Sergey Afonin wrote:
> > 
> > > > Потому что упаковывать полагается в /lib/udev/rules.d/, а редактировать -
> > > > уже в /etc/rules.d/, такая там концепция.
> > > 
> > > А что делать с такой концепцией?
> > > 
> > > %ghost %_sysconfdir/udev/rules.d/70-persistent-cd.rules
> > > %ghost %_sysconfdir/udev/rules.d/70-persistent-net.rules
> > 
> > Так что, оставлять эти файлы бесхозными после генерации?
> > В /lib/udev/rules.d/ им делать точно нечего, так как они,
> > иногда, очень не постоянны. Как минимум net.rules.
> 
> Присоединяюсь к вопросу и напоминаю о нём. Как правильно
> поступить в данном случае?

Может быть, разрешить %ghost files в /etc/udev/rules.d/?


-- 
ldv


^ permalink raw reply	[flat|nested] 31+ messages in thread

* Re: [devel] [#317085] TESTED sisyphus_check.git=0.8.67-alt1
  2023-08-23  8:25                     ` Dmitry V. Levin
@ 2023-08-23  9:04                       ` Sergey Afonin
  2023-08-23  9:07                         ` Dmitry V. Levin
  0 siblings, 1 reply; 31+ messages in thread
From: Sergey Afonin @ 2023-08-23  9:04 UTC (permalink / raw)
  To: ALT Linux Team development discussions

On Wednesday 23 August 2023, Dmitry V. Levin wrote:

> > > Так что, оставлять эти файлы бесхозными после генерации?
> > > В /lib/udev/rules.d/ им делать точно нечего, так как они,
> > > иногда, очень не постоянны. Как минимум net.rules.
> > 
> > Присоединяюсь к вопросу и напоминаю о нём. Как правильно
> > поступить в данном случае?
> 
> Может быть, разрешить %ghost files в /etc/udev/rules.d/?
 
Это точно надо, но тут вдруг вспомнилось на примере всё того
же persistent-net.rules, что ещё могут быть заглушки вида 

ln -s /dev/null /etc/udev/rules.d/80-net-setup-link.rules

Так что одними %ghost обойтись сложно. С /dev/null вариант
тоже, как бы, явный, но вдруг где надо что-то более материальное.

-- 
С уважением, Сергей Афонин.


^ permalink raw reply	[flat|nested] 31+ messages in thread

* Re: [devel] [#317085] TESTED sisyphus_check.git=0.8.67-alt1
  2023-08-23  9:04                       ` Sergey Afonin
@ 2023-08-23  9:07                         ` Dmitry V. Levin
  2023-08-23  9:18                           ` Dmitry V. Levin
  2023-08-23 10:02                           ` Sergey Afonin
  0 siblings, 2 replies; 31+ messages in thread
From: Dmitry V. Levin @ 2023-08-23  9:07 UTC (permalink / raw)
  To: ALT Devel discussion list

On Wed, Aug 23, 2023 at 01:04:48PM +0400, Sergey Afonin wrote:
> On Wednesday 23 August 2023, Dmitry V. Levin wrote:
> 
> > > > Так что, оставлять эти файлы бесхозными после генерации?
> > > > В /lib/udev/rules.d/ им делать точно нечего, так как они,
> > > > иногда, очень не постоянны. Как минимум net.rules.
> > > 
> > > Присоединяюсь к вопросу и напоминаю о нём. Как правильно
> > > поступить в данном случае?
> > 
> > Может быть, разрешить %ghost files в /etc/udev/rules.d/?
>  
> Это точно надо, но тут вдруг вспомнилось на примере всё того
> же persistent-net.rules, что ещё могут быть заглушки вида 
> 
> ln -s /dev/null /etc/udev/rules.d/80-net-setup-link.rules

Не совсем понятно, где и почему могут быть упакованы такие файлы.


-- 
ldv


^ permalink raw reply	[flat|nested] 31+ messages in thread

* Re: [devel] [#317085] TESTED sisyphus_check.git=0.8.67-alt1
  2023-08-23  9:07                         ` Dmitry V. Levin
@ 2023-08-23  9:18                           ` Dmitry V. Levin
  2023-08-23 10:04                             ` Sergey Afonin
  2023-08-23 10:02                           ` Sergey Afonin
  1 sibling, 1 reply; 31+ messages in thread
From: Dmitry V. Levin @ 2023-08-23  9:18 UTC (permalink / raw)
  To: ALT Devel discussion list

On Wed, Aug 23, 2023 at 12:07:49PM +0300, Dmitry V. Levin wrote:
> On Wed, Aug 23, 2023 at 01:04:48PM +0400, Sergey Afonin wrote:
> > On Wednesday 23 August 2023, Dmitry V. Levin wrote:
> > 
> > > > > Так что, оставлять эти файлы бесхозными после генерации?
> > > > > В /lib/udev/rules.d/ им делать точно нечего, так как они,
> > > > > иногда, очень не постоянны. Как минимум net.rules.
> > > > 
> > > > Присоединяюсь к вопросу и напоминаю о нём. Как правильно
> > > > поступить в данном случае?
> > > 
> > > Может быть, разрешить %ghost files в /etc/udev/rules.d/?
> >  
> > Это точно надо, но тут вдруг вспомнилось на примере всё того
> > же persistent-net.rules, что ещё могут быть заглушки вида 
> > 
> > ln -s /dev/null /etc/udev/rules.d/80-net-setup-link.rules
> 
> Не совсем понятно, где и почему могут быть упакованы такие файлы.

Точнее говоря, почему такие файлы должны находиться именно в /etc.


-- 
ldv


^ permalink raw reply	[flat|nested] 31+ messages in thread

* Re: [devel] [#317085] TESTED sisyphus_check.git=0.8.67-alt1
  2023-08-23  9:07                         ` Dmitry V. Levin
  2023-08-23  9:18                           ` Dmitry V. Levin
@ 2023-08-23 10:02                           ` Sergey Afonin
  1 sibling, 0 replies; 31+ messages in thread
From: Sergey Afonin @ 2023-08-23 10:02 UTC (permalink / raw)
  To: ALT Linux Team development discussions

On Wednesday 23 August 2023, Dmitry V. Levin wrote:

> > > Может быть, разрешить %ghost files в /etc/udev/rules.d/?
> >  
> > Это точно надо, но тут вдруг вспомнилось на примере всё того
> > же persistent-net.rules, что ещё могут быть заглушки вида 
> > 
> > ln -s /dev/null /etc/udev/rules.d/80-net-setup-link.rules
> 
> Не совсем понятно, где и почему могут быть упакованы такие файлы.

В данном случае для отключения /lib/udev/rules.d/80-net-setup-link.rules.
Добавлено это было, правда, до меня в пакет. Нужно ли это реально, я
не знаю, сейчас попробовал эту заглушку убрать, вроде бы и без неё
тоже работает. На https://wiki.debian.org/NetworkInterfaceNames пишут,
что это действие (симлинк на /dev/null сделать) некий аналог параметра
net.ifnames=0 для ядра.

-- 
С уважением, Сергей Афонин.


^ permalink raw reply	[flat|nested] 31+ messages in thread

* Re: [devel] [#317085] TESTED sisyphus_check.git=0.8.67-alt1
  2023-08-23  9:18                           ` Dmitry V. Levin
@ 2023-08-23 10:04                             ` Sergey Afonin
  0 siblings, 0 replies; 31+ messages in thread
From: Sergey Afonin @ 2023-08-23 10:04 UTC (permalink / raw)
  To: ALT Linux Team development discussions

On Wednesday 23 August 2023, Dmitry V. Levin wrote:

> > > ln -s /dev/null /etc/udev/rules.d/80-net-setup-link.rules
> > 
> > Не совсем понятно, где и почему могут быть упакованы такие файлы.
> 
> Точнее говоря, почему такие файлы должны находиться именно в /etc.
 
А как в данном случае, если 80-net-setup-link.rules есть в
/lib/udev/rules.d, и его надо заблокировать?

-- 
С уважением, Сергей Афонин.


^ permalink raw reply	[flat|nested] 31+ messages in thread

end of thread, other threads:[~2023-08-23 10:04 UTC | newest]

Thread overview: 31+ messages (download: mbox.gz / follow: Atom feed)
-- links below jump to the message on this page --
2023-03-21  8:53 ` [devel] [#317085] TESTED sisyphus_check.git=0.8.67-alt1 Dmitry V. Levin
2023-03-21 10:56   ` Alexey Gladkov
2023-03-21 11:01     ` [devel] bugs in udev rules Dmitry V. Levin
2023-04-27  9:12   ` [devel] [#317085] TESTED sisyphus_check.git=0.8.67-alt1 Sergey Afonin
2023-04-27  9:14     ` Sergey Afonin
2023-04-27  9:31       ` Anton Farygin
2023-04-27  9:43         ` Dmitry V. Levin
2023-04-27 10:12           ` Anton Farygin
2023-04-27 10:17             ` Dmitry V. Levin
2023-04-27 10:24               ` Sergey Afonin
2023-05-10 11:55                 ` Sergey Afonin
2023-05-21 18:49                   ` Sergey Y. Afonin
2023-08-14  8:38                   ` Ivan A. Melnikov
2023-08-16 11:25                     ` Sergey Afonin
2023-08-23  8:25                     ` Dmitry V. Levin
2023-08-23  9:04                       ` Sergey Afonin
2023-08-23  9:07                         ` Dmitry V. Levin
2023-08-23  9:18                           ` Dmitry V. Levin
2023-08-23 10:04                             ` Sergey Afonin
2023-08-23 10:02                           ` Sergey Afonin
2023-04-27 10:26               ` Anton Farygin
2023-04-27 10:42                 ` Dmitry V. Levin
2023-04-27 11:03                   ` Ruslandh
2023-04-27 11:29                     ` Anton Farygin
2023-04-27 11:32                       ` Ruslandh
2023-04-27 13:52                         ` Anton Farygin
2023-04-27 14:13                           ` Vladimir D. Seleznev
2023-04-27 11:28                   ` Anton Farygin
2023-04-27 11:52               ` Alexey Gladkov
2023-07-04 17:53   ` [devel] Запрет и других конфигов в /etc " Alexey Shabalin
2023-07-04 18:06     ` Alexey Shabalin

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