Re: MSVC: Improve warning options set - Mailing list pgsql-hackers

From Peter Eisentraut
Subject Re: MSVC: Improve warning options set
Date
Msg-id 406a52f4-36d2-4e3a-85de-d8d6ab9b7787@eisentraut.org
Whole thread Raw
In response to Re: MSVC: Improve warning options set  (Thomas Munro <thomas.munro@gmail.com>)
List pgsql-hackers
On 08.11.25 22:40, Thomas Munro wrote:
> Unfortunately
> it didn't ever seem to become unbuildable, but apparently things break
> in undiagnosed ways at runtime (at a guess it might have some API
> calls that are stubbed out with empty implementations or something
> like that, but there is zero reason to investigate that, it's dead).
> What we should do to make this clearer and avoid spurious problem
> reports is error out unless you're on UCRT, but a patch for that got
> stuck waiting for the Debian images used on CI to be upgraded to
> Debian trixie, because that shipped the necessary newer
> MinGW/headers/etc in its cross-build packages.  That has now happened,
> so we should probably go ahead with something like the patch I posted
> here:
> 
> https://www.postgresql.org/message-id/flat/CA%2BhUKG%2BQJv- 
> g7C2APVV7O_jEJkxH1AmvgAe8X2vDR8mRdSKn3A%40mail.gmail.com#e6d0c91e2f59e6e39eb61095da4cc598
> 
> In theory we could even back-patch that to 18, since we already know
> it won't fully work and we already declared that we don't support it.
> Or we could just let sleeping dogs lie and do that for 19.

When you build master under the msys2 msvcrt environment now, various 
regression tests fail, related to floating point differences and locales 
not being found.  So I think this regulates itself and we don't really 
need to do anything further.




pgsql-hackers by date:

Previous
From: Andrei Lepikhov
Date:
Subject: Re: Sequence Access Methods, round two
Next
From: Andres Freund
Date:
Subject: Re: Move SLRU_PAGES_PER_SEGMENT to pg_config_manual.h