From: | Peter Eisentraut <peter(dot)eisentraut(at)enterprisedb(dot)com> |
---|---|
To: | Michael Paquier <michael(at)paquier(dot)xyz>, Pavel Stehule <pavel(dot)stehule(at)gmail(dot)com> |
Cc: | PostgreSQL Hackers <pgsql-hackers(at)lists(dot)postgresql(dot)org> |
Subject: | Re: broken regress tests on fedora 36 |
Date: | 2022-06-09 13:41:08 |
Message-ID: | 5115fa52-224e-d731-2042-2b00789a2748@enterprisedb.com |
Views: | Raw Message | Whole Thread | Download mbox | Resend email |
Thread: | |
Lists: | pgsql-hackers |
On 07.06.22 14:56, Michael Paquier wrote:
> On Tue, Jun 07, 2022 at 10:52:45AM +0200, Pavel Stehule wrote:
>> # Failed test '\timing with query error: timing output appears'
>> # at t/001_basic.pl line 95.
>> # 'Time: 0,293 ms'
>> # doesn't match '(?^m:^Time: \d+\.\d\d\d ms)'
>> # Looks like you failed 2 tests of 58.
>
> Fun. The difference is in the separator: dot vs comma. This should
> fail with French the same way. Perhaps it would fail differently in
> other languages? There is no need to be that precise with the regex
> IMO, so I would just cut the regex with the number, checking only the
> unit at the end.
Shouldn't we reset the locale setting (LC_NUMERIC?) to a known value?
We clearly already do that for other categories, or it wouldn't say "Time:".
From | Date | Subject | |
---|---|---|---|
Next Message | Tom Lane | 2022-06-09 13:50:54 | |
Previous Message | Robert Haas | 2022-06-09 13:40:29 | Re: BTMaxItemSize seems to be subtly incorrect |