On Tue, Aug 30, 2011 at 9:05 PM, Andrew Trick <span dir="ltr"><<a href="mailto:atrick@apple.com">atrick@apple.com</a>></span> wrote:<br><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">Thanks, please commit.<div><br><div><div><div></div><div class="h5"><div>On Aug 30, 2011, at 9:49 AM, arrowdodger wrote:</div><br></div></div><blockquote type="cite"><div><div></div><div class="h5">

On Tue, Aug 30, 2011 at 8:20 PM, Andrew Trick <span dir="ltr"><<a href="mailto:atrick@apple.com" target="_blank">atrick@apple.com</a>></span> wrote:<br><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><div></div><div><div>On Aug 30, 2011, at 5:09 AM, arrowdodger wrote:</div><br></div></div><blockquote type="cite"><div><div></div><div>On Tue, Aug 30, 2011 at 2:20 AM, Andrew Trick <span dir="ltr"><<a href="mailto:atrick@apple.com" target="_blank">atrick@apple.com</a>></span> wrote:<br>



<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">Can we call this ignoreStdErr instead of noStdErr</div></blockquote><div><br>Done. <br><br></div><blockquote class="gmail_quote" style="margin-top:0pt;margin-right:0pt;margin-bottom:0pt;margin-left:0.8ex;border-left-width:1px;border-left-style:solid;border-left-color:rgb(204, 204, 204);padding-left:1ex">





<div style="word-wrap:break-word">, then add the command line option --ignore-stderr?<div><br></div><div>("noStdErr" could be misinterpreted as something like --quiet).</div>
<div><br></div><div>I'm curious how you're currently setting litConfig.noStdErr.</div><div><br></div><div>-Andy<br><div><br><div><div>On Aug 27, 2011, at 4:40 AM, arrowdodger wrote:</div><br><blockquote type="cite">






<span style="border-collapse:separate;font-family:Helvetica;font-style:normal;font-variant:normal;font-weight:normal;letter-spacing:normal;line-height:normal;text-align:-webkit-auto;text-indent:0px;text-transform:none;white-space:normal;word-spacing:0px;font-size:medium"><span><lit.patch></span></span></blockquote>






</div><br></div></div></div></blockquote></div><br>Firstly, i've solved this exactly  in such way, but Daniel Dunbar suggested me to remove command line option and add default constructor to TclTest class. So this option can be turned on/off in test/lit.site.cfg.<br>






</div></div><span><lit.patch></span></blockquote></div><br><div>Looks fine to me. Can you add a comment in lit.site.cfg with an example? I know how to set TestingConfig options, but not LitConfig options.</div><div>



<br></div><div>-Andy</div></div>
</blockquote></div><br>Sorry, i've mistaken lit.site.cfg with lit.cfg. The latter is the right file to put comment in.<br>
</div></div><span><lit.patch></span></blockquote></div><br></div></div>
</blockquote></div><br>I don't have commit access, please commit it for me.<br>