Received: from malur.postgresql.org ([217.196.149.56]) by arkaria.postgresql.org with esmtps (TLS1.3:ECDHE_RSA_AES_256_GCM_SHA384:256) (Exim 4.92) (envelope-from ) id 1pNHyU-0004yL-7d for pgsql-hackers@arkaria.postgresql.org; Wed, 01 Feb 2023 18:36:50 +0000 Received: from localhost ([127.0.0.1] helo=malur.postgresql.org) by malur.postgresql.org with esmtp (Exim 4.92) (envelope-from ) id 1pNHyS-0002bC-T7 for pgsql-hackers@arkaria.postgresql.org; Wed, 01 Feb 2023 18:36:48 +0000 Received: from magus.postgresql.org ([2a02:c0:301:0:ffff::29]) by malur.postgresql.org with esmtps (TLS1.3:ECDHE_RSA_AES_256_GCM_SHA384:256) (Exim 4.92) (envelope-from ) id 1pNHyS-0002ax-Ju for pgsql-hackers@lists.postgresql.org; Wed, 01 Feb 2023 18:36:48 +0000 Received: from sss.pgh.pa.us ([66.207.139.130]) by magus.postgresql.org with esmtps (TLS1.3:ECDHE_RSA_AES_256_GCM_SHA384:256) (Exim 4.92) (envelope-from ) id 1pNHyM-0002UF-Sg for pgsql-hackers@lists.postgresql.org; Wed, 01 Feb 2023 18:36:48 +0000 Received: from sss1.sss.pgh.pa.us (localhost [127.0.0.1]) by sss.pgh.pa.us (8.15.2/8.15.2) with ESMTP id 311IacIh1400498; Wed, 1 Feb 2023 13:36:38 -0500 From: Tom Lane To: Andres Freund cc: Aleksander Alekseev , PostgreSQL Hackers , Peter Eisentraut Subject: Re: MacOS: xsltproc fails with "warning: failed to load external entity" In-reply-to: <20230201174900.n7aa35mgwwdtkxum@alap3.anarazel.de> References: <489604.1675095648@sss.pgh.pa.us> <1182c94f-bd43-262c-9452-597ca7d7a742@enterprisedb.com> <1234349.1675196522@sss.pgh.pa.us> <1275436.1675209271@sss.pgh.pa.us> <20230201002206.zmilf5zlwhz2ahow@alap3.anarazel.de> <20230201103204.ddqqjbuxe5e2jct2@alap3.anarazel.de> <1383731.1675272207@sss.pgh.pa.us> <20230201174900.n7aa35mgwwdtkxum@alap3.anarazel.de> Comments: In-reply-to Andres Freund message dated "Wed, 01 Feb 2023 09:49:00 -0800" MIME-Version: 1.0 Content-Type: text/plain; charset="us-ascii" Content-ID: <1400496.1675276598.1@sss.pgh.pa.us> Content-Transfer-Encoding: quoted-printable Date: Wed, 01 Feb 2023 13:36:38 -0500 Message-ID: <1400497.1675276598@sss.pgh.pa.us> List-Id: List-Help: List-Subscribe: List-Post: List-Owner: List-Archive: Archived-At: Precedence: bulk Andres Freund writes: > On 2023-02-01 12:23:27 -0500, Tom Lane wrote: >> It's unlike what "make -C doc/src/sgml all" does in the Makefile >> system, and I don't find it to be an improvement. > Well, that'd actually build the manpages too, afaics :). But I get the > point. Ah, sorry, I too had forgotten that "all" isn't the default target there. I actually just go into that directory and type "make". > I really have no opinion on what we should should build under what > name. Happy to change what's included in 'docs', add additional targets, > etc. I think "docs" for just the html and "alldocs" for all supported outputs is probably reasonable. If we ever get to the point of building distribution tarballs with meson, we might need another target for html+man, but I suppose that's a long way off. >> And the minimum version appears to be newer than RHEL8's 1.8.2, which >> I find pretty unfortunate. On RHEL8, it fails with >> $ ninja >> ninja: error: build.ninja:6771: multiple outputs aren't (yet?) supporte= d by depslog; bring this up on the mailing list if it affects you > What's in that line +- 2 lines? And/or what are the steps that got you > to that point? "meson setup build" is sufficient to see it --- apparently ninja gets invoked at the end of that, and it's already unhappy. But it repeats after "cd build; ninja". It seems to be unhappy about the stanza for building sql_help.c? Line 6771 is the blank line after "description" in this bit: build src/bin/psql/sql_help.c src/bin/psql/sql_help.h: CUSTOM_COMMAND_DEP = | ../src/bin/psql/create_help.pl /usr/bin/perl DEPFILE =3D src/bin/psql/sql_help.dep DEPFILE_UNQUOTED =3D src/bin/psql/sql_help.dep COMMAND =3D /usr/bin/perl ../src/bin/psql/create_help.pl --docdir ../doc/= src/sgml/ref --depfile src/bin/psql/sql_help.dep --outdir src/bin/psql --b= asename sql_help description =3D Generating$ psql_help$ with$ a$ custom$ command build src/bin/psql/psql.p/meson-generated_.._psqlscanslash.c.o: c_COMPILER= src/bin/psql/psqlscanslash.c || src/bin/psql/sql_help.h src/include/catal= og/pg_aggregate_d.h src/include/catalog/pg_am_d.h src/include/catalog/pg_a= mop_d.h src/include/catalog/pg_amproc_d.h src/include/catalog/pg_attrdef_d= .h src/include/catalog/pg_attribute_d.h src/include/catalog/pg_auth_member= s_d.h src/include/catalog/pg_authid_d.h src/include/catalog/pg_cast_d.h sr= c/include/catalog/pg_class_d.h src/include/catalog/pg_collation_d.h src/in= clude/catalog/pg_constraint_d.h src/include/catalog/pg_conversion_d.h src/= include/catalog/pg_database_d.h src/include/catalog/pg_db_role_setting_d.h= src/include/catalog/pg_default_acl_d.h src/include/catalog/pg_depend_d.h = src/include/catalog/pg_description_d.h src/include/catalog/pg_enum_d.h src= /include/catalog/pg_event_trigger_d.h src/include/catalog/pg_extension_d.h= src/include/catalog/pg_foreign_data_wrapper_d.h src/include/catalog/pg_fo= reign_server_d.h src/include/catalog/pg_foreign_table_d.h src/include/cata= log/pg_index_d.h src/include/catalog/pg_inherits_d.h src/include/catalog/p= g_init_privs_d.h src/include/catalog/pg_language_d.h src/include/catalog/p= g_largeobject_d.h src/include/catalog/pg_largeobject_metadata_d.h src/incl= ude/catalog/pg_namespace_d.h src/include/catalog/pg_opclass_d.h src/includ= e/catalog/pg_operator_d.h src/include/catalog/pg_opfamily_d.h src/include/= catalog/pg_parameter_acl_d.h src/include/catalog/pg_partitioned_table_d.h = src/include/catalog/pg_policy_d.h src/include/catalog/pg_proc_d.h src/incl= ude/catalog/pg_publication_d.h src/include/catalog/pg_publication_namespac= e_d.h src/include/catalog/pg_publication_rel_d.h src/include/catalog/pg_ra= nge_d.h src/include/catalog/pg_replication_origin_d.h src/include/catalog/= pg_rewrite_d.h src/include/catalog/pg_seclabel_d.h src/include/catalog/pg_= sequence_d.h src/include/catalog/pg_shdepend_d.h src/include/catalog/pg_sh= description_d.h src/include/catalog/pg_shseclabel_d.h src/include/catalog/= pg_statistic_d.h src/include/catalog/pg_statistic_ext_d.h src/include/cata= log/pg_statistic_ext_data_d.h src/include/catalog/pg_subscription_d.h src/= include/catalog/pg_subscription_rel_d.h src/include/catalog/pg_tablespace_= d.h src/include/catalog/pg_transform_d.h src/include/catalog/pg_trigger_d.= h src/include/catalog/pg_ts_config_d.h src/include/catalog/pg_ts_config_ma= p_d.h src/include/catalog/pg_ts_dict_d.h src/include/catalog/pg_ts_parser_= d.h src/include/catalog/pg_ts_template_d.h src/include/catalog/pg_type_d.h= src/include/catalog/pg_user_mapping_d.h src/include/catalog/postgres.bki = src/include/catalog/schemapg.h src/include/catalog/system_constraints.sql = src/include/catalog/system_fk_info.h src/include/nodes/nodetags.h src/incl= ude/utils/errcodes.h DEPFILE =3D src/bin/psql/psql.p/meson-generated_.._psqlscanslash.c.o.d DEPFILE_UNQUOTED =3D src/bin/psql/psql.p/meson-generated_.._psqlscanslash= .c.o.d ARGS =3D -Isrc/bin/psql/psql.p -Isrc/bin/psql -I../src/bin/psql -Isrc/inc= lude -I../src/include -Isrc/interfaces/libpq -I../src/interfaces/libpq -fd= iagnostics-color=3Dalways -D_FILE_OFFSET_BITS=3D64 -Wall -Winvalid-pch -O3= -fno-strict-aliasing -fwrapv -fexcess-precision=3Dstandard -D_GNU_SOURCE = -Wmissing-prototypes -Wpointer-arith -Werror=3Dvla -Wendif-labels -Wmissin= g-format-attribute -Wimplicit-fallthrough=3D3 -Wcast-function-type -Wshado= w=3Dcompatible-local -Wformat-security -Wdeclaration-after-statement -Wno-= format-truncation -Wno-stringop-truncation -pthread >> I did manage to test this stuff on bleeding-edge Fedora, >> but ... > Yea, I worked a fair bit to avoid requiring a too new version, I'll try > to figure out what went wrong. I did built on rhel8 not long ago, so I > suspect it's a corner case somewhere. Oh, interesting. Let me know if you want me to test anything in particular. regards, tom lane