Hello,

Would it be convinient to show the plperl version on the meson setup summary screen? Now it displays only 'YES' or 'NO'. This is the expected behavior of
the meson build system and explains in its source code:
https://github.com/mesonbuild/meson/blob/5f0bd8ff1e7fc43199d4b371fc4625f80baba810/mesonbuild/dependencies/base.py#L311C15-L311C27
With the attached patch we can see the "perlversion" in the summary
information table. But without a beautiful console colorize.

Additionaly, we found out that "perlversion" gets from the perl
"api_versionstring" config variable. When the configure script parses the
"perl -v" output:

pgac_perl_version=`$PERL -v 2>/dev/null | sed -n 's/This is perl.*v[a-...

Is this behavior correct?

--
Roman Zharkov
diff --git a/meson.build b/meson.build
index 58e67975e85..cdee5880efa 100644
--- a/meson.build
+++ b/meson.build
@@ -990,6 +990,7 @@ endif
 
 perlopt = get_option('plperl')
 perl_dep = not_found_dep
+perlversion = ''
 if not perlopt.disabled()
   perl_may_work = true
 
@@ -3716,7 +3717,7 @@ if meson.version().version_compare('>=0.57')
       'nls': libintl,
       'openssl': ssl,
       'pam': pam,
-      'plperl': perl_dep,
+      'plperl': [perl_dep, perlversion],
       'plpython': python3_dep,
       'pltcl': tcl_dep,
       'readline': readline,
@@ -3727,6 +3728,7 @@ if meson.version().version_compare('>=0.57')
       'zstd': zstd,
     },
     section: 'External libraries',
+    list_sep: ' ',
   )
 
 endif

Reply via email to