Re: broken regress tests on fedora 36
Michael Paquier <michael@paquier.xyz>
From: Michael Paquier <michael@paquier.xyz>
To: Andrew Dunstan <andrew@dunslane.net>
Cc: Pavel Stehule <pavel.stehule@gmail.com>, PostgreSQL Hackers <pgsql-hackers@lists.postgresql.org>
Date: 2022-06-07T23:59:10Z
Lists: pgsql-hackers
On Tue, Jun 07, 2022 at 10:54:07AM -0400, Andrew Dunstan wrote: > On 2022-06-07 Tu 08: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. > > or just replace '\.' with '[.,]' I was wondering about other separators actually: https://en.wikipedia.org/wiki/Decimal_separator#Usage_worldwide These two should be enough, though. So changing only that sounds fine by me. -- Michael
Commits
-
Fix portability issue in TAP tests of psql for locales
- ca899d98b4aa 15.0 landed