Date: Mon, 27 Mar 2023 16:24:41 +0800
From: Julien Rouhaud <rjuju123@gmail.com>
To: Tom Lane <tgl@sss.pgh.pa.us>
Cc: Andres Freund <andres@anarazel.de>,
	Aleksander Alekseev <aleksander@timescale.com>,
	PostgreSQL Hackers <pgsql-hackers@lists.postgresql.org>,
	Peter Eisentraut <peter.eisentraut@enterprisedb.com>
Subject: Re: MacOS: xsltproc fails with "warning: failed to load external
 entity"
Message-ID: <20230327082441.h7pa2vqiobbyo7rd@jrouhaud>
References: <20230201002206.zmilf5zlwhz2ahow@alap3.anarazel.de>
 <CAJ7c6TOifNa77Oii2-PVZP323ifai3pPqC--y0=5tmByx-_4Ow@mail.gmail.com>
 <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>
MIME-Version: 1.0
Content-Type: multipart/mixed; boundary="d3j7adnnxvsczcrx"
Content-Disposition: inline
In-Reply-To: <3788434.1675894693@sss.pgh.pa.us>
Archived-At: 
 <https://www.postgresql.org/message-id/20230327082441.h7pa2vqiobbyo7rd%40jrouhaud>
Precedence: bulk


--d3j7adnnxvsczcrx
Content-Type: text/plain; charset=us-ascii
Content-Disposition: inline

Hi,

On Wed, Feb 08, 2023 at 05:18:13PM -0500, Tom Lane wrote:
> I pushed the discussed documentation improvements, and changed the
> behavior of "ninja docs" to only build the HTML docs.  However,
> I've not done anything about documenting what is the minimum
> ninja version.

FTR the documented XML_CATALOG_FILES environment variable is only valid for
Intel based machines, as homebrew installs everything in a different location
for M1...

I'm attaching a patch to make that distinction, hoping that no one else will
have to waste time trying to figure out how to get it working on such hardware.

--d3j7adnnxvsczcrx
Content-Type: text/plain; charset=us-ascii
Content-Disposition: attachment;
	filename="0001-Fix-XML_CATALOG_FILES-env-var-for-Apple-M1-base-mach.patch"

From e51f1b13dc70798b37e9d8f4bb34664fe138dd86 Mon Sep 17 00:00:00 2001
From: Julien Rouhaud <julien.rouhaud@free.fr>
Date: Mon, 27 Mar 2023 16:18:12 +0800
Subject: [PATCH] Fix XML_CATALOG_FILES env var for Apple M1 base machines.

Author: Julien Rouhaud
Reviewed-by: FIXME
Discussion: FIXME
---
 doc/src/sgml/docguide.sgml | 6 +++++-
 1 file changed, 5 insertions(+), 1 deletion(-)

diff --git a/doc/src/sgml/docguide.sgml b/doc/src/sgml/docguide.sgml
index cf8c072a49..81626d51cd 100644
--- a/doc/src/sgml/docguide.sgml
+++ b/doc/src/sgml/docguide.sgml
@@ -209,9 +209,13 @@ brew install docbook docbook-xsl libxslt fop
 
    <para>
     The Homebrew-supplied programs require the following environment variable
-    to be set:
+    to be set.  For Intel based machines:
 <programlisting>
 export XML_CATALOG_FILES=/usr/local/etc/xml/catalog
+</programlisting>
+    or for Apple M1 based machines:
+<programlisting>
+export XML_CATALOG_FILES=/opt/homebrew//etc/xml/catalog
 </programlisting>
     Without it, <command>xsltproc</command> will throw errors like this:
 <programlisting>
-- 
2.37.0


--d3j7adnnxvsczcrx--