<div dir="ltr">Right, I agree with your comments.<div>The patch is reverted.</div><div><br></div><div>Thanks.</div></div><div class="gmail_extra"><br><br><div class="gmail_quote">On Thu, Jul 24, 2014 at 6:34 PM, Bryn M. Reeves <span dir="ltr"><<a href="mailto:bmr@redhat.com" target="_blank">bmr@redhat.com</a>></span> wrote:<br>
<blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex"><div class="">On Thu, Jul 24, 2014 at 04:55:59PM +0200, Sebastian Herbszt wrote:<br>
> Bart Van Assche wrote:<br>
> > Inside libmultipath regcomp() is used to compile regular expressions<br>
> > specified in /etc/multipath.conf. Many multipath.conf examples contain<br>
> > 'product_type "*"'. However, "*" is not a valid POSIX regular expression.<br>
> > Hence this patch that changes the regular expression "*" into ".*".<br>
> ><br>
</div><div class="">> Is this change really required? With patch 4 we now get a proper error:<br>
> multipath.conf +14 parsing failed:            vendor "*"<br>
> multipath.conf +15 parsing failed:            product "*"<br>
> error parsing config file<br>
><br>
> I think it should be enough to modify the man page to mention vendor/product<br>
> are both regular expressions. This change might also confuse users since this<br>
> automagic "*" to ".*" only applies to the blacklist exceptions.<br>
<br>
</div>Agreed; the examples are broken here so let's fix them rather than add magic<br>
parsing that will only come back to confuse people in the future.<br>
<br>
Regards,<br>
Bryn.<br>
<br>
--<br>
dm-devel mailing list<br>
<a href="mailto:dm-devel@redhat.com">dm-devel@redhat.com</a><br>
<a href="https://www.redhat.com/mailman/listinfo/dm-devel" target="_blank">https://www.redhat.com/mailman/listinfo/dm-devel</a><br>
</blockquote></div><br></div>