Let's rename --verbose --verbose to --debug at least.<div><br></div><div>I'd also prefer renaming --details to --trace or something because --details sounds less detailed than --debug but I'm not strongly opinionated about this.</div>
<div><br></div><div>- Ryosuke<br><br><div class="gmail_quote">On Tue, Jul 31, 2012 at 7:09 PM, Dirk Pranke <span dir="ltr"><<a href="mailto:dpranke@chromium.org" target="_blank">dpranke@chromium.org</a>></span> wrote:<br>
<blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex"><div class="HOEnZb"><div class="h5">On Tue, Jul 31, 2012 at 7:02 PM, Ryosuke Niwa <<a href="mailto:rniwa@webkit.org">rniwa@webkit.org</a>> wrote:<br>
> On Tue, Jul 31, 2012 at 6:59 PM, Dirk Pranke <<a href="mailto:dpranke@chromium.org">dpranke@chromium.org</a>> wrote:<br>
>><br>
>> On Tue, Jul 31, 2012 at 6:36 PM, Ryosuke Niwa <<a href="mailto:rniwa@webkit.org">rniwa@webkit.org</a>> wrote:<br>
>> > On Tue, Jul 31, 2012 at 6:29 PM, Dirk Pranke <<a href="mailto:dpranke@chromium.org">dpranke@chromium.org</a>><br>
>> > wrote:<br>
>> >><br>
>> >> I'm finally getting around to cleaning up the byzantine mass of<br>
>> >> options in new-run-webkit-tests that controls what gets printed to<br>
>> >> stderr and stdout during a test run.<br>
>> >><br>
>> >> The patch is posted in <a href="https://bugs.webkit.org/show_bug.cgi?id=92432" target="_blank">https://bugs.webkit.org/show_bug.cgi?id=92432</a>.<br>
>> >> To quote the changelog:<br>
>> >><br>
>> >> [All of the --print X,Y,Z options are] replaced by five printing<br>
>> >> modes/options:<br>
>> >><br>
>> >> 1) by default, we print some configuration information, errors,<br>
>> >> warnings, and a progress bar. We didn't use to print the configuration<br>
>> >> information by default, and I've found that (a) it's really useful and<br>
>> >> (b) most people didn't know about it. However, if you don't want that<br>
>> >> output, use ...<br>
>> >><br>
>> >> 2) if --quiet is passed, we don't print the configuration information<br>
>> >> (this was the default before)<br>
>> >><br>
>> >> 3) if --verbose is specified once, you will get a one-line-per test<br>
>> >> summary in addition to the default output of (1). This is close to the<br>
>> >> --verbose mode from ORWT and is probably more useful than the older<br>
>> >> form of --verbose, which printed out a lot more debug-type stuff. To<br>
>> >> get the debug-type stuff ...<br>
>> >><br>
>> >> 4) if --verbose is specified twice, you get the old-style debug log<br>
>> >> output<br>
>> >><br>
>> >> And, if you used to use the obscure but occasionally really helpful<br>
>> >> --print trace-everything flag:<br>
>> >><br>
>> >> 5) if --details is specified, you get the per-test details you used to<br>
>> >> get with --print trace-everything or --print trace-unexpected.<br>
>> >> --details handling is largely independendent of --quiet/--verbose.<br>
>> ><br>
>> ><br>
>> > We've still got too many --verbose/--details options.<br>
>> ><br>
>> > Can we just combine two --verbose and --details or remove one of them<br>
>> > and<br>
>> > call it --debug?<br>
>> ><br>
>><br>
>> I'm not sure if I understand your suggestion. Are you suggesting we<br>
>> call --details --debug instead? or have --verbose --verbose be<br>
>> replaced by --debug?<br>
><br>
><br>
> My suggestion is to merge --verbose --verbose and --details into one option:<br>
> --debug.<br>
<br>
</div></div>Right, I wouldn't want to do that; it would make the already-too-big<br>
bot output 6x bigger, I'm guessing.<br>
<div class="im"><br>
>> We definitely don't want --details to be enabled by default either<br>
>> interactively or on the bots; it logs way too much if you're running a<br>
>> lot of tests. I also personally would not want to have to slog through<br>
>> the debug output to extract the stuff that --details will print.<br>
>><br>
>> I would be fine with changing '--verbose --verbose' to '--debug', but<br>
>> I'm not sure if that's either much of an improvement or enough to make<br>
>> you happy?<br>
><br>
><br>
> So we need to have --verbose --verbose separately from --verbose and<br>
> --details to be used on bots? Why don't we get rid of single --verbose then?<br>
<br>
</div>ap@ and others have requested a single --verbose that is closer to<br>
what ORWT used to do (bug 88702); something more than what we get by<br>
default, but less verbose than either what --details or<br>
--verbose/--debug would be.<br>
<span class="HOEnZb"><font color="#888888"><br>
-- Dirk<br>
</font></span></blockquote></div><br></div>