[PATCH spice-common] meson: Fix checking for python

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

 



When running with -Dpython-checks=false, the build fails. To fix this,
we move the python variable declaration outside of the get_option()
block as it will be used for calling the generators. Also removes the
unnecessary check for python3-devel.

Signed-off-by: Eduardo Lima (Etrunko) <etrunko@xxxxxxxxxx>
---
 .gitlab-ci.yml | 2 +-
 meson.build    | 7 ++++---
 2 files changed, 5 insertions(+), 4 deletions(-)

diff --git a/.gitlab-ci.yml b/.gitlab-ci.yml
index 0223bff..2169b82 100644
--- a/.gitlab-ci.yml
+++ b/.gitlab-ci.yml
@@ -4,7 +4,7 @@ before_script:
   - >
     dnf install 'dnf-command(copr)' git libtool make libasan
     python3 python3-six python3-pyparsing glib-networking
-    python3-devel meson ninja-build gdk-pixbuf2-devel
+    meson ninja-build gdk-pixbuf2-devel
     -y
   - dnf copr enable @spice/nightly -y
   - dnf builddep spice -y
diff --git a/meson.build b/meson.build
index 28e7c29..226d9f4 100644
--- a/meson.build
+++ b/meson.build
@@ -128,11 +128,12 @@ foreach dep : optional_deps
 endforeach
 
 # Python
+py_module = import('python3')
+python = py_module.find_python()
+
 if get_option('python-checks')
-  dependency('python3')
-  py_module = import('python3')
-  python = py_module.find_python()
   foreach module : ['six', 'pyparsing']
+    message('Checking for python module @0@'.format(module))
     cmd = run_command(python, '-m', module)
     if cmd.returncode() != 0
       error('Python module @0@ not found'.format(module))
-- 
2.14.4

_______________________________________________
Spice-devel mailing list
Spice-devel@xxxxxxxxxxxxxxxxxxxxx
https://lists.freedesktop.org/mailman/listinfo/spice-devel




[Index of Archives]     [Linux Virtualization]     [Linux Virtualization]     [Linux ARM Kernel]     [Linux ARM]     [Linux Omap]     [Fedora ARM]     [IETF Annouce]     [Security]     [Bugtraq]     [Linux OMAP]     [Linux MIPS]     [ECOS]     [Asterisk Internet PBX]     [Linux API]     [Monitors]