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

Поиск
Список
Период
Сортировка
От Palle Girgensohn
Тема Re: [pgsql-packagers] Palle Girgensohn's ICU patch
Дата
Msg-id D9AC014E-7016-4014-9726-8BB3B726B541@pingpong.net
обсуждение исходный текст
Ответ на Re: [pgsql-packagers] Palle Girgensohn's ICU patch  (Jakob Egger <jakob@eggerapps.at>)
Ответы Re: [pgsql-packagers] Palle Girgensohn's ICU patch
Re: [pgsql-packagers] Palle Girgensohn's ICU patch
Список 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 по дате отправления:

Предыдущее
От: Alvaro Herrera
Дата:
Сообщение: Re: 9.2 recovery/startup problems
Следующее
От: Stephen Frost
Дата:
Сообщение: Re: superuser() shortcuts