Re: MSVC build broken (again)

From: Tom Lane <tgl(at)sss(dot)pgh(dot)pa(dot)us>
To: "Magnus Hagander" <mha(at)sollentuna(dot)net>
Cc: pgsql-hackers(at)postgresql(dot)org, pgsql-patches(at)postgresql(dot)org
Subject: Re: MSVC build broken (again)
Date: 2006-10-03 20:36:13
Message-ID: 10793.1159907773@sss.pgh.pa.us
Views: Raw Message | Whole Thread | Download mbox | Resend email
Thread:
Lists: pgsql-hackers pgsql-patches

"Magnus Hagander" <mha(at)sollentuna(dot)net> writes:
>> Would it be possible to move the whole <crtdefs.h> block into win32.h?

> Nope, it needs to go before <stdio.h> and friends, unfortunatly.

OK, patch committed as-is then. The whole thing still looks awfully
icky though, particularly the way pg_config_os.h is included in one
place for WIN32 and a different place everywhere else.

Would it make sense to split win32.h into two files, one that's included
in the normal pg_config_os.h place and one included after the system
includes?

regards, tom lane

In response to

Responses

Browse pgsql-hackers by date

  From Date Subject
Next Message Tom Lane 2006-10-03 20:40:18 Re: vcbuild bison check
Previous Message Zdenek Kotala 2006-10-03 20:31:20 Re: [PATCHES] Generic Monitoring Framework with DTrace patch

Browse pgsql-patches by date

  From Date Subject
Next Message Tom Lane 2006-10-03 20:40:18 Re: vcbuild bison check
Previous Message Zdenek Kotala 2006-10-03 20:31:20 Re: [PATCHES] Generic Monitoring Framework with DTrace patch