forked frompostgres/postgres
- Notifications
You must be signed in to change notification settings - Fork6
Commitce0da62
committed
Replace our hacked version of ax_pthread.m4 with latest upstream version.
Our version was different from the upstream version in that we tried to useall possible pthread-related flags that the compiler accepts, rather thanjust the first one that works. That change was made in commite48322a, to work-around a bug affecting GCCversions 3.2 and below (https://gcc.gnu.org/bugzilla/show_bug.cgi?id=8888),although we didn't realize that it was a GCC bug at the time. We hardly careabout that old GCC versions anymore, so we no longer need that workaround.This fixes the macro for compilers that print warnings with the chosenflags. That's pretty annoying on its own right, but it also inconspicuouslydisabled thread-safety, because we refused to use any pthread-related flagsif the compiler produced warnings. Max Filippov reported that problem whenlinking with uClibc and OpenSSL. The warnings-check was added because theworkaround for the GCC bug caused warnings otherwise, so it's no longerneeded either. We can just use the upstream version as is.If you really want to compile with GCC version 3.2 or older, you can stillwork-around it manually by setting PTHREAD_CFLAGS="-pthread -lpthread"manually on the configure command line.Backpatch to 9.5. I don't want to unnecessarily rock the boat on stablebranches, but 9.5 seems like fair game.1 parentd5f551a commitce0da62
File tree
5 files changed
+583
-234
lines changed- config
5 files changed
+583
-234
lines changedLines changed: 1 addition & 1 deletion
| Original file line number | Diff line number | Diff line change | |
|---|---|---|---|
| |||
1 | 1 | | |
2 | 2 | | |
3 | | - | |
| 3 | + | |
4 | 4 | | |
5 | 5 | | |
6 | 6 | | |
| |||
Lines changed: 0 additions & 171 deletions
This file was deleted.
0 commit comments
Comments
(0)