From: | Alexander Korotkov <aekorotkov(at)gmail(dot)com> |
---|---|
To: | Tom Lane <tgl(at)sss(dot)pgh(dot)pa(dot)us> |
Cc: | pgsql-committers(at)lists(dot)postgresql(dot)org, Andrew Dunstan <andrew(at)dunslane(dot)net> |
Subject: | Re: pgsql: psql: add an optional execution-count limit to \watch. |
Date: | 2023-04-07 14:06:34 |
Message-ID: | CAPpHfdtgQVRZOwov-Ei4Rh=gQtFE8Hb9uKkO1d_XYmQ2XEXt0g@mail.gmail.com |
Views: | Raw Message | Whole Thread | Download mbox | Resend email |
Thread: | |
Lists: | pgsql-committers pgsql-hackers |
On Fri, Apr 7, 2023 at 5:00 PM Tom Lane <tgl(at)sss(dot)pgh(dot)pa(dot)us> wrote:
> Alexander Korotkov <aekorotkov(at)gmail(dot)com> writes:
> > On Thu, Apr 6, 2023 at 8:18 PM Tom Lane <tgl(at)sss(dot)pgh(dot)pa(dot)us> wrote:
> >> psql: add an optional execution-count limit to \watch.
>
> > This commit makes tests fail for me. psql parses 'i' option of
> > '\watch' using locale-aware strtod(), but 001_basic.pl uses hard-coded
> > decimal separator.
>
> Huh, yeah, I see it too if I set LANG=ru_RU.utf8 before running psql's
> TAP tests. It seems unfortunate that none of the buildfarm has noticed
> this. I guess all the TAP tests are run under C locale?
I wonder if we can setup as least some buildfarm members to exercise
TAP tests on non-C locales.
> > The proposed fix is attached.
>
> LGTM, will push in a bit (unless you want to?)
Please push.
------
Regards,
Alexander Korotkov
From | Date | Subject | |
---|---|---|---|
Next Message | Tom Lane | 2023-04-07 14:35:14 | pgsql: Fix locale-dependent test case. |
Previous Message | Tom Lane | 2023-04-07 14:00:23 | Re: pgsql: psql: add an optional execution-count limit to \watch. |
From | Date | Subject | |
---|---|---|---|
Next Message | Greg Stark | 2023-04-07 14:20:56 | Re: Commitfest 2023-03 starting tomorrow! |
Previous Message | Tom Lane | 2023-04-07 14:00:23 | Re: pgsql: psql: add an optional execution-count limit to \watch. |