Re: [PATCH] meson: don't install sysconf files unconditionally

[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]

 



On Wed, Aug 26, 2020 at 06:00:26PM +0400, Roman Bogorodskiy wrote:
> There's no need to install sysconf files when init script installation
> was not requested, i.e. when configured with init_script=none.
> 
> Signed-off-by: Roman Bogorodskiy <bogorodskiy@xxxxxxxxx>
> ---
>  src/meson.build | 16 +++++++++-------
>  1 file changed, 9 insertions(+), 7 deletions(-)

Reviewed-by: Daniel P. Berrangé <berrange@xxxxxxxxxx>

and desirable for this release


> 
> diff --git a/src/meson.build b/src/meson.build
> index d058cec2e4..5d8deaf548 100644
> --- a/src/meson.build
> +++ b/src/meson.build
> @@ -858,13 +858,15 @@ if conf.has('WITH_LIBVIRTD')
>    endif
>  endif
>  
> -foreach sysconf : sysconf_files
> -  install_data(
> -    sysconf['file'],
> -    install_dir: sysconfdir / 'sysconfig',
> -    rename: [ sysconf['name'] ],
> -  )
> -endforeach
> +if init_script != 'none'
> +  foreach sysconf : sysconf_files
> +    install_data(
> +      sysconf['file'],
> +      install_dir: sysconfdir / 'sysconfig',
> +      rename: [ sysconf['name'] ],
> +    )
> +  endforeach
> +endif
>  
>  if conf.has('WITH_DTRACE_PROBES')
>    custom_target(
> -- 
> 2.27.0
> 

Regards,
Daniel
-- 
|: https://berrange.com      -o-    https://www.flickr.com/photos/dberrange :|
|: https://libvirt.org         -o-            https://fstop138.berrange.com :|
|: https://entangle-photo.org    -o-    https://www.instagram.com/dberrange :|




[Index of Archives]     [Virt Tools]     [Libvirt Users]     [Lib OS Info]     [Fedora Users]     [Fedora Desktop]     [Fedora SELinux]     [Big List of Linux Books]     [Yosemite News]     [KDE Users]     [Fedora Tools]

  Powered by Linux