On Sat, 2023-05-20 at 01:33 +0300, Dmitry V. Levin wrote: > Fix the following warnings reported by udevadm verify: > > multipath/11-dm-mpath.rules:18 Whitespace after comma is expected. > multipath/11-dm-mpath.rules:73 Whitespace after comma is expected. > multipath/11-dm-mpath.rules:73 Whitespace after comma is expected. > multipath/11-dm-mpath.rules:78 Whitespace after comma is expected. > multipath/11-dm-mpath.rules:78 Whitespace after comma is expected. > multipath/11-dm-mpath.rules:78 Whitespace after comma is expected. > multipath/11-dm-mpath.rules: udev rules check failed > > Signed-off-by: Dmitry V. Levin <ldv@xxxxxxxxx> Maybe you should have mentioned that you've just invented this syntax rule yourself (https://github.com/systemd/systemd/pull/26980). I see no requirement for adding whitespace after a comma in the udev man page. Is this an attempt to change the udev rule syntax retroactively? Furthermore, there is actually whitespace after the comma in the code this patch changes, it just happens to be at the beginning of the following line, which your syntax check ignores. Regards, Martin -- dm-devel mailing list dm-devel@xxxxxxxxxx https://listman.redhat.com/mailman/listinfo/dm-devel
