From: Tom Lane <tgl@sss.pgh.pa.us>
To: Jeff Davis <pgsql@j-davis.com>
cc: Daniel Verite <daniel@manitou-mail.org>,
        Andrew Gierth <andrew@tao11.riddles.org.uk>,
        Peter Eisentraut <peter.eisentraut@enterprisedb.com>,
        Sandro Santilli <strk@kbt.io>, Regina Obe <lr@pcorp.us>,
        pgsql-hackers@lists.postgresql.org
Subject: Re: Order changes in PG16 since ICU introduction
In-reply-to: <37520ec1ae9591f83132f82dbd625f3fc2d69c16.camel@j-davis.com>
References: <543e4ce6-8205-4aea-b54b-12b9ef747bf6@manitou-mail.org> <654a49f7ff7461bcf47be4181430678d45f93858.camel@j-davis.com> <37520ec1ae9591f83132f82dbd625f3fc2d69c16.camel@j-davis.com>
Comments: In-reply-to Jeff Davis <pgsql@j-davis.com>
	message dated "Wed, 03 May 2023 22:35:41 -0700"
MIME-Version: 1.0
Content-Type: text/plain; charset="us-ascii"
Content-ID: <483825.1683582475.1@sss.pgh.pa.us>
Content-Transfer-Encoding: quoted-printable
Date: Mon, 08 May 2023 17:47:55 -0400
Message-ID: <483826.1683582475@sss.pgh.pa.us>
Archived-At: <https://www.postgresql.org/message-id/483826.1683582475%40sss.pgh.pa.us>
Precedence: bulk

Jeff Davis <pgsql@j-davis.com> writes:
> =3D=3D=3D 0001: do not convert C to en-US-u-va-posix

> I plan to commit this soon.

Several buildfarm animals have failed since this went in.  The
only one showing enough info to diagnose is siskin [1]:

@@ -1043,16 +1043,15 @@
 ERROR:  ICU locale "nonsense-nowhere" has unknown language "nonsense"
 HINT:  To disable ICU locale validation, set parameter icu_validation_lev=
el to DISABLED.
 CREATE COLLATION testx (provider =3D icu, locale =3D 'C'); -- fails
-ERROR:  could not convert locale name "C" to language tag: U_ILLEGAL_ARGU=
MENT_ERROR
+NOTICE:  using standard form "en-US-u-va-posix" for locale "C"
 CREATE COLLATION testx (provider =3D icu, locale =3D '@colStrength=3Dprim=
ary;nonsense=3Dyes'); -- fails
 ERROR:  could not convert locale name "@colStrength=3Dprimary;nonsense=3D=
yes" to language tag: U_ILLEGAL_ARGUMENT_ERROR
 SET icu_validation_level =3D WARNING;
 CREATE COLLATION testx (provider =3D icu, locale =3D '@colStrength=3Dprim=
ary;nonsense=3Dyes'); DROP COLLATION testx;
 WARNING:  could not convert locale name "@colStrength=3Dprimary;nonsense=3D=
yes" to language tag: U_ILLEGAL_ARGUMENT_ERROR
+ERROR:  collation "testx" already exists
 CREATE COLLATION testx (provider =3D icu, locale =3D 'C'); DROP COLLATION=
 testx;
-WARNING:  could not convert locale name "C" to language tag: U_ILLEGAL_AR=
GUMENT_ERROR
-WARNING:  ICU locale "C" has unknown language "c"
-HINT:  To disable ICU locale validation, set parameter icu_validation_lev=
el to DISABLED.
+NOTICE:  using standard form "en-US-u-va-posix" for locale "C"
 CREATE COLLATION testx (provider =3D icu, locale =3D 'nonsense-nowhere');=
 DROP COLLATION testx;
 WARNING:  ICU locale "nonsense-nowhere" has unknown language "nonsense"
 HINT:  To disable ICU locale validation, set parameter icu_validation_lev=
el to DISABLED.

I suppose this is environment-dependent.  Sadly, the buildfarm
client does not show the prevailing LANG or LC_XXX settings.

			regards, tom lane

[1] https://buildfarm.postgresql.org/cgi-bin/show_log.pl?nm=3Dsiskin&dt=3D=
2023-05-08%2020%3A09%3A26