public inbox for [email protected]  

help / color / mirror / Atom feed

From: Peter Eisentraut <[email protected]>
To: Vladlen Popolitov <[email protected]>
To: [email protected]
Cc: Kohei Harikae <[email protected]>
Subject: Re: Windows meson build
Date: Fri, 27 Mar 2026 15:13:36 +0100
Message-ID: <[email protected]> (raw)
In-Reply-To: <[email protected]>
References: <TYVPR01MB113306AB19264E89AD360A40EE4522@TYVPR01MB11330.jpnprd01.prod.outlook.com>
	<TYVPR01MB113302864D12BF0AF9F12BE4EE4E32@TYVPR01MB11330.jpnprd01.prod.outlook.com>
	<173944432278.159115.3197178459315468186.pgcf@coridan.postgresql.org>
	<[email protected]>

On 13.02.25 12:38, Vladlen Popolitov wrote:
> Vladlen Popolitov писал(а) 2025-02-13 17:58:
>> The following review has been posted through the commitfest application:
>> make installcheck-world:  tested, failed
>> Implements feature:       tested, failed
>> Spec compliant:           tested, failed
>> Documentation:            tested, failed
> Hi!
> 
>   Previous email has text "failed". It is generated by "send review" 
> function,
> I do not know, how to change it (tried in other browsers).
> 
> Nothing failed in this patch.

This patch is pretty old but it was still hanging around as open in the 
commitfest app.

I realize that the way the PostgreSQL build process finds all the 
dependencies is pretty complicated right now.  With configure some of 
them are found the old way using CPPFLAGS/LDFLAGS, some via specific 
foo-config scripts, some via pkg-config, some via specific custom code. 
With meson, similarly, but the actual details are different in each 
case.  Maybe we should document all of this accurately, or maybe it 
would be unmaintainable.  In any case, I don't think your patch has 
enough coverage to make this better, it just describes some specific 
scenario that works for you, and it moves some text out of context where 
it doesn't make sense anymore.  Maybe this kind of information is better 
in a how-to document or blog post, and from there we could generalize 
the information and maybe feed it back into the documentation.

view thread (3+ messages)

reply

Reply instructions:

You may reply publicly to this message via plain-text email
using any one of the following methods:

* Reply to all the recipients using the --to and --cc options:
  reply via email

  To: [email protected]
  Cc: [email protected], [email protected], [email protected], [email protected]
  Subject: Re: Windows meson build
  In-Reply-To: <[email protected]>

* Save the following mbox file, import it into your mail client,
  and reply-to-all from there: mbox

This inbox is served by agora; see mirroring instructions
for how to clone and mirror all data and code used for this inbox