forked frompostgres/postgres
- Notifications
You must be signed in to change notification settings - Fork6
Commitaff45c8
committed
configure: don't probe for libldap_r if libldap is 2.5 or newer.
In OpenLDAP 2.5 and later, libldap itself is always thread-safe andthere's never a libldap_r. Our existing coding dealt with thatby assuming it wouldn't find libldap_r if libldap is thread-safe.But that rule fails to cope if there are multiple OpenLDAP versionsvisible, as is likely to be the case on macOS in particular. We'dend up using shiny new libldap in the backend and a hoary libldap_rin libpq.Instead, once we've found libldap, check if it's >= 2.5 (byprobing for a function introduced then) and don't bother lookingfor libldap_r if so. While one can imagine library setups thatthis'd still give the wrong answer for, they seem unlikely tooccur in practice.Per report from Peter Eisentraut. Back-patch to all supported branches.Discussion:https://postgr.es/m/fedacd7c-2a38-25c9-e7ff-dea549d0e979@enterprisedb.com1 parentb0d4b3c commitaff45c8
2 files changed
+20
-2
lines changedLines changed: 12 additions & 1 deletion
Original file line number | Diff line number | Diff line change | |
---|---|---|---|
| |||
13410 | 13410 |
| |
13411 | 13411 |
| |
13412 | 13412 |
| |
13413 |
| - | |
| 13413 | + | |
| 13414 | + | |
| 13415 | + | |
| 13416 | + | |
| 13417 | + | |
| 13418 | + | |
| 13419 | + | |
| 13420 | + | |
| 13421 | + | |
| 13422 | + | |
| 13423 | + | |
| 13424 | + | |
13414 | 13425 |
| |
13415 | 13426 |
| |
13416 | 13427 |
| |
|
Lines changed: 8 additions & 1 deletion
Original file line number | Diff line number | Diff line change | |
---|---|---|---|
| |||
1374 | 1374 |
| |
1375 | 1375 |
| |
1376 | 1376 |
| |
1377 |
| - | |
| 1377 | + | |
| 1378 | + | |
| 1379 | + | |
| 1380 | + | |
| 1381 | + | |
| 1382 | + | |
| 1383 | + | |
| 1384 | + | |
1378 | 1385 |
| |
1379 | 1386 |
| |
1380 | 1387 |
| |
|
0 commit comments
Comments
(0)