From: Tom Lane <tgl@sss.pgh.pa.us>
To: Andres Freund <andres@anarazel.de>
cc: Peter Eisentraut <peter.eisentraut@enterprisedb.com>,
        Aleksander Alekseev <aleksander@timescale.com>,
        PostgreSQL Hackers <pgsql-hackers@lists.postgresql.org>
Subject: Re: MacOS: xsltproc fails with "warning: failed to load external
 entity"
In-reply-to: <20230209181645.pyy322pfm6neuhp2@awork3.anarazel.de>
References: <20230201103204.ddqqjbuxe5e2jct2@alap3.anarazel.de>
 <1383731.1675272207@sss.pgh.pa.us>
 <20230201174900.n7aa35mgwwdtkxum@alap3.anarazel.de>
 <20230201190428.m2syfhxxhyjx6vpl@alap3.anarazel.de>
 <1404694.1675279219@sss.pgh.pa.us>
 <20230206005207.djeq3322p4focdxo@alap3.anarazel.de>
 <2563864.1675646734@sss.pgh.pa.us> <3788434.1675894693@sss.pgh.pa.us>
 <b3f8f45c-6a7a-6457-722d-4a1656389262@enterprisedb.com>
 <3892633.1675954662@sss.pgh.pa.us>
 <20230209181645.pyy322pfm6neuhp2@awork3.anarazel.de>
Comments: In-reply-to Andres Freund <andres@anarazel.de>
	message dated "Thu, 09 Feb 2023 10:16:45 -0800"
MIME-Version: 1.0
Content-Type: text/plain; charset="us-ascii"
Content-ID: <3938014.1675968526.1@sss.pgh.pa.us>
Content-Transfer-Encoding: quoted-printable
Date: Thu, 09 Feb 2023 13:48:46 -0500
Message-ID: <3938015.1675968526@sss.pgh.pa.us>
Archived-At: 
 <https://www.postgresql.org/message-id/3938015.1675968526%40sss.pgh.pa.us>
Precedence: bulk

Andres Freund <andres@anarazel.de> writes:
> I think this misunderstanding is again due to the confusion between the =
'all'
> target in doc/src/sgml and the default target, just like earlier in the =
thread
> / why I ended up with the prior set of targets under 'docs'.

>   # Make "html" the default target, since that is what most people tend
>   # to want to use.
>   html:
>   ...
>   all: html man

> Given the repeated confusion from that, among fairly senior hackers, per=
haps
> we ought to at least put those lines next to each other? It's certainly =
not
> obvious as-is.

I think there are ordering constraints between these and the
Makefile.global inclusion.  But we could add a comment beside the "all:"
line pointing out that that's not the default target.

> Perhaps we should have doc-html, doc-man, doc-all or such?

No objection here.

If we intend to someday build tarballs with meson, there'd need to be
a target that builds html+man, but that could perhaps be named
"distprep".

			regards, tom lane