Feedback-ID: i1ff147bf:Fastmail
From: =?utf-8?Q?Dagfinn_Ilmari_Manns=C3=A5ker?= <ilmari@ilmari.org>
To: Daniel Gustafsson <daniel@yesql.se>
Cc: Julien Rouhaud <rjuju123@gmail.com>,  Tom Lane <tgl@sss.pgh.pa.us>,
  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"
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>
	<20230327082441.h7pa2vqiobbyo7rd@jrouhaud>
	<B574DD48-A526-4799-AF03-2C39C69868F9@yesql.se>
	<20230327084133.qzluydma36nhuaa3@jrouhaud>
	<10DA84BF-1D80-462D-8CED-F8898DD0DD94@yesql.se>
Date: Mon, 27 Mar 2023 13:04:03 +0100
In-Reply-To: <10DA84BF-1D80-462D-8CED-F8898DD0DD94@yesql.se> (Daniel
	Gustafsson's message of "Mon, 27 Mar 2023 12:07:10 +0200")
Message-ID: <87mt3ys8ng.fsf@wibble.ilmari.org>
User-Agent: Gnus/5.13 (Gnus v5.13) Emacs/27.1 (gnu/linux)
MIME-Version: 1.0
Content-Type: text/plain
Archived-At: 
 <https://www.postgresql.org/message-id/87mt3ys8ng.fsf%40wibble.ilmari.org>
Precedence: bulk

Daniel Gustafsson <daniel@yesql.se> writes:

>> On 27 Mar 2023, at 10:41, Julien Rouhaud <rjuju123@gmail.com> wrote:
>> On Mon, Mar 27, 2023 at 10:32:52AM +0200, Daniel Gustafsson wrote:
>>>> On 27 Mar 2023, at 10:24, Julien Rouhaud <rjuju123@gmail.com> wrote:
>
>>>> 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.
>>> 
>>> LGTM apart from the double // in the export which is easy enough to fix before
>>> pushing.
>>> 
>>> +export XML_CATALOG_FILES=/opt/homebrew//etc/xml/catalog
>> 
>> Oh, I didn't notice it.  Apparently apple's find isn't smart enough to trim a /
>> when fed with a directory with a trailing /
>
> Applied with a tiny but of changes to make it look like the rest of the
> paragraph more. Thanks!

Doesn't this apply to Apple Silicon generally, not just M1? M2 already
exists, and M3 etc. will presumably also appear at some point. The
linked Homebrew issue refers to Apple Silicon, not any specific models.

- ilmari