public inbox for [email protected]  

help / color / mirror / Atom feed

From: Andres Freund <[email protected]>
To: Daniel Gustafsson <[email protected]>
Cc: Peter Eisentraut <[email protected]>
Cc: [email protected]
Subject: Re: [RFC] building postgres with meson
Date: Mon, 21 Feb 2022 08:52:28 -0800
Message-ID: <[email protected]> (raw)
In-Reply-To: <[email protected]>
References: <[email protected]>
	<[email protected]>
	<[email protected]>
	<[email protected]>
	<[email protected]>

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

view thread (2+ messages)

reply

Reply instructions:

You may reply publicly to this message via plain-text email
using any one of the following methods:

* Reply to all the recipients using the --to and --cc options:
  reply via email

  To: [email protected]
  Cc: [email protected], [email protected], [email protected]
  Subject: Re: [RFC] building postgres with meson
  In-Reply-To: <[email protected]>

* Save the following mbox file, import it into your mail client,
  and reply-to-all from there: mbox

This inbox is served by agora; see mirroring instructions
for how to clone and mirror all data and code used for this inbox