On Fri, Apr 7, 2023 at 5:00 PM Tom Lane <[email protected]> wrote: > Alexander Korotkov <[email protected]> writes: > > On Thu, Apr 6, 2023 at 8:18 PM Tom Lane <[email protected]> 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
