Re: plperl version on the meson setup summary screen

From: vignesh C <vignesh21(at)gmail(dot)com>
To: Zharkov Roman <r(dot)zharkov(at)postgrespro(dot)ru>
Cc: Andrew Dunstan <andrew(at)dunslane(dot)net>, Nazir Bilal Yavuz <byavuz81(at)gmail(dot)com>, pgsql-hackers(at)lists(dot)postgresql(dot)org
Subject: Re: plperl version on the meson setup summary screen
Date: 2025-03-14 06:08:09
Message-ID: CALDaNm2fK3TS6q6qG4Lh=K6GXhW45WzhWWVEQ60o3wqauMbeaA@mail.gmail.com
Views: Raw Message | Whole Thread | Download mbox | Resend email
Thread:
Lists: pgsql-hackers

On Tue, 4 Feb 2025 at 12:07, Zharkov Roman <r(dot)zharkov(at)postgrespro(dot)ru> wrote:
>
> Hello,
>
> Here is a new patch version.
> I tried to use perl 'version' instead of 'api_versionstring' to sync
> with configure script.

One suggestion, there are many other external libraries for which we
don't display the version, can we include the version for them too:
@@ -3711,7 +3712,7 @@ if meson.version().version_compare('>=0.57')
'nls': libintl,
'openssl': ssl,
'pam': pam,
- 'plperl': perl_dep,
+ 'plperl': [perl_dep, perlversion],
'plpython': python3_dep,

Regards,
Vignesh

In response to

Responses

Browse pgsql-hackers by date

  From Date Subject
Next Message Alexander Pyhalov 2025-03-14 06:33:48 Re: SQLFunctionCache and generic plans
Previous Message Laurenz Albe 2025-03-14 05:58:22 Re: Reducing the log spam