forked frompostgres/postgres
- Notifications
You must be signed in to change notification settings - Fork6
Commite52a44b
committed
Skip citext_utf8 test on Windows.
Back-patch of commitcff4e5a to 15 and 16, per request from OlegTselebrovskiy. Original commit message:On other Windows build farm animals it is already skipped because theydon't use UTF-8 encoding. On "hamerkop", UTF-8 is used, and then thetest fails.It is not clear to me (a non-Windows person looking only at buildfarmevidence) whether Windows is less sophisticated than other OSes anddoesn't know how to downcase Turkish İ with the standard Unicodedatabase, or if it is more sophisticated than other systems and useslocale-specific behavior like ICU does.Whichever the reason, the result is the same: we need to skip the teston Windows, just as we already do for ICU, at least until aWindows-savvy developer comes up with a better idea. The technique fordetecting the OS is borrowed from collate.windows.win1252.sql.This was anticipated by commitc2e8bd2, but the problem only surfacedwhen Windows build farm animals started using Meson.Reviewed-by: Tom Lane <tgl@sss.pgh.pa.us>Discussion:https://postgr.es/m/CA%2BhUKGJ1LeC3aE2qQYTK95rFVON3ZVoTQpTKJqxkHdtEyawH4A%40mail.gmail.com1 parentd7bc9c9 commite52a44b
File tree
3 files changed
+3
-0
lines changed- contrib/citext
- expected
- sql
3 files changed
+3
-0
lines changedOriginal file line number | Diff line number | Diff line change | |
---|---|---|---|
| |||
3 | 3 |
| |
4 | 4 |
| |
5 | 5 |
| |
| 6 | + | |
6 | 7 |
| |
7 | 8 |
| |
8 | 9 |
| |
|
Original file line number | Diff line number | Diff line change | |
---|---|---|---|
| |||
3 | 3 |
| |
4 | 4 |
| |
5 | 5 |
| |
| 6 | + | |
6 | 7 |
| |
7 | 8 |
| |
8 | 9 |
| |
|
Original file line number | Diff line number | Diff line change | |
---|---|---|---|
| |||
4 | 4 |
| |
5 | 5 |
| |
6 | 6 |
| |
| 7 | + | |
7 | 8 |
| |
8 | 9 |
| |
9 | 10 |
| |
|
0 commit comments
Comments
(0)