From: Peter Eisentraut <peter.eisentraut@enterprisedb.com>
To: Michael Paquier <michael@paquier.xyz>
Cc: Postgres hackers <pgsql-hackers@lists.postgresql.org>
Subject: Re: Add documentation for coverage reports with meson
Date: Wed, 8 Mar 2023 17:23:48 +0100
Message-ID: <d3751ca3-1ac9-cc5e-9315-cf9fb0eaa9e9@enterprisedb.com> (raw)
In-Reply-To: <ZAHWFQK7VYGXhcZ0@paquier.xyz>
References: <Y/3AI+/MqKcjLk/T@paquier.xyz>
<f7e5de60-7a69-88ee-27c1-1cf11c1cbbe2@enterprisedb.com>
<ZAHWFQK7VYGXhcZ0@paquier.xyz>
On 03.03.23 12:12, Michael Paquier wrote:
> +<screen>
> +meson setup -Db_coverage=true ... OTHER OPTIONS ... builddir/
> +cd builddir/
> +meson compile
> +meson test
> +ninja coverage-html
> +</screen>
The "cd" command needs to be moved after the meson commands, and the
meson commands need to have a -C builddir option. So it should be like
<screen>
meson setup -Db_coverage=true ... OTHER OPTIONS ... builddir/
meson compile -C builddir
meson test -C builddir
cd builddir/
ninja coverage-html
</screen>
Otherwise, this looks good to me.
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: pgsql-hackers@postgresql.org
Cc: peter.eisentraut@enterprisedb.com, michael@paquier.xyz, pgsql-hackers@lists.postgresql.org
Subject: Re: Add documentation for coverage reports with meson
In-Reply-To: <d3751ca3-1ac9-cc5e-9315-cf9fb0eaa9e9@enterprisedb.com>
* Save the following mbox file, import it into your mail client,
and reply-to-all from there: mbox
This inbox is served by DDX for PostgreSQL; see mirroring instructions
for how to clone and mirror all data and code used for this inbox