<div dir="ltr"><br><div class="gmail_extra"><br><div class="gmail_quote">On Sun, Jun 5, 2016 at 10:06 AM, Dmitry Vyukov <span dir="ltr"><<a href="mailto:dvyukov@google.com" target="_blank">dvyukov@google.com</a>></span> wrote:<br><blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex"><span class="">On Sun, Jun 5, 2016 at 7:02 PM, Xinliang David Li <<a href="mailto:xinliangli@gmail.com">xinliangli@gmail.com</a>> wrote:<br>
> 1) the output is less verbose<br>
> 2) The actionable stuff is captured in one script.<br>
><br>
> isn't it what you want?<br>
<br>
<br>
</span>The output must be useful and actionable in the first place. If we<br>
would care only about verbosity then we would print nothing. Empty<br>
output is the shortest.<br></blockquote><div><br></div><div>This is not the right argument by going to the extreme. The goal should be using the minimal output to capture the max possible context, so verbosity is just one side of the equation.</div><div> </div><blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex">
<br>
With what you propose it is not possible to see output on buildbots.<br>
<div class="HOEnZb"><div class="h5"><br></div></div></blockquote><div><br></div><div>You are right about buildbot use case -- which I missed.  I agree in that mode perhaps the more verbose the better :)</div><div><br></div><div>David</div><div><br></div><div><br></div><div> </div><blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex"><div class="HOEnZb"><div class="h5">
<br>
<br>
<br>
> On Sun, Jun 5, 2016 at 9:46 AM, Dmitry Vyukov <<a href="mailto:dvyukov@google.com">dvyukov@google.com</a>> wrote:<br>
>><br>
>> Why is it better? When you don't want to see the input?<br>
>><br>
>> On Sun, Jun 5, 2016 at 6:25 PM, Xinliang David Li <<a href="mailto:xinliangli@gmail.com">xinliangli@gmail.com</a>><br>
>> wrote:<br>
>> > I think the better approach is to capture the input and re-run command<br>
>> > line<br>
>> > in a temporary file by default.<br>
>> > 1) create a re-run file: rerun_script<br>
>> > 2) dump the following to rerun_script:<br>
>> > cat << EOF > input_tmp<br>
>> >   ..... input here<br>
>> > EOF<br>
>> ><br>
>> > cat input_tmp | <FileCheck Command><br>
>> ><br>
>> > 3) in error message, show " To rerun, run rerun_script ".<br>
>> ><br>
>> >  David<br>
>> ><br>
>> ><br>
>> > On Sun, Jun 5, 2016 at 8:49 AM, Dmitry Vyukov via llvm-commits<br>
>> > <<a href="mailto:llvm-commits@lists.llvm.org">llvm-commits@lists.llvm.org</a>> wrote:<br>
>> >><br>
>> >> dvyukov added a comment.<br>
>> >><br>
>> >> > FYI, see also <a href="http://reviews.llvm.org/D20105" rel="noreferrer" target="_blank">http://reviews.llvm.org/D20105</a><br>
>> >><br>
>> >><br>
>> >> Answered there. I don't understand why printing input is not the<br>
>> >> default.<br>
>> >><br>
>> >><br>
>> >> <a href="http://reviews.llvm.org/D20925" rel="noreferrer" target="_blank">http://reviews.llvm.org/D20925</a><br>
>> >><br>
>> >><br>
>> >><br>
>> >> _______________________________________________<br>
>> >> llvm-commits mailing list<br>
>> >> <a href="mailto:llvm-commits@lists.llvm.org">llvm-commits@lists.llvm.org</a><br>
>> >> <a href="http://lists.llvm.org/cgi-bin/mailman/listinfo/llvm-commits" rel="noreferrer" target="_blank">http://lists.llvm.org/cgi-bin/mailman/listinfo/llvm-commits</a><br>
>> ><br>
>> ><br>
><br>
><br>
</div></div></blockquote></div><br></div></div>