ALT Linux Team development discussions
 help / color / mirror / Atom feed
From: "Dmitry V. Levin" <ldv@altlinux.org>
To: ALT Devel discussion list <devel@lists.altlinux.org>
Subject: Re: [devel] bugs in udev rules
Date: Tue, 21 Mar 2023 13:01:25 +0200
Message-ID: <20230321110125.GB21408@altlinux.org> (raw)
In-Reply-To: <ZBmNW3ikpnljRux2@example.org>

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


  reply	other threads:[~2023-03-21 11:01 UTC|newest]

Thread overview: 31+ messages / expand[flat|nested]  mbox.gz  Atom feed  top
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     ` Dmitry V. Levin [this message]
2023-04-27  9:12   ` 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

Reply instructions:

You may reply publicly to this message via plain-text email
using any one of the following methods:

* Save the following mbox file, import it into your mail client,
  and reply-to-all from there: mbox

  Avoid top-posting and favor interleaved quoting:
  https://en.wikipedia.org/wiki/Posting_style#Interleaved_style

* Reply using the --to, --cc, and --in-reply-to
  switches of git-send-email(1):

  git send-email \
    --in-reply-to=20230321110125.GB21408@altlinux.org \
    --to=ldv@altlinux.org \
    --cc=devel@lists.altlinux.org \
    /path/to/YOUR_REPLY

  https://kernel.org/pub/software/scm/git/docs/git-send-email.html

* If your mail client supports setting the In-Reply-To header
  via mailto: links, try the mailto: link

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