On Wed, Aug 23, 2023 at 09:36:01AM -0700, Jeff Davis wrote:
> On Tue, 2023-08-22 at 22:23 -0400, Bruce Momjian wrote:
> > > I notice that this item is still listed:
> > >
> > > * Determine the ICU default locale from the environment (Jeff
> > > Davis)
> > >
> > > But that was reverted as part of 2535c74b1a.
> >
> > The original commit is:
> >
> > Author: Jeff Davis <jda...@postgresql.org>
> > 2023-03-10 [c45dc7ffb] initdb: derive encoding from locale
> > for ICU; similar to
> >
> > and I don't see that reverted by 2535c74b1a. Is that a problem?
>
> c45dc7ffb causes initdb to choose the encoding based on the environment
> for ICU just like libc, and that was not reverted, so in v16:
>
> $ export LANG=en_US
> $ initdb -D data --locale-provider=icu --icu-locale=en
> ...
> The default database encoding has accordingly been set to "LATIN1".
>
> Whereas previously in v15 that would cause an error like:
>
> initdb: error: encoding mismatch
> initdb: detail: The encoding you selected (UTF8) and the encoding
> that the selected locale uses (LATIN1) do not match...
>
> "Determine the ICU default locale from the environment" to me refers to
> what happened in 27b62377b4, where initdb would select an ICU locale if
> one was not provided. 2535c74b1a reverted that, so in v16:
>
> $ initdb -D data --locale-provider=icu
> initdb: error: ICU locale must be specified
>
> Just like in v15.
Okay, so what I hear you saying is that commit c45dc7ffb needs to remain
in the release notes, but its description sounds like 27b62377b4, which
was reverted, so my description is wrong for c45dc7ffb.
I would love to blame the patch revert on this mistake, but looking at
the history of this entry, I just didn't understand it when I initiallly
wrote it. Updated applied patch attached.
--
Bruce Momjian <br...@momjian.us> https://momjian.us
EDB https://enterprisedb.com
Only you can decide what is important to you.
diff --git a/doc/src/sgml/release-16.sgml b/doc/src/sgml/release-16.sgml
index 488887c72b..ddd8bc3f3b 100644
--- a/doc/src/sgml/release-16.sgml
+++ b/doc/src/sgml/release-16.sgml
@@ -1566,15 +1566,12 @@ Author: Jeff Davis <jda...@postgresql.org>
<listitem>
<para>
- Determine the <acronym>ICU</acronym> default locale from the
- environment (Jeff Davis)
+ Determine the default encoding from the locale when using
+ <acronym>ICU</acronym> (Jeff Davis)
</para>
<para>
- However, <link
- linkend="locale-providers"><acronym>ICU</acronym></link> doesn't
- support the C locale so <literal>UTF-8</literal> is used in such
- cases. Previously the default was always <literal>UTF-8</literal>.
+ Previously the default was always <literal>UTF-8</literal>.
</para>
</listitem>
