<div dir="ltr"><br><div class="gmail_extra"><br><div class="gmail_quote">On Tue, Oct 11, 2016 at 1:38 PM, Renato Golin <span dir="ltr"><<a href="mailto:renato.golin@linaro.org" target="_blank" class="cremed">renato.golin@linaro.org</a>></span> wrote:<br><blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex">On 11 October 2016 at 07:13, Daniel Jasper via cfe-commits<br>
<span class=""><<a href="mailto:cfe-commits@lists.llvm.org" class="cremed">cfe-commits@lists.llvm.org</a>> wrote:<br>
> Author: djasper<br>
> Date: Tue Oct 11 01:13:18 2016<br>
> New Revision: 283853<br>
><br>
> URL: <a href="http://llvm.org/viewvc/llvm-project?rev=283853&view=rev" rel="noreferrer" target="_blank" class="cremed">http://llvm.org/viewvc/llvm-<wbr>project?rev=283853&view=rev</a><br>
> Log:<br>
> Explicitly ignore return code in test for test systems that use pipefail<br>
<br>
</span>Hi Daniel,<br>
<br>
I missed this commit and reverted the original patch.<br>
<br>
However, before we reapply, why not use "not" instead of "|| true" ?<br></blockquote><div><br></div><div>Just because I don't know shell very well :). Feel free to use "not" instead.</div><div> </div><blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex">
<br>
cheers,<br>
--renato<br>
</blockquote></div><br></div></div>