<div dir="ltr">Nah, doesn't look like it. Removed it (& the place I copied it from) in: r324738<br><br>Thanks!<br><br><div class="gmail_quote"><div dir="ltr">On Fri, Feb 9, 2018 at 7:28 AM George Rimar <<a href="mailto:grimar@accesssoftek.com" target="_blank">grimar@accesssoftek.com</a>> wrote:<br></div><blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex"><div dir="ltr" style="font-size:12pt;color:#000000;background-color:#ffffff;font-family:Calibri,Arial,Helvetica,sans-serif">
<p><span style="color:rgb(33,33,33);font-size:12pt">>Totally fair call, sorry it took me a while to come back around to this - added in r324702</span><br>
</p>
</div><div dir="ltr" style="font-size:12pt;color:#000000;background-color:#ffffff;font-family:Calibri,Arial,Helvetica,sans-serif"><div style="color:rgb(33,33,33)">
<div><br>
Thanks ! Have a question: does it need "requres shell" ?<br>
</div>
<div>I tried without and it worked for me under windows (had to change check to CHECK: .{{\\|/}}test.cpp:2:51 though).<br>
</div>
<div>And I see nothing special probably that might require shell I think. We use cd/rm/cp in LLD testcases<br>
</div>
<div>without requiring shell.<br>
</div>
<div>(also noticed other tests do that, so it seems consistent, though did not look deeply)<br>
</div></div></div><div dir="ltr" style="font-size:12pt;color:#000000;background-color:#ffffff;font-family:Calibri,Arial,Helvetica,sans-serif"><div style="color:rgb(33,33,33)">
<div><br>
</div>
<div>George.<br>
</div>
</div></div></blockquote></div></div>