diff mbox series

[6/7] meson: use dependency to gate block modules

Message ID 20201216162006.433850-7-pbonzini@redhat.com (mailing list archive)
State New, archived
Headers show
Series First round of meson bugfixes and cleanups for 6.0 | expand

Commit Message

Paolo Bonzini Dec. 16, 2020, 4:20 p.m. UTC
This will allow converting the dependencies to meson options one by one,
because moving the tests to meson.build will get rid of the symbols
in config-host.mak.

Signed-off-by: Paolo Bonzini <pbonzini@redhat.com>
---
 block/meson.build | 20 ++++++++++----------
 1 file changed, 10 insertions(+), 10 deletions(-)

Comments

Marc-André Lureau Dec. 16, 2020, 4:57 p.m. UTC | #1
On Wed, Dec 16, 2020 at 8:26 PM Paolo Bonzini <pbonzini@redhat.com> wrote:

> This will allow converting the dependencies to meson options one by one,
> because moving the tests to meson.build will get rid of the symbols
> in config-host.mak.
>
> Signed-off-by: Paolo Bonzini <pbonzini@redhat.com>
>

Reviewed-by: Marc-André Lureau <marcandre.lureau@redhat.com>

---
>  block/meson.build | 20 ++++++++++----------
>  1 file changed, 10 insertions(+), 10 deletions(-)
>
> diff --git a/block/meson.build b/block/meson.build
> index 5dcc1e5cce..b02cb14aad 100644
> --- a/block/meson.build
> +++ b/block/meson.build
> @@ -70,14 +70,14 @@ block_modules = {}
>
>  modsrc = []
>  foreach m : [
> -  ['CONFIG_CURL', 'curl', [curl, glib], 'curl.c'],
> -  ['CONFIG_GLUSTERFS', 'gluster', glusterfs, 'gluster.c'],
> -  ['CONFIG_LIBISCSI', 'iscsi', libiscsi, 'iscsi.c'],
> -  ['CONFIG_LIBNFS', 'nfs', libnfs, 'nfs.c'],
> -  ['CONFIG_LIBSSH', 'ssh', libssh, 'ssh.c'],
> -  ['CONFIG_RBD', 'rbd', rbd, 'rbd.c'],
> +  [curl, 'curl', [curl, glib], 'curl.c'],
> +  [glusterfs, 'gluster', glusterfs, 'gluster.c'],
> +  [libiscsi, 'iscsi', libiscsi, 'iscsi.c'],
> +  [libnfs, 'nfs', libnfs, 'nfs.c'],
> +  [libssh, 'ssh', libssh, 'ssh.c'],
> +  [rbd, 'rbd', rbd, 'rbd.c'],
>  ]
> -  if config_host.has_key(m[0])
> +  if m[0].found()
>      if enable_modules
>        modsrc += files(m[3])
>      endif
> @@ -90,10 +90,10 @@ endforeach
>  # those are not exactly regular block modules, so treat them apart
>  if 'CONFIG_DMG' in config_host
>    foreach m : [
> -    ['CONFIG_LZFSE', 'dmg-lzfse', liblzfse, 'dmg-lzfse.c'],
> -    ['CONFIG_BZIP2', 'dmg-bz2', [glib, libbzip2], 'dmg-bz2.c']
> +    [liblzfse, 'dmg-lzfse', liblzfse, 'dmg-lzfse.c'],
> +    [libbzip2, 'dmg-bz2', [glib, libbzip2], 'dmg-bz2.c']
>    ]
> -    if config_host.has_key(m[0])
> +    if m[0].found()
>        module_ss = ss.source_set()
>        module_ss.add(when: m[2], if_true: files(m[3]))
>        block_modules += {m[1] : module_ss}
> --
> 2.29.2
>
>
>
>
diff mbox series

Patch

diff --git a/block/meson.build b/block/meson.build
index 5dcc1e5cce..b02cb14aad 100644
--- a/block/meson.build
+++ b/block/meson.build
@@ -70,14 +70,14 @@  block_modules = {}
 
 modsrc = []
 foreach m : [
-  ['CONFIG_CURL', 'curl', [curl, glib], 'curl.c'],
-  ['CONFIG_GLUSTERFS', 'gluster', glusterfs, 'gluster.c'],
-  ['CONFIG_LIBISCSI', 'iscsi', libiscsi, 'iscsi.c'],
-  ['CONFIG_LIBNFS', 'nfs', libnfs, 'nfs.c'],
-  ['CONFIG_LIBSSH', 'ssh', libssh, 'ssh.c'],
-  ['CONFIG_RBD', 'rbd', rbd, 'rbd.c'],
+  [curl, 'curl', [curl, glib], 'curl.c'],
+  [glusterfs, 'gluster', glusterfs, 'gluster.c'],
+  [libiscsi, 'iscsi', libiscsi, 'iscsi.c'],
+  [libnfs, 'nfs', libnfs, 'nfs.c'],
+  [libssh, 'ssh', libssh, 'ssh.c'],
+  [rbd, 'rbd', rbd, 'rbd.c'],
 ]
-  if config_host.has_key(m[0])
+  if m[0].found()
     if enable_modules
       modsrc += files(m[3])
     endif
@@ -90,10 +90,10 @@  endforeach
 # those are not exactly regular block modules, so treat them apart
 if 'CONFIG_DMG' in config_host
   foreach m : [
-    ['CONFIG_LZFSE', 'dmg-lzfse', liblzfse, 'dmg-lzfse.c'],
-    ['CONFIG_BZIP2', 'dmg-bz2', [glib, libbzip2], 'dmg-bz2.c']
+    [liblzfse, 'dmg-lzfse', liblzfse, 'dmg-lzfse.c'],
+    [libbzip2, 'dmg-bz2', [glib, libbzip2], 'dmg-bz2.c']
   ]
-    if config_host.has_key(m[0])
+    if m[0].found()
       module_ss = ss.source_set()
       module_ss.add(when: m[2], if_true: files(m[3]))
       block_modules += {m[1] : module_ss}