forked frompostgres/postgres
- Notifications
You must be signed in to change notification settings - Fork6
Commit1758d42
committed
Require ucrt if using MinGW.
Historically we tolerated the absence of various C runtime libraryfeatures for the benefit of the MinGW tool chain, because it usedancient msvcrt.dll for a long period of time. It now uses ucrt bydefault (like Windows 10+, Visual Studio 2015+), and that's the onlyconfiguration we're testing.In practice, we effectively required ucrt already in PostgreSQL 17, whencommit8d9a9f0 required _create_locale etc, first available inmsvcr120.dll (Visual Studio 2013, the last of the pre-ucrt series ofruntimes), and for MinGW users that practically meant ucrt because itwas difficult or impossible to use msvcr120.dll. That may even not havebeen the first such case, but old MinGW configurations had alreadydropped off our testing radar so we weren't paying much attention.This commit formalizes the requirement. It also removes a couple ofobsolete comments that discussed msvcrt.dll limitations, and some testsof !defined(_MSC_VER) to imply msvcrt.dll. There are many moreanachronisms, but it'll take some time to figure out how to remove themall. APIs affected relate to locales, UTF-8, threads, large files andmore.Thanks to Peter Eisentraut for the documentation change. It's notreally necessary to talk about ucrt explicitly in such a short section,since it's the default for MinGW-w64 and MSYS2. It's enough to prunereferences and broken links to much older tools.Reviewed-by: Peter Eisentraut <peter@eisentraut.org>Discussion:https://postgr.es/m/d9e7731c-ca1b-477c-9298-fa51e135574a%40eisentraut.org1 parentf1da075 commit1758d42
File tree
2 files changed
+5
-74
lines changed- doc/src/sgml
- src/backend/utils/adt
2 files changed
+5
-74
lines changedLines changed: 5 additions & 55 deletions
Original file line number | Diff line number | Diff line change | |
---|---|---|---|
| |||
3582 | 3582 |
| |
3583 | 3583 |
| |
3584 | 3584 |
| |
3585 |
| - | |
3586 |
| - | |
3587 |
| - | |
3588 |
| - | |
3589 |
| - | |
3590 |
| - | |
3591 |
| - | |
3592 |
| - | |
3593 |
| - | |
3594 |
| - | |
3595 |
| - | |
3596 |
| - | |
3597 |
| - | |
3598 |
| - | |
3599 |
| - | |
3600 |
| - | |
3601 |
| - | |
3602 |
| - | |
3603 |
| - | |
3604 |
| - | |
3605 |
| - | |
3606 |
| - | |
3607 |
| - | |
3608 |
| - | |
3609 |
| - | |
3610 |
| - | |
3611 |
| - | |
| 3585 | + | |
| 3586 | + | |
| 3587 | + | |
3612 | 3588 |
| |
3613 | 3589 |
| |
3614 | 3590 |
| |
| |||
3838 | 3814 |
| |
3839 | 3815 |
| |
3840 | 3816 |
| |
| 3817 | + | |
| 3818 | + | |
3841 | 3819 |
| |
3842 |
| - | |
3843 |
| - | |
3844 |
| - | |
3845 |
| - | |
3846 |
| - | |
3847 |
| - | |
3848 |
| - | |
3849 |
| - | |
3850 |
| - | |
3851 |
| - | |
3852 |
| - | |
3853 |
| - | |
3854 |
| - | |
3855 |
| - | |
3856 |
| - | |
3857 |
| - | |
3858 |
| - | |
3859 |
| - | |
3860 |
| - | |
3861 |
| - | |
3862 |
| - | |
3863 |
| - | |
3864 |
| - | |
3865 |
| - | |
3866 |
| - | |
3867 |
| - | |
3868 |
| - | |
3869 |
| - | |
3870 | 3820 |
| |
3871 | 3821 |
| |
3872 | 3822 |
| |
|
Lines changed: 0 additions & 19 deletions
Original file line number | Diff line number | Diff line change | |
---|---|---|---|
| |||
1017 | 1017 |
| |
1018 | 1018 |
| |
1019 | 1019 |
| |
1020 |
| - | |
1021 |
| - | |
1022 |
| - | |
1023 |
| - | |
1024 |
| - | |
1025 |
| - | |
1026 |
| - | |
1027 | 1020 |
| |
1028 | 1021 |
| |
1029 | 1022 |
| |
1030 | 1023 |
| |
1031 | 1024 |
| |
1032 | 1025 |
| |
1033 | 1026 |
| |
1034 |
| - | |
1035 |
| - | |
1036 | 1027 |
| |
1037 | 1028 |
| |
1038 | 1029 |
| |
| |||
1201 | 1192 |
| |
1202 | 1193 |
| |
1203 | 1194 |
| |
1204 |
| - | |
1205 |
| - | |
1206 |
| - | |
1207 |
| - | |
1208 |
| - | |
1209 |
| - | |
1210 |
| - | |
1211 |
| - | |
1212 |
| - | |
1213 |
| - | |
1214 | 1195 |
| |
1215 | 1196 |
| |
1216 | 1197 |
| |
|
0 commit comments
Comments
(0)