From: Andrew Dunstan <andrew@dunslane.net>
To: Tom Lane <tgl@sss.pgh.pa.us>
Cc: Andres Freund <andres@anarazel.de>
Cc: pgsql-committers@lists.postgresql.org
Subject: Re: pgsql: meson: docs: Add {html,man} targets, rename install-doc-*
Date: Wed, 29 Nov 2023 10:05:26 -0500
Message-ID: <0bc75483-2f4a-982b-2146-ce6bb0f03769@dunslane.net> (raw)
In-Reply-To: <286943.1701265792@sss.pgh.pa.us>
References: <E1r5Fwj-006RcU-Sh@gemulon.postgresql.org>
<746ba786-85bb-d1f7-b613-57bec35c642a@dunslane.net>
<20231129022843.6vfvgxjqpbgb62p5@awork3.anarazel.de>
<703110e4-b495-e409-26a5-28e9fca8f3a5@dunslane.net>
<286943.1701265792@sss.pgh.pa.us>
On 2023-11-29 We 08:49, Tom Lane wrote:
> Andrew Dunstan <andrew@dunslane.net> writes:
>> On 2023-11-28 Tu 21:28, Andres Freund wrote:
>>> I don't really understand why meson compile complains in this case. I assume
>>> you don't want to disambiguate as suggested, by building html:alias instead?
>> I've done that as a temporary fix to get crake out of the hole, but it's
>> pretty ugly, and I don't want to do it in a release if at all possible.
> Our documentation says specifically that "ninja html" will build the
> HTML format. I would expect that to work by analogy with the "make"
> target; having to spell it differently seems like clearly a bug.
>
>
"ninja html" does in fact work. What's not working is "meson compile
html". And it looks like the reason I used that in the buildfarm code is
that ninja doesn't know about other targets like "postgres-US.pdf". Up
to now "meson compile postgres-US.pdf html" has worked.
FWIW, the buildfarm code doesn't use ninja explicitly anywhere else.
cheers
andrew
--
Andrew Dunstan
EDB: https://www.enterprisedb.com
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-committers@postgresql.org
Cc: andrew@dunslane.net, tgl@sss.pgh.pa.us, andres@anarazel.de, pgsql-committers@lists.postgresql.org
Subject: Re: pgsql: meson: docs: Add {html,man} targets, rename install-doc-*
In-Reply-To: <0bc75483-2f4a-982b-2146-ce6bb0f03769@dunslane.net>
* 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