forked frompostgres/postgres
- Notifications
You must be signed in to change notification settings - Fork6
Commit51057fe
committed
Fix up ecpg's configuration so it handles "long long int" in MSVC builds.
Although configure-based builds correctly define HAVE_LONG_LONG_INT whenappropriate (in both pg_config.h and ecpg_config.h), builds using the MSVCscripts failed to do so. This currently has no impact on the backend,since it uses that symbol nowhere; but it does prevent ecpg fromsupporting "long long int". Fix that.Also, adjust Solution.pm so that in the constructed ecpg_config.h file,the "#if (_MSC_VER > 1200)" covers only the LONG_LONG_INT-related#defines, not the whole file. AFAICS this was a thinko on somebody'spart: ENABLE_THREAD_SAFETY should always be defined in Windows builds,and in branches using USE_INTEGER_DATETIMES, the setting of that shouldn'tdepend on the compiler version either. If I'm wrong, I imagine thebuildfarm will say so.Per bug #15080 from Jonathan Allen; issue diagnosed by Michael Meskesand Andrew Gierth. Back-patch to all supported branches.Discussion:https://postgr.es/m/151935568942.1461.14623890240535309745@wrigleys.postgresql.org1 parente98a4de commit51057fe
2 files changed
+7
-1
lines changedLines changed: 5 additions & 0 deletions
Original file line number | Diff line number | Diff line change | |
---|---|---|---|
| |||
236 | 236 |
| |
237 | 237 |
| |
238 | 238 |
| |
| 239 | + | |
| 240 | + | |
| 241 | + | |
| 242 | + | |
| 243 | + | |
239 | 244 |
| |
240 | 245 |
| |
241 | 246 |
| |
|
Lines changed: 2 additions & 1 deletion
Original file line number | Diff line number | Diff line change | |
---|---|---|---|
| |||
424 | 424 |
| |
425 | 425 |
| |
426 | 426 |
| |
| 427 | + | |
427 | 428 |
| |
| 429 | + | |
428 | 430 |
| |
429 | 431 |
| |
430 |
| - | |
431 | 432 |
| |
432 | 433 |
| |
433 | 434 |
| |
|
0 commit comments
Comments
(0)