<div dir="ltr"><div dir="ltr">On Sat, Nov 16, 2019 at 11:54 AM Michael Biebl <<a href="mailto:mbiebl@gmail.com">mbiebl@gmail.com</a>> wrote:<br></div><div class="gmail_quote"><blockquote class="gmail_quote" style="margin:0px 0px 0px 0.8ex;border-left:1px solid rgb(204,204,204);padding-left:1ex">Am Sa., 16. Nov. 2019 um 09:20 Uhr schrieb Andrei Borzenkov<br>
<<a href="mailto:arvidjaar@gmail.com" target="_blank">arvidjaar@gmail.com</a>>:<br>
><br>
> Likely result of mass-rewrite in<br>
><br>
> commit 25de7aa7b90c23d33ea50ada1e50c5834a414237<br>
> Author: Yu Watanabe <<a href="mailto:watanabe.yu%2Bgithub@gmail.com" target="_blank">watanabe.yu+github@gmail.com</a>><br>
> Date:   Thu Apr 25 01:21:11 2019 +0200<br>
><br>
>     udev: modernize udev-rules.c<br>
<br>
<br>
A bug then? Or is there an error in the udev rule that I don't see?<br></blockquote><div><br></div><div>IMHO it is a bug, since it asks for the wrong operator...</div><div><br></div><div>As far as I know, IMPORT{} is not a list, it has immediate effect on the following rules, so it is normally used with simple assignment '='. (So although '+=' *could* be accepted, it would practically mean the exact same thing as '='.)</div><div><br></div><div>And even though the error message recommends '==', I don't see how that would make sense since IMPORT is by definition an update and not a comparison. There are two dm rules using IMPORT{}=="..." and it really seems like they should be using '=' instead.</div><div><br></div><div>It seems that the old code accepted literally any operator except '-=', so maybe that's where the weird rules with '==' and '+=' come from.</div></div><div><br></div>-- <br><div dir="ltr" class="gmail_signature"><div dir="ltr">Mantas Mikulėnas</div></div></div>