Re: Windows meson build

From: Vladlen Popolitov <v(dot)popolitov(at)postgrespro(dot)ru>
To: "Kohei Harikae (Fujitsu)" <harikae(dot)kohei(at)fujitsu(dot)com>
Cc: 'Andres Freund' <andres(at)anarazel(dot)de>, "'pgsql-hackers(at)lists(dot)postgresql(dot)org'" <pgsql-hackers(at)lists(dot)postgresql(dot)org>
Subject: Re: Windows meson build
Date: 2024-12-27 09:08:12
Message-ID: 29790e2f24e3579296e4a296ff8825a8@postgrespro.ru
Views: Raw Message | Whole Thread | Download mbox | Resend email
Thread:
Lists: pgsql-hackers

Vladlen Popolitov писал(а) 2024-12-27 09:57:
> Kohei Harikae (Fujitsu) писал(а) 2024-12-27 04:31:
>> Hi,
>>
>> Thank you for your advice.
>> I added a sample setting PKG_CONFIG_PATH based on your advice.
>>
>> How do you think about this?
>>
>> Regards,
>> Kohei Harikae
>>
>
> Hi,
> From my point of view it looks good. Actual information is added,
> excessive information is removed.
> I think in the phrase:
> "add the directory where the .pc file resides to PKG_CONFIG_PATH."
> better to add wording, that PKG_CONFIG_PATH is the list of directories
> separated
> by PATH separator ( ; in Windows, : in POSIX). Otherwise the reader has
> to
> search additional information about this variable in meson
> documentation.

Hi

I think, it is better to add the other options how to configure external
libraries, if they do not have .pc file:
add meson options defined by PostgreSQL for its configuration (gettext
library example)

-Dextra_include_dirs=c:\postgres\gettext\x64-windows\include,c:\otherlibs\include
-Dextra_lib_dirs=c:\postgres\gettext\x64-windows\lib,c:\otherlibs\lib

extra_include_dirs and extra_lib_dirs are options defined by PostgreSQL.
This options are
comma separated lists of directories, if every value does not contain
comma itself.

--
Best regards,

Vladlen Popolitov.

In response to

Browse pgsql-hackers by date

  From Date Subject
Next Message Heikki Linnakangas 2024-12-27 09:19:27 Re: pure parsers and reentrant scanners
Previous Message Vladlen Popolitov 2024-12-27 08:50:32 Re: cannot to compile extension by meson on windows