<div dir="ltr">Oops no !<div>Thanks for noting.</div><div><br></div><div>An incremental fix is now merged.</div><div><br></div><div>You are right about the spelling fix that accidentaly was wrapped into this patch, but the patch has already been pushed online as-is so I don't see how I can improve that now.</div><div><br></div><div>Regards,</div><div>Christophe</div></div><div class="gmail_extra"><br><div class="gmail_quote">On Mon, Aug 1, 2016 at 6:02 PM, Bart Van Assche <span dir="ltr"><<a href="mailto:bart.vanassche@sandisk.com" target="_blank">bart.vanassche@sandisk.com</a>></span> wrote:<br><blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex"><span class="">On 07/31/2016 12:26 PM, Christophe Varoqui wrote:<br>
<blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex">
Can you review this patch, adding a new 'serial' keyword to the<br>
weightedpath prioritizer.<br>
</blockquote>
<br></span>
Is the following part of the patch intended?<span class=""><br>
<br>
        } else if (!strcmp(regex, WWN)) {<br>
-               if (build_wwn_path(pp, path, FILE_NAME_SIZE) != 0) {<br>
+               if (build_serial_path(pp, path, FILE_NAME_SIZE) != 0) {<br>
<br></span>
Additionally, have you considered to apply the mulitpathd -> multipathd spelling fix in this patch as a separate patch?<br>
<br>
Thanks,<br>
<br>
Bart.<br>
</blockquote></div><br></div>