[Bug 796092] meson: use new python module

GStreamer (GNOME Bugzilla) bugzilla at gnome.org
Wed Sep 26 13:18:58 UTC 2018


https://bugzilla.gnome.org/show_bug.cgi?id=796092

--- Comment #6 from Jan Tojnar <jtojnar at gmail.com> ---
Review of attachment 371989:
 --> (https://bugzilla.gnome.org/review?bug=796092&attachment=371989)

The meson_version attribute should be bumped:

    WARNING: Project specifies a minimum meson_version '>= 0.36.0' but uses
features which were added in newer versions:
     * 0.37.0: {'shared_module'}
     * 0.46.0: {'Python Module'}

::: meson.build
@@ +23,2 @@
 pygobject_dep = dependency('pygobject-3.0', version : '>= 3.8')
 python_dep = dependency('python3')

This can be removed?

@@ +38,3 @@
 pygi_override_dir = get_option('pygi-overrides-dir')
 if pygi_override_dir == ''
+    cres = run_command(python, '-c','''

Why not save this to a separate file? It would support syntax highlighting
other dev tools.

@@ +47,3 @@
+        print(path)
+        exit(0)
+exit(1)

This does not have the “Otherwise follow python's way of install site packages
inside the provided prefix” branch from the original detector script, making it
fail in cases where `gi.overrides.__path__` does not start with `prefix` (e.g.
on NixOS). But since the  value can be fixed using meson option, it is probably
alright.

-- 
You are receiving this mail because:
You are the QA Contact for the bug.
You are the assignee for the bug.


More information about the gstreamer-bugs mailing list