| From: | Peter Geoghegan <pg(at)bowt(dot)ie> |
|---|---|
| To: | Robert Haas <robertmhaas(at)gmail(dot)com> |
| Cc: | Ashutosh Sharma <ashu(dot)coek88(at)gmail(dot)com>, Amit Kapila <amit(dot)kapila16(at)gmail(dot)com>, Peter Eisentraut <peter(dot)eisentraut(at)2ndquadrant(dot)com>, pgsql-hackers <pgsql-hackers(at)postgresql(dot)org> |
| Subject: | Re: Getting server crash on Windows when using ICU collation |
| Date: | 2017-08-10 22:57:37 |
| Message-ID: | CAH2-Wz=nKd5P2_MAyMZLzTpTSN7b0OoO-TDLKPJ9pxUsoECn5A@mail.gmail.com |
| Views: | Raw Message | Whole Thread | Download mbox | Resend email |
| Thread: | |
| Lists: | pgsql-hackers |
On Thu, Aug 10, 2017 at 11:02 AM, Robert Haas <robertmhaas(at)gmail(dot)com> wrote:
> On Fri, Jun 23, 2017 at 1:14 AM, Ashutosh Sharma <ashu(dot)coek88(at)gmail(dot)com> wrote:
>> Okay, attached is the patch which first detects the platform type and
>> runs the uconv commands accordingly from the corresponding icu bin
>> path. Please have a look and let me know for any issues. Thanks.
>
> Should this be on the open items list?
Maybe. Peter E has a patch to fix the other issue (ICU 52 has a buggy
ucol_strcollUTF8()) by never using it on ICU versions prior to 53. He
posted this just yesterday.
This removes the existing configure test, replacing it with something
that will work just the same on Windows, presuming Peter also reverts
the commit that had ICU never use ucol_strcollUTF8() on Windows.
--
Peter Geoghegan
| From | Date | Subject | |
|---|---|---|---|
| Next Message | Michael Paquier | 2017-08-10 23:06:46 | Re: Server crash (FailedAssertion) due to catcache refcount mis-handling |
| Previous Message | Andres Freund | 2017-08-10 22:35:49 | Re: Thoughts on unit testing? |