<div dir="ltr"><div class="gmail_quote"><div dir="ltr">On Fri, Jul 24, 2015 at 2:34 PM Aaron Ballman <<a href="mailto:aaron@aaronballman.com">aaron@aaronballman.com</a>> wrote:<br></div><blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex">On Fri, Jul 24, 2015 at 8:32 AM, Manuel Klimek <<a href="mailto:klimek@google.com" target="_blank">klimek@google.com</a>> wrote:<br>
> Seems like we need different instructions for different shells then :(<br>
> The problem is that otherwise the -*... can be subject to shell expansion if<br>
> it happens to match some files.<br>
<br>
Ah, I kind of wondered if this was a shell issue. Thank you for the<br>
verification!<br>
<br>
Do you think it makes sense to update the option parsing code to strip<br>
the single quotes if they are present?<br></blockquote><div><br></div><div>No, I don't think it's the tool's job to handle idiosyncrasies of the various shells.</div><div>For the docs I see two possibilities:</div><div>a) have 2 versions, one for cmd.exe, one for *sh.</div><div>b) the probability that users will actually have file named -something, is not that high, we use the non-quoted version</div><div><br></div><div>Alex, thoughts?</div><div>  </div><blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex">
<br>
~Aaron<br>
<br>
><br>
><br>
> On Thu, Jul 23, 2015 at 7:49 PM Aaron Ballman <<a href="mailto:aaron@aaronballman.com" target="_blank">aaron@aaronballman.com</a>><br>
> wrote:<br>
>><br>
>> This patch addresses two issues (I can split the patch if it is desired):<br>
>><br>
>> 1) The docs have some non-ASCII characters in them that aren't really<br>
>> required.<br>
>> 2) The docs suggest setting the checks using single quotes, which does<br>
>> not work (at least, on Windows).<br>
>><br>
>> When you specify checks like -checks='-*,misc-some-check', the single<br>
>> quotes are not stripped by the option parser. When converting the<br>
>> flags into globs to pass along to regex, the single quotes remain as<br>
>> part of the regular expression, and do not match appropriately. When<br>
>> the single quotes are left off, the globs are correctly generated.<br>
>><br>
>> ~Aaron<br>
</blockquote></div></div>