Re: [PATCH] Fix PostgreSQL server build and install problems under MSYS2

From: Tom Lane <tgl(at)sss(dot)pgh(dot)pa(dot)us>
To: Гурам Дука <guram(dot)duka(at)gmail(dot)com>
Cc: pgsql-hackers(at)postgresql(dot)org
Subject: Re: [PATCH] Fix PostgreSQL server build and install problems under MSYS2
Date: 2019-11-11 15:56:51
Message-ID: 1918.1573487811@sss.pgh.pa.us
Views: Raw Message | Whole Thread | Download mbox | Resend email
Thread:
Lists: pgsql-hackers

=?UTF-8?B?0JPRg9GA0LDQvCDQlNGD0LrQsA==?= <guram(dot)duka(at)gmail(dot)com> writes:
> I made a patch fixing build and install problems under MSYS2, including
> llvmjit.

This seems like it probably breaks a lot of other cases along the way.
Why have you made all these #if tests dependent on defined(__cplusplus)?
That's surely not specific to MSYS2. (I'm a bit bemused by the idea
that our code compiles at all on a C++ compiler; we have not tried
to make the .c files C++-clean. But if it does work, this probably
breaks it for non-Windows cases.)

The GSSAPI changes seem like they might be better considered
separately from the basic problem of getting a working MSYS2 build.

In any case, you need to explain these changes individually,
not expect that we're just going to adopt them without questions.

regards, tom lane

In response to

Responses

Browse pgsql-hackers by date

  From Date Subject
Next Message Liudmila Mantrova 2019-11-11 16:00:22 Re: Does 'instead of delete' trigger support modification of OLD
Previous Message Tom Lane 2019-11-11 15:46:07 Re: psql \d for wide tables / pattern for individual columns