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

Поиск
Список
Период
Сортировка
От Michael Paquier
Тема Re: [PATCH] Fix PostgreSQL server build and install problems underMSYS2
Дата
Msg-id 20191112035049.GH1549@paquier.xyz
обсуждение исходный текст
Ответ на Re: [PATCH] Fix PostgreSQL server build and install problems under MSYS2  (Tom Lane <tgl@sss.pgh.pa.us>)
Ответы Re: [PATCH] Fix PostgreSQL server build and install problems under MSYS2
Список pgsql-hackers
On Mon, Nov 11, 2019 at 10:56:51AM -0500, Tom Lane wrote:
> 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.

Yeah.  We have fairywen in the buildfarm but it does not compile with
GSSAPI and LLVM.  If we were to do something, it would be better to
separate those changes into minimum, separate patches, with one each
for each library dependency you are trying to fix so as they can be
evaluated separately.  We should also have a buildfarm member on that
if some of those changes actually make sense and are
platform-dependent.

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

I am doubtful that the changes in c.h, elog.h, win32_port.h,
miscadmin.h and src/backend/jit/llvm/Makefile are actually needed
thanks to fairywen.

+#if defined(HAVE_GSS_API_H) && !defined(GSS_DLLIMP)
+static gss_OID_desc GSS_C_NT_USER_NAME_desc =
+{10, (void *) "\x2a\x86\x48\x86\xf7\x12\x01\x02\x01\x02"};
This also deserves an explanation.
--
Michael

Вложения

В списке pgsql-hackers по дате отправления:

Предыдущее
От: Michael Paquier
Дата:
Сообщение: Re: [BUG FIX] Uninitialized var fargtypes used.
Следующее
От: Thomas Munro
Дата:
Сообщение: Re: PHJ file leak.