pg.ddx.io  pgsql-hackers@postgresql.org mailing list archive  
help / color / mirror / Atom feed
From: Peter Eisentraut <peter.eisentraut@2ndquadrant.com>
To: Andreas Karlsson <andreas@proxel.se>
To: pgsql-hackers <pgsql-hackers@postgresql.org>
Subject: Re: ICU integration
Date: Thu, 23 Mar 2017 15:34:48 -0400
Message-ID: <c0235559-e1c0-561d-5ec1-2d96ea36e212@2ndquadrant.com> (raw)
In-Reply-To: <cde141a0-0e82-1d2d-9902-408baaf10fe6@proxel.se>
References: <85364fde-091f-bbc0-fec2-e3ede39840a6@2ndquadrant.com>
	<3a88fa29-275b-f547-9957-86d4b2cd68db@2ndquadrant.com>
	<CAFj8pRAxMEUXcQoihYMWP8t6EfaKSZB3njJom9eAZT=EA54LMQ@mail.gmail.com>
	<dc111c2d-cf76-dc5d-e179-75869dbb6a8a@2ndquadrant.com>
	<ce46c712-4776-c5e3-9121-864673522f92@2ndquadrant.com>
	<48b736bd-e589-6e71-3fd1-2d058c319c66@2ndquadrant.com>
	<47e95371-2f86-d5b7-8dd3-88944cd14b94@proxel.se>
	<5291804b-169e-3ba9-fdaf-fae8e7d2d959@2ndquadrant.com>
	<29b8c74d-4029-b7f4-1dae-6e622e9ce70b@proxel.se>
	<272a46ca-4596-563d-1d9c-7d08a0d35006@2ndquadrant.com>
	<cde141a0-0e82-1d2d-9902-408baaf10fe6@proxel.se>
List-Unsubscribe: <mailto:majordomo@postgresql.org?body=unsub%20pgsql-hackers>

On 3/23/17 05:34, Andreas Karlsson wrote:
> I am fine with this version of the patch. The issues I have with it, 
> which I mentioned earlier in this thread, seem to be issues with ICU 
> rather than with this patch. For example there seems to be no way for 
> ICU to validate the syntax of the BCP 47 locales (or ICU's old format). 
> But I think we will just have to accept the weirdness of how ICU handles 
> locales.
> 
> I think this patch is ready to be committed.
> 
> Found a typo in the documentation:
> 
> "The inspect the currently available locales" should be "To inspect the 
> currently available locales".

Committed.

-- 
Peter Eisentraut              http://www.2ndQuadrant.com/
PostgreSQL Development, 24x7 Support, Remote DBA, Training & Services


-- 
Sent via pgsql-hackers mailing list (pgsql-hackers@postgresql.org)
To make changes to your subscription:
http://www.postgresql.org/mailpref/pgsql-hackers



view thread (83+ messages)  latest in thread

Message-ID: <c0235559-e1c0-561d-5ec1-2d96ea36e212@2ndquadrant.com>
Permalink:  ../c0235559-e1c0-561d-5ec1-2d96ea36e212@2ndquadrant.com/
Also on:    postgresql.org/message-id/c0235559-e1c0-561d-5ec1-2d96ea36e212@2ndquadrant.com

 · 

reply

Reply instructions:

You may reply publicly to this message via plain-text email
using any one of the following methods:

* Reply to all the recipients using the --to and --cc options:
  reply via email

  To: pgsql-hackers@postgresql.org
  Cc: peter.eisentraut@2ndquadrant.com, andreas@proxel.se
  Subject: Re: ICU integration
  In-Reply-To: <c0235559-e1c0-561d-5ec1-2d96ea36e212@2ndquadrant.com>

* Save the following mbox file, import it into your mail client,
  and reply-to-all from there: mbox

This inbox is served by DDX for PostgreSQL; see mirroring instructions
for how to clone and mirror all data and code used for this inbox