From: Andrea Bolognani <abologna@redhat.com>
Date: Tue, 19 Sep 2023 13:36:18 +0000 (+0200)
Subject: systemd: Only set @sockmode@ once
X-Git-Url: http://xenbits.xensource.com/gitweb?a=commitdiff_plain;h=18a1b25c22b64666475fda2890e1d9d9debe27d8;p=libvirt.git

systemd: Only set @sockmode@ once

The decision is based only on whether Polkit support is enabled,
so there's no need to go through it again for every single
service.

Signed-off-by: Andrea Bolognani <abologna@redhat.com>
Reviewed-by: Daniel P. Berrangé <berrange@redhat.com>
---

diff --git a/src/meson.build b/src/meson.build
index 3fae80d67c..9d5085a8aa 100644
--- a/src/meson.build
+++ b/src/meson.build
@@ -797,13 +797,13 @@ if conf.has('WITH_LIBVIRTD')
       install_dir: systemd_unit_dir,
     )
 
-    foreach unit : virt_daemon_units
-      if conf.has('WITH_POLKIT')
-        sockmode = '0666'
-      else
-        sockmode = '0600'
-      endif
+    if conf.has('WITH_POLKIT')
+      sockmode = '0666'
+    else
+      sockmode = '0600'
+    endif
 
+    foreach unit : virt_daemon_units
       unit_conf = configuration_data({
         'runstatedir': runstatedir,
         'sbindir': sbindir,