[gvfs] build: Remove gdbus codegen workaround



commit 32bde011b2522d302d72506884866bd809ef5bda
Author: Iñigo Martínez <inigomartinez gmail com>
Date:   Mon Dec 17 22:52:40 2018 +0100

    build: Remove gdbus codegen workaround
    
    When gvfs was ported to meson `gdbus-codegen` was not able to
    generate the source and header files independently, and this caused
    issues on highly parallelized build.
    
    To avoid this issue a workaround was placed by using an external
    script. However, recently glib and meson acquired support for
    generating source and header files independently, so the
    workaround has been removed.

 codegen.py           | 28 ----------------------------
 common/meson.build   | 16 ++++++----------
 meson.build          |  6 ------
 metadata/meson.build | 16 ++++++----------
 4 files changed, 12 insertions(+), 54 deletions(-)
---
diff --git a/common/meson.build b/common/meson.build
index a718783f..beef8c26 100644
--- a/common/meson.build
+++ b/common/meson.build
@@ -14,16 +14,12 @@ sources = files(
   'gvfsutils.c',
 )
 
-# FIXME: Ugly workaround that simulates the generation of
-#        two different targets.
-namespace = 'GVfsDBus'
-name = namespace.to_lower()
-
-dbus_sources = custom_target(
-  name,
-  input: gvfs_namespace + '.xml',
-  output: [name + '.c', name + '.h'],
-  command: [codegen, gvfs_namespace + '.', name, namespace, meson.current_build_dir(), '@INPUT@', 
'@OUTPUT@'],
+dbus_sources = gnome.gdbus_codegen(
+  'gvfsdbus',
+  gvfs_namespace + '.xml',
+  interface_prefix: gvfs_namespace + '.',
+  namespace: 'GVfsDBus',
+  autocleanup: 'objects',
 )
 
 deps = [
diff --git a/meson.build b/meson.build
index 19e91c3a..43e126c0 100644
--- a/meson.build
+++ b/meson.build
@@ -453,12 +453,6 @@ endif
 enable_devel_utils = get_option('devel_utils')
 enable_installed_tests = get_option('installed_tests')
 
-# FIXME: Opaque target return from gdbus_codegen
-# Please see:
-#   https://bugzilla.gnome.org/show_bug.cgi?id=791015
-#   https://github.com/mesonbuild/meson/pull/2930
-codegen = find_program('codegen.py')
-
 gnome = import('gnome')
 i18n = import('i18n')
 pkg = import('pkgconfig')
diff --git a/metadata/meson.build b/metadata/meson.build
index bbfd09fd..e51f1564 100644
--- a/metadata/meson.build
+++ b/metadata/meson.build
@@ -29,16 +29,12 @@ metadata_service = configure_file(
   install_dir: dbus_session_bus_services_dir,
 )
 
-# FIXME: Ugly workaround that simulates the generation of
-#        two different targets.
-namespace = 'GVfs'
-name = 'metadata-dbus'
-
-dbus_sources = custom_target(
-  name,
-  input: 'dbus-interface.xml',
-  output: [name + '.c', name + '.h'],
-  command: [codegen, gvfs_namespace + '.', name, namespace, meson.current_build_dir(), '@INPUT@', 
'@OUTPUT@'],
+dbus_sources = gnome.gdbus_codegen(
+  'metadata-dbus',
+  'dbus-interface.xml',
+  interface_prefix: gvfs_namespace + '.',
+  namespace: 'GVfs',
+  autocleanup: 'objects',
 )
 
 sources = files(


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