Re: [PATCH] fix ICU explorer link in locale documentation

2021-11-02 Thread Bruce Momjian
On Tue, Nov 2, 2021 at 02:31:29PM +0100, Laurenz Albe wrote: > On Mon, 2021-10-04 at 17:11 -0400, Bruce Momjian wrote: > > Patch applied back to PG 10, where ICU was added.  Thanks everyone for > > the work on this. > > Thanks! > > I think you can close the commitfest entry: > https://commitfest

Re: [PATCH] fix ICU explorer link in locale documentation

2021-11-02 Thread Laurenz Albe
On Mon, 2021-10-04 at 17:11 -0400, Bruce Momjian wrote: > Patch applied back to PG 10, where ICU was added.  Thanks everyone for > the work on this. Thanks! I think you can close the commitfest entry: https://commitfest.postgresql.org/35/3059/ Yours, Laurenz Albe

Re: [PATCH] fix ICU explorer link in locale documentation

2021-10-04 Thread Bruce Momjian
On Wed, Sep 29, 2021 at 06:58:55PM +0700, Anton Voloshin wrote: > On 29/09/2021 17:13, Laurenz Albe wrote: > > I had a look at that and tried to build the software from source. > > It requires you to build ICU, and even then it failed to build for me. > > The build documentation is not very detaile

Re: [PATCH] fix ICU explorer link in locale documentation

2021-09-29 Thread Anton Voloshin
On 29/09/2021 17:13, Laurenz Albe wrote: I had a look at that and tried to build the software from source. It requires you to build ICU, and even then it failed to build for me. The build documentation is not very detailed. So I think we should not link to that. I noticed that two more external

Re: [PATCH] fix ICU explorer link in locale documentation

2021-09-29 Thread Laurenz Albe
On Mon, 2021-09-27 at 14:31 -0400, Bruce Momjian wrote: > On Fri, Sep  3, 2021 at 09:39:45PM +0200, Laurenz Albe wrote: > > On Thu, 2021-05-27 at 08:59 +0200, Laurenz Albe wrote: > > > On Thu, 2021-05-27 at 11:10 +0700, Anton Voloshin wrote: > > > > So I guess the https://icu4c-demos.unicode.org/ic

Re: [PATCH] fix ICU explorer link in locale documentation

2021-09-27 Thread Bruce Momjian
On Fri, Sep 3, 2021 at 09:39:45PM +0200, Laurenz Albe wrote: > On Thu, 2021-05-27 at 08:59 +0200, Laurenz Albe wrote: > > On Thu, 2021-05-27 at 11:10 +0700, Anton Voloshin wrote: > > > So I guess the https://icu4c-demos.unicode.org/icu-bin/locexp link is > > > still the proper one, and eventually

Re: [PATCH] fix ICU explorer link in locale documentation

2021-09-03 Thread Laurenz Albe
On Thu, 2021-05-27 at 08:59 +0200, Laurenz Albe wrote: > On Thu, 2021-05-27 at 11:10 +0700, Anton Voloshin wrote: > > So I guess the https://icu4c-demos.unicode.org/icu-bin/locexp link is > > still the proper one, and eventually it will be fixed. Anyway, the old > > link, https://ssl.icu-project.

Re: [PATCH] fix ICU explorer link in locale documentation

2021-05-27 Thread Laurenz Albe
On Thu, 2021-05-27 at 11:10 +0700, Anton Voloshin wrote: > On 18/05/2021 03:46, Magnus Hagander wrote: > > https://icu4c-demos.unicode.org/icu-bin/locexp generates a 404 for me now. > > > > This might be something temporary though, because AFAICT it's where > > you end up when you follow the links

Re: [PATCH] fix ICU explorer link in locale documentation

2021-05-26 Thread Anton Voloshin
On 18/05/2021 03:46, Magnus Hagander wrote: https://icu4c-demos.unicode.org/icu-bin/locexp generates a 404 for me now. This might be something temporary though, because AFAICT it's where you end up when you follow the links from the ICU git repo... Yes, same 404 for me. This error has been rep

Re: [PATCH] fix ICU explorer link in locale documentation

2021-05-17 Thread Magnus Hagander
On Sun, Apr 4, 2021 at 9:53 AM Anton Voloshin wrote: > > Hello, > > Existing Postgres documentation states in doc/src/sgml/charset.sgml: > The https://ssl.icu-project.org/icu-bin/locexp";>ICU Locale > Explorer can be used to check the details of a particular locale > definition. > > But the link i

[PATCH] fix ICU explorer link in locale documentation

2021-04-04 Thread Anton Voloshin
Hello, Existing Postgres documentation states in doc/src/sgml/charset.sgml: The https://ssl.icu-project.org/icu-bin/locexp";>ICU Locale Explorer can be used to check the details of a particular locale definition. But the link is broken (SSL certificate does not include ssl.icu-project.org as al