Page MenuHomePhabricator

[dotest] Remove -q (quiet) flag.
ClosedPublic

Authored by JDevlieghere on Aug 27 2019, 3:35 PM.

Details

Summary

I propose removing the -q (quiet) flag and changing the default behavior. Currently the flag serves two purposes that are somewhat contradictory, as illustrated by the difference between the argument name (quiet) and the configuration flag (parsable). On the one hand it reduces output, but on the other hand it prints more output, like the result of individual tests. My proposal is to guard the extra output behind the verbose flag and always print the individual test results.

Diff Detail

Repository
rL LLVM

Event Timeline

JDevlieghere created this revision.Aug 27 2019, 3:35 PM
Herald added a project: Restricted Project. · View Herald TranscriptAug 27 2019, 3:35 PM
Herald added a subscriber: teemperor. · View Herald Transcript
This revision was not accepted when it landed; it landed in state Needs Review.Aug 28 2019, 9:30 AM
This revision was automatically updated to reflect the committed changes.
Herald added a project: Restricted Project. · View Herald TranscriptAug 28 2019, 9:30 AM