<div dir="ltr"><br><br><div class="gmail_quote">On Thu Feb 19 2015 at 5:01:43 PM Enrico Granata <<a href="mailto:egranata@apple.com">egranata@apple.com</a>> wrote:<br><blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex"><div style="word-wrap:break-word"><div><blockquote type="cite"><div>On Feb 19, 2015, at 4:55 PM, Zachary Turner <<a href="mailto:zturner@google.com" target="_blank">zturner@google.com</a>> wrote:</div></blockquote></div></div><div style="word-wrap:break-word"><blockquote type="cite"><div dir="ltr"><div class="gmail_quote"><div>The reason I didn't bring this up earlier is because Windows actually can glob with *. so the examples i had seen so far would have worked.  If we want to say that you get arbitrary shell expansion, then that's going to make tests very hard to write portably, and I'm not crazy about the idea of seeing even more platform specific tests going in.</div><div><br></div><div>So instead, why not just agree on a very simple subset of what the shell does -- in particular allow the use of the character set [*.?] and just write the glob function ourselves?</div><div><br></div></div></div></blockquote><div><span style="font-size:13.1999998092651px">I just don’t see the value of doing more work for less feature.</span></div></div></blockquote><div> </div><div>I see it as more work now for all the feature you actually need, and less work later.</div><div> </div><blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex"><div style="word-wrap:break-word"><div><blockquote type="cite"><div><div dir="ltr"><div class="gmail_quote"><blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex"><div style="word-wrap:break-word"><div><div>PlatformWindows could very well decide that it does not support globbing if this is such a big deal.</div></div></div></blockquote><div><span style="font-size:13.1999998092651px">It does though, just with a different set of rules.  So now any test that goes in to test this has to have tests for each platform's shell syntax.  We should be trying to find abstractions to bring all the platforms together IMO, not increasing the separation.</span><br></div></div></div>
</div></blockquote></div></div><div style="word-wrap:break-word"><div></div><div><br></div><div>That which will be part of testing the platform-specific bits of your platform</div></div></blockquote><div><br></div><div> There's inherent value in having as few platform specific bits as possible.  When the same rules apply to every platform, </div><div>a) everyone gets better test coverage</div><div>b) everyone gets to benefit from everyone else's changes and bugfixes</div><div>c) as a user, it makes it easier and more comfortable to switch platforms since you don't have to do a mental switch.</div><div>d) It makes the code easier to maintain</div></div></div>