Finish the work to stop hardcoding paths at build time

[PATCH 17/20] meson: stop setting conf var for optional programs

Posted by Daniel P. Berrangé via Devel 11 months, 3 weeks ago

From: Daniel P. Berrangé <berrange@redhat.com>

There is no reference to AUGPARSE, BLACK, FLAKE8, PDWTAGS or PYTEST conf
variables anywhere, only the ${name}_prog meson variables are used.

Signed-off-by: Daniel P. Berrangé <berrange@redhat.com>
---
 meson.build | 1 -
 1 file changed, 1 deletion(-)

diff --git a/meson.build b/meson.build
index 4223f5ab86..68b955a02c 100644
--- a/meson.build
+++ b/meson.build
@@ -864,7 +864,6 @@ foreach name : optional_programs
     missing_optional_programs += [ name ]
   endif
 
-  conf.set_quoted(varname.to_upper(), prog_path)
   set_variable('@0@_prog'.format(varname), prog)
 endforeach
 
-- 
2.49.0

Re: [PATCH 17/20] meson: stop setting conf var for optional programs

Posted by Peter Krempa via Devel 11 months, 3 weeks ago

On Tue, Apr 29, 2025 at 12:14:17 +0100, Daniel P. Berrangé via Devel wrote:
> From: Daniel P. Berrangé <berrange@redhat.com>
> 
> There is no reference to AUGPARSE, BLACK, FLAKE8, PDWTAGS or PYTEST conf
> variables anywhere, only the ${name}_prog meson variables are used.
> 
> Signed-off-by: Daniel P. Berrangé <berrange@redhat.com>
> ---
>  meson.build | 1 -
>  1 file changed, 1 deletion(-)

Reviewed-by: Peter Krempa <pkrempa@redhat.com>