Hi, On 2021-10-13 13:54:10 +0200, Daniel Gustafsson wrote: > I added a --tap option for TAP output to pg_regress together with Jinbao Chen > for giggles and killing some time a while back.
Sorry for not replying to this earlier. I somehow thought I had, but the archives disagree. I think this would be great. > If it's helpful and there's any interest for this I'm happy to finish it up > now. Yes! Probably worth starting a new thread for... > One thing that came out of this, is that we don't really handle the ignored > tests in the way the code thinks it does for normal output, the attached > treats > ignored tests as SKIP tests. I can't really parse the first sentence... > if (exit_status != 0) > log_child_failure(exit_status); > @@ -2152,6 +2413,7 @@ regression_main(int argc, char *argv[], > {"config-auth", required_argument, NULL, 24}, > {"max-concurrent-tests", required_argument, NULL, 25}, > {"make-testtablespace-dir", no_argument, NULL, 26}, > + {"tap", no_argument, NULL, 27}, > {NULL, 0, NULL, 0} > }; I'd make it a --format=(regress|tap) or such. Greetings, Andres Freund