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
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 |