On 12/12/2018 15:57, Daniel Verite wrote:
> I think one related issue that the patch works around by using a libc locale
> as a proxy is knowing what to put into libc's LC_CTYPE and LC_COLLATE.
> In fact I've been wondering if that's the main reason for the interface
> implemented by the patch.
So it seems, but then it should be called out more clearly.
> Otherwise, how should these env variables be initialized for ICU
> databases?
I think when using ICU by default, then it should not matter because we
shouldn't be calling any libc functions that use those settings. Maybe
there need to be some exceptions, but again we should call those out
more clearly.
We could set them to "C" for consistency perhaps.
--
Peter Eisentraut http://www.2ndQuadrant.com/
PostgreSQL Development, 24x7 Support, Remote DBA, Training & Services