Re: [libvirt PATCH 266/351] meson: tools: install libvirt-guests sysconf file

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

 



On Tue, Jul 28, 2020 at 10:32:20AM +0200, Peter Krempa wrote:
> On Thu, Jul 16, 2020 at 11:58:22 +0200, Pavel Hrdina wrote:
> > Signed-off-by: Pavel Hrdina <phrdina@xxxxxxxxxx>
> > ---
> >  tools/Makefile.am | 13 ++-----------
> >  tools/meson.build |  8 ++++++++
> >  2 files changed, 10 insertions(+), 11 deletions(-)
> 
> [...]
> 
> > diff --git a/tools/meson.build b/tools/meson.build
> > index 446831557e1..b95ced3728b 100644
> > --- a/tools/meson.build
> > +++ b/tools/meson.build
> > @@ -261,3 +261,11 @@ configure_file(
> >    install: true,
> >    install_dir: libexecdir,
> >  )
> > +
> > +if init_script == 'systemd'
> > +  install_data(
> > +    'libvirt-guests.sysconf',
> > +    install_dir: sysconfdir / 'sysconfig',
> > +    rename: 'libvirt-guests',
> > +  )
> > +endif
> 
> Arguably it's data at this point as it doesn't need to be modified, but
> shouldn't we use the config_file directive for any config file?

If by 'config_file' you mean 'configure_file' I don't think so. There is
no need to copy it into build directory, it just needs to be installed.

Pavel

Attachment: signature.asc
Description: PGP signature


[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