This makes is possible to reduce the number of cases we have to
consider, because 'sles' declares itself to be like 'suse' and
both 'rhel' and 'centos' declare themselves to be like 'fedora'.

We have to move the check for Ubuntu before the one for Debian,
however, because 'ubuntu' declares itself to be like 'debian'
and it would end up with the wrong defaults otherwise.

Suggested-by: Olaf Hering <o...@aepfle.de>
Signed-off-by: Andrea Bolognani <abolo...@redhat.com>
---
 meson.build | 13 +++++--------
 1 file changed, 5 insertions(+), 8 deletions(-)

diff --git a/meson.build b/meson.build
index ab5f7433a6..388e2cfa5e 100644
--- a/meson.build
+++ b/meson.build
@@ -1659,24 +1659,21 @@ if not get_option('driver_qemu').disabled()
       default_qemu_user = 'root'
       default_qemu_group = 'wheel'
     else
-      os_release = run_command('grep', '^ID=', '/etc/os-release', check: 
false).stdout()
+      os_release = run_command('grep', '-E', '^ID(_LIKE)*=', 
'/etc/os-release', check: false).stdout()
       if os_release.contains('arch')
         default_qemu_user = 'nobody'
         default_qemu_group = 'nobody'
-      elif (os_release.contains('centos') or
-            os_release.contains('fedora') or
+      elif (os_release.contains('fedora') or
             os_release.contains('gentoo') or
-            os_release.contains('rhel') or
-            os_release.contains('sles') or
             os_release.contains('suse'))
         default_qemu_user = 'qemu'
         default_qemu_group = 'qemu'
-      elif os_release.contains('debian')
-        default_qemu_user = 'libvirt-qemu'
-        default_qemu_group = 'libvirt-qemu'
       elif os_release.contains('ubuntu')
         default_qemu_user = 'libvirt-qemu'
         default_qemu_group = 'kvm'
+      elif os_release.contains('debian')
+        default_qemu_user = 'libvirt-qemu'
+        default_qemu_group = 'libvirt-qemu'
       else
         default_qemu_user = 'root'
         default_qemu_group = 'root'
-- 
2.34.1

Reply via email to