<div dir="ltr">On Fri, Jul 26, 2013 at 10:26 AM, Rafael Espíndola <span dir="ltr"><<a href="mailto:rafael.espindola@gmail.com" target="_blank" class="cremed">rafael.espindola@gmail.com</a>></span> wrote:<br><div class="gmail_extra">
<div class="gmail_quote"><blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex"><div class="im">> Ok, here is a strong LGTM. =]<br>
><br>
> Please make the change, and do the following things to aid out-of-tree<br>
> maintainers:<br>
><br>
> 1) Add a flag to lit and an option to configure/make (I don't care about<br>
> CMake here as that is much less frequently used for out-of-tree work) to<br>
> disable pipefail.<br>
><br>
<br>
</div>I have just fixed the last failures on windows. I have also added<br>
documentation and the support for disabling pipefail in a directory.<br>
The one thing I have not implemented yet is the configure change.<br>
<br>
The reason is that after thinking a bit about it it looks like<br>
something we don't want to have. What we want to provide is an easy<br>
way for people doing out of tree work to get their tests passing after<br>
an upgrade. We do want upstream tests to fail for them if they, for<br>
example, break opt so that it crashes on exit. This is exactly what<br>
the lit.local.cfg provides.<br></blockquote><div><br></div><div>I can see your argument here and am fine with it on further thought. Thanks.</div><div> </div><blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex">
A new patch is attached. What do you think?<br></blockquote><div><br></div><div>Looks good. I might add in the release notes a quick "paste the following into a lit.local.cfg file in your test subtree to turn this off if you need to" bit? I'm just trying to avoid grumbling by giving a recipe for ignoring this change on old out-of-tree targets that aren't likely to be updated to test correctly.</div>
<div> </div><blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex">
<div class="im"><br>
> Also, please send a note (in a new thread) to llvmdev when the switch is<br>
> flipped with a reminder about how to disable the new behavior for folks that<br>
> can't update their test suite. You'll probably want to flip the switch when<br>
> you have time to track down lots of build bot failures. =D<br>
<br>
</div>OK.<br></blockquote><div><br></div><div>If you can give the recipe in th erelease notes, I'd paste it into the email as well.</div><div><br></div><div>Thanks for working on this.</div></div></div></div>