forked frompostgres/postgres
- Notifications
You must be signed in to change notification settings - Fork6
Commit7bc9a8b
committed
Fix warnings about declaration of environ on MinGW.
POSIX says that the global variable environ shouldn't be declared in aheader, and that you have to declare it yourself. MinGW declares it in<stdlib.h> with some macrology that messes up our declarations. VisualStudio doesn't warn (there are clues that it may also declare it, but ifso, apparently compatibly). Suppress our declarations, on MinGW only.This clears the last warnings on CI's optional MinGW task, and hopefullyon build farm animal fairywren too.Like1319997, no back-patch for now as it's not known to be breakinganything, and my humble goal is just to keep the MinGW build clean goingforward.Reviewed-by: Tom Lane <tgl@sss.pgh.pa.us> (earlier version)Discussion:https://postgr.es/m/CA%2BhUKGJLMh%2B6W5E4M_jSFb43gnrA_-Q6-%2BBf3HkBXyGfRFcBsQ%40mail.gmail.com1 parent48c142f commit7bc9a8b
File tree
3 files changed
+6
-0
lines changed- src
- backend
- postmaster
- utils/misc
- test/regress
3 files changed
+6
-0
lines changedLines changed: 2 additions & 0 deletions
Original file line number | Diff line number | Diff line change | |
---|---|---|---|
| |||
854 | 854 |
| |
855 | 855 |
| |
856 | 856 |
| |
| 857 | + | |
857 | 858 |
| |
| 859 | + | |
858 | 860 |
| |
859 | 861 |
| |
860 | 862 |
| |
|
Lines changed: 2 additions & 0 deletions
Original file line number | Diff line number | Diff line change | |
---|---|---|---|
| |||
23 | 23 |
| |
24 | 24 |
| |
25 | 25 |
| |
| 26 | + | |
26 | 27 |
| |
| 28 | + | |
27 | 29 |
| |
28 | 30 |
| |
29 | 31 |
| |
|
Lines changed: 2 additions & 0 deletions
Original file line number | Diff line number | Diff line change | |
---|---|---|---|
| |||
647 | 647 |
| |
648 | 648 |
| |
649 | 649 |
| |
| 650 | + | |
650 | 651 |
| |
| 652 | + | |
651 | 653 |
| |
652 | 654 |
| |
653 | 655 |
| |
|
0 commit comments
Comments
(0)