[libvirt PATCH] meson: Fix libvirtd|virtproxyd socket prefixes

[Pavel Hrdina]

On Tue, Aug 04, 2020 at 09:48:58AM +0200, Pavel Hrdina wrote:
> On Tue, Aug 04, 2020 at 09:30:42AM +0200, Erik Skultety wrote:
> > For the daemons in question the correct socket prefix is "libvirt-",
> > not "libvirtd-".
> > 
> > Fixes: dd4f2c73ad7f9fc0eae5325d5bf5786afd3a467e
> > 
> > Signed-off-by: Erik Skultety <eskultet at redhat.com>
> > ---
> >  src/remote/meson.build | 4 ++--
> >  1 file changed, 2 insertions(+), 2 deletions(-)
> > 
> > diff --git a/src/remote/meson.build b/src/remote/meson.build
> > index 25712c943b..5983238a0a 100644
> > --- a/src/remote/meson.build
> > +++ b/src/remote/meson.build
> > @@ -184,7 +184,7 @@ if conf.has('WITH_REMOTE')
> >        'service': 'libvirtd',
> >        'service_in': files('libvirtd.service.in'),
> >        'name': 'Libvirt',
> > -      'sockprefix': 'libvirtd',
> > +      'sockprefix': 'libvirt',
> >        'sockets': [ 'main', 'ro', 'admin', 'tcp', 'tls' ],
> >      }
> >  
> > @@ -218,7 +218,7 @@ if conf.has('WITH_REMOTE')
> >        'service': 'virtproxyd',
> >        'service_in': files('virtproxyd.service.in'),
> >        'name': 'Libvirt proxy',
> > -      'sockprefix': 'libvirtd',
> > +      'sockprefix': 'libvirt',
> >        'sockets': [ 'main', 'ro', 'admin', 'tcp', 'tls' ],
> >        'deps': libvirtd_socket_conflicts,
> 
> How did you figure out that the prefix is libvirt? It's not correct.
> Check the autoconf version or even in your fedora:
> 
>     rpm -ql libvirt-daemon | grep ".*\.socket"
> 
> and you will see that it's libvirtd.

So I double checked is as it looked strange to me and even though the
service and the systemd socket files use libvirtd the runtime unix
socket has libvirt as prefix. Sigh.

So the patch is correct, sorry about the noise.

Reviewed-by: Pavel Hrdina <phrdina at redhat.com>
-------------- next part --------------
A non-text attachment was scrubbed...
Name: signature.asc
Type: application/pgp-signature
Size: 833 bytes
Desc: not available
URL: <http://listman.redhat.com/archives/libvir-list/attachments/20200804/e2cb9139/attachment-0001.sig>