Re: pgsql: Add option to use ICU as global locale provider
От | Julien Rouhaud |
---|---|
Тема | Re: pgsql: Add option to use ICU as global locale provider |
Дата | |
Msg-id | 20220320114626.jqqbgg4isfyyefif@jrouhaud обсуждение исходный текст |
Ответ на | Re: pgsql: Add option to use ICU as global locale provider (Peter Eisentraut <peter.eisentraut@enterprisedb.com>) |
Список | pgsql-hackers |
On Sun, Mar 20, 2022 at 11:03:38AM +0100, Peter Eisentraut wrote: > On 19.03.22 05:14, Julien Rouhaud wrote: > > On Fri, Mar 18, 2022 at 03:09:59PM -0700, Andres Freund wrote: > > > Hi, > > > > > > On 2022-03-18 20:28:58 +0100, Peter Eisentraut wrote: > > > > Why does your patch introduce a function check_icu_locale() that is only > > > > called once? Did you have further plans for that? > > > > > > I like that it moves ICU code out of dbcommands.c > > > > Yes, it seemed cleaner this way. But more importantly code outside pg_locale.c > > really shouldn't have to deal with ICU specific version code. > > > > I'm attaching a v2, addressing Peter and Tom comments to not duplicate the old > > ICU versions attribute function. I removed the ICU locale check entirely (for > > consistency across ICU version) thus removing any need for ucol.h include in > > initdb. > > committed Thanks!
В списке pgsql-hackers по дате отправления: