Re: [PATCH 3/3] meson: Drop python3_prog

On Tue, Aug 15, 2023 at 02:45:37PM +0200, Michal Privoznik wrote: > Signed-off-by: Michal Privoznik <mprivozn(a)redhat.com&gt; > --- > docs/meson.build | 7 +++---- > meson.build | 4 ++-- > src/access/meson.build | 6 +++--- > src/meson.build | 43 +++++++++++++++++++---------------------- > src/network/meson.build | 2 +- > src/qemu/meson.build | 2 +- > tests/meson.build | 2 +- > 7 files changed, 31 insertions(+), 35 deletions(-) Not sure we can do this. In meson.build we have the following: required_programs = [ 'perl', 'python3', 'xmllint', 'xsltproc', ] foreach name : required_programs prog = find_program(name, dirs: libvirt_sbin_path) varname = name.underscorify() conf.set_quoted(varname.to_upper(), prog.full_path()) set_variable(&#39;@0(a)_prog&#39;.format(varname), prog) endforeach which will set the python3_prog variable and we use that as our python executable. I did a quick testing using the following meson.build file: ----------------------------------------------------------------------- project('mesonpy', 'c') required_programs = [ 'python3', ] foreach name : required_programs prog = find_program(name) varname = name.underscorify() set_variable(&#39;@0(a)_prog&#39;.format(varname), prog) endforeach res1 = run_command(python3_prog, 'script.py') res2 = run_command('script.py') warning(res1.stdout()) warning(res2.stdout()) ----------------------------------------------------------------------- with the script.py having the following: ----------------------------------------------------------------------- #!/usr/bin/env python3 import sys print(sys.version) ----------------------------------------------------------------------- and when I changed PATH to have python3 pointing to python3.12 but my system python is python3.11 I've got the following resutl: meson.build:16: WARNING: 3.12.0b4 (main, Jul 12 2023, 00:00:00) [GCC 13.1.1 20230614 (Red Hat 13.1.1-4)] meson.build:17: WARNING: 3.11.4 (main, Jun 7 2023, 00:00:00) [GCC 13.1.1 20230511 (Red Hat 13.1.1-2)] Don't remember the origin of the wrapper but my guess is that this was the main reason why we have it.