Re: [pgsql-packagers] Palle Girgensohn's ICU patch - Mailing list pgsql-hackers

From Palle Girgensohn
Subject Re: [pgsql-packagers] Palle Girgensohn's ICU patch
Date
Msg-id D9AC014E-7016-4014-9726-8BB3B726B541@pingpong.net
Whole thread Raw
In response to Re: [pgsql-packagers] Palle Girgensohn's ICU patch  (Jakob Egger <jakob@eggerapps.at>)
Responses Re: [pgsql-packagers] Palle Girgensohn's ICU patch  (Greg Stark <stark@mit.edu>)
Re: [pgsql-packagers] Palle Girgensohn's ICU patch  (Palle Girgensohn <girgen@pingpong.net>)
List pgsql-hackers
> 26 nov 2014 kl. 11:44 skrev Jakob Egger <jakob@eggerapps.at>:
>
>
>> Am 26.11.2014 um 11:20 schrieb Dave Page <dpage@postgresql.org>:
>>
>> On Wed, Nov 26, 2014 at 10:13 AM, Jakob Egger <jakob@eggerapps.at> wrote:
>>> Am 26.11.2014 um 11:05 schrieb Dave Page <dpage@postgresql.org>:
>>>
>>> You may want to bear in mind that postgres.app is on the main PG
>>> downloads page on the website. If you're patching Postgres to add a
>>> feature like this, it would become a fork and would have to be moved
>>> out of the "PostgreSQL Core Distribution" section of the download area
>>> as we only include "pure" distributions there.
>>>
>>>
>>> I wasn't aware of this. I'll have to bring this up on the Postgres.app
>>> Github page.
>>>
>>> Personally, I don't think that shipping a database with broken text sorting
>>> is acceptable; but I can't speak on behalf of the other contributors to
>>> Postgres.app without consulting them first.
>>
>> Right - but the correct course of action would be to get the problem
>> fixed in PostgreSQL itself, not to fork the code which could lead to
>> other problems for users.
>
> Agreed. Since this isn't a priority for Palle I'll have a look at the patch to see if I can extend it to make it
suitablefor submitting it, but since I have never contributed source to PostgreSQL I don't know yet if I can handle it. 

Well, this discussion actually pushes the priority quite a bit for me -- someone else actually beeing interested about
thepatch... I thought it was just me... :)= 

Just for reference, the Linux collation is actaully also broken wrt to utf-8. It is better than others, but not
correct.And lower()/upper() has many rather common cases where it is not working with "wide characters". For example,
thetowupper only looks at one character at the time, but proper handling needs to look at adjacent characters in some
languages.

Either way, getting it into core would not happen before 9.5 anyway.

>
> I've opened an issue on Github to discuss what to do about Postgres.app and the upcoming 9.4 release:
> https://github.com/PostgresApp/PostgresApp/issues/233
>
> Best regards,
> Jakob




pgsql-hackers by date:

Previous
From: Alvaro Herrera
Date:
Subject: Re: 9.2 recovery/startup problems
Next
From: Stephen Frost
Date:
Subject: Re: superuser() shortcuts