Re: Bogus HAVE_DECL_FOO entries in msvc/Solution.pm

2021-07-13 Thread Peter Eisentraut
On 13.07.21 09:53, Michael Paquier wrote: I was thinking to just do the easiest move and fix this issue down to 13, not bothering about older branches :p Looking at the commit, a backpatch is not that complicated and it is possible to check the generation of pg_config.h on non-MSVC environments

Re: Bogus HAVE_DECL_FOO entries in msvc/Solution.pm

2021-07-13 Thread Tom Lane
Michael Paquier writes: > On Tue, Jul 13, 2021 at 12:25:06AM -0400, Tom Lane wrote: >> That's easy enough in v13 and up, which have 8f4fb4c64 so that >> Solution.pm looks like this. We could make it consistent in older >> branches by manually hacking pg_config.h.win32 ... but I'm wondering >> if

Re: Bogus HAVE_DECL_FOO entries in msvc/Solution.pm

2021-07-13 Thread Michael Paquier
On Tue, Jul 13, 2021 at 12:25:06AM -0400, Tom Lane wrote: > That's easy enough in v13 and up, which have 8f4fb4c64 so that > Solution.pm looks like this. We could make it consistent in older > branches by manually hacking pg_config.h.win32 ... but I'm wondering > if the smarter plan wouldn't be to

Re: Bogus HAVE_DECL_FOO entries in msvc/Solution.pm

2021-07-12 Thread Tom Lane
Michael Paquier writes: > On Mon, Jul 12, 2021 at 07:46:32PM -0400, Tom Lane wrote: >> Autoconf's AC_CHECK_DECLS always defines HAVE_DECL_whatever >> as 1 or 0, but some of the entries in msvc/Solution.pm show >> such symbols as "undef" instead. Shouldn't we fix it as >> per attached? This is pr

Re: Bogus HAVE_DECL_FOO entries in msvc/Solution.pm

2021-07-12 Thread Peter Eisentraut
On 13.07.21 01:46, Tom Lane wrote: Autoconf's AC_CHECK_DECLS always defines HAVE_DECL_whatever as 1 or 0, but some of the entries in msvc/Solution.pm show such symbols as "undef" instead. Shouldn't we fix it as per attached? This is probably only cosmetic at the moment, but it could bite us som

Re: Bogus HAVE_DECL_FOO entries in msvc/Solution.pm

2021-07-12 Thread Michael Paquier
On Mon, Jul 12, 2021 at 07:46:32PM -0400, Tom Lane wrote: > Autoconf's AC_CHECK_DECLS always defines HAVE_DECL_whatever > as 1 or 0, but some of the entries in msvc/Solution.pm show > such symbols as "undef" instead. Shouldn't we fix it as > per attached? This is probably only cosmetic at the mom

Bogus HAVE_DECL_FOO entries in msvc/Solution.pm

2021-07-12 Thread Tom Lane
Autoconf's AC_CHECK_DECLS always defines HAVE_DECL_whatever as 1 or 0, but some of the entries in msvc/Solution.pm show such symbols as "undef" instead. Shouldn't we fix it as per attached? This is probably only cosmetic at the moment, but it could bite us someday if someone wrote a complex condi