Re: ICU for global collation - Mailing list pgsql-hackers

From Justin Pryzby
Subject Re: ICU for global collation
Date
Msg-id 20220909164625.GN31833@telsasoft.com
Whole thread Raw
In response to Re: ICU for global collation  (Marina Polyakova <m.polyakova@postgrespro.ru>)
Responses Re: ICU for global collation
List pgsql-hackers
In pg14:
|postgres=# create database a LC_COLLATE C LC_CTYPE C LOCALE C;
|ERROR:  conflicting or redundant options
|DETAIL:  LOCALE cannot be specified together with LC_COLLATE or LC_CTYPE.

In pg15:
|postgres=# create database a LC_COLLATE "en_US.UTF-8" LC_CTYPE "en_US.UTF-8" LOCALE "en_US.UTF-8" ;
|CREATE DATABASE

f2553d430 actually relaxed the restriction by removing this check:

-       if (dlocale && (dcollate || dctype))
-               ereport(ERROR,
-                               (errcode(ERRCODE_SYNTAX_ERROR),
-                                errmsg("conflicting or redundant options"),
-                                errdetail("LOCALE cannot be specified together with LC_COLLATE or LC_CTYPE.")));

But isn't the right fix to do the corresponding thing in createdb
(relaxing the frontend restriction rather than reverting its relaxation
in the backend).

diff --git a/src/bin/scripts/createdb.c b/src/bin/scripts/createdb.c
index e523e58b218..5b80e56dfd9 100644
--- a/src/bin/scripts/createdb.c
+++ b/src/bin/scripts/createdb.c
@@ -159,15 +159,10 @@ main(int argc, char *argv[])
             exit(1);
     }
 
-    if (locale)
-    {
-        if (lc_ctype)
-            pg_fatal("only one of --locale and --lc-ctype can be specified");
-        if (lc_collate)
-            pg_fatal("only one of --locale and --lc-collate can be specified");
+    if (locale && !lc_ctype)
         lc_ctype = locale;
+    if (locale && !lc_collate)
         lc_collate = locale;
-    }
 
     if (encoding)
     {


BTW it's somewhat crummy that it uses a string comparison, so if you
write "UTF8" without a dash, it says this; it took me a few minutes to
see the difference...

postgres=# create database a LC_COLLATE "en_US.UTF8" LC_CTYPE "en_US.UTF8" LOCALE "en_US.UTF8";
ERROR:  new collation (en_US.UTF8) is incompatible with the collation of the template database (en_US.UTF-8)



pgsql-hackers by date:

Previous
From: Stephen Frost
Date:
Subject: Re: Add tracking of backend memory allocated to pg_stat_activity
Next
From: Tom Lane
Date:
Subject: Re: configure --with-uuid=bsd fails on NetBSD