GNOME Bugzilla – Bug 784133
meson: Fix some issues when using wip/meson branch as a subproject in meson
Last modified: 2017-10-16 12:11:06 UTC
See commit messages.
Created attachment 354326 [details] [review] gdbus-codegen: Make header inclusion simply relative Otherwise in "complex" cases, for example using glib as a subproject in meson the output dir is relative to the toplevel build directory (meaning subprojects/glib/gio/...) and the include paths won't fit. The header file and C file being always generated in the same output directory, the simplest solution is to just do a relative import using the header filename.
Created attachment 354327 [details] [review] meson: Defining MAJOR_IN_ as done by AC_HEADER_MAJOR
Created attachment 354328 [details] [review] meson: Fix building as a subproject meson.source_root() returns the toplevel source directory of the toplevel project, thus the paths were wrong when using it. Simply using files() gets us the right path
Created attachment 354329 [details] [review] meson: Fix the build defining HAVE_RTLD_X Otherwise RTLD_XXX get redefined.
I think someone else needs to look at the gdbus-codegen.py patch, and how/if it affects other builds.
Comment on attachment 354328 [details] [review] meson: Fix building as a subproject >From 66594b04d8745720923511988aa66acca2485ebb Mon Sep 17 00:00:00 2001 >From: Thibault Saunier <thibault.saunier@osg.samsung.com> >Date: Fri, 23 Jun 2017 12:00:13 -0400 >Subject: [PATCH] meson: Fix building as a subproject > >meson.source_root() returns the toplevel source directory >of the toplevel project, thus the paths were wrong when using >it. Simply using files() gets us the right path > >https://bugzilla.gnome.org/show_bug.cgi?id=784133 >--- > gio/meson.build | 2 +- > 1 file changed, 1 insertion(+), 1 deletion(-) > >diff --git a/gio/meson.build b/gio/meson.build >index 603b0b3af..31153bb1f 100644 >--- a/gio/meson.build >+++ b/gio/meson.build >@@ -732,7 +732,7 @@ if libelf.found() > endif > > gconstructor_as_data_h = custom_target('gconstructor_as_data.h', >- input : ['data-to-c.py', meson.source_root() + '/glib/gconstructor.h'], >+ input : ['data-to-c.py', files('../glib/gconstructor.h')], > output : ['gconstructor_as_data.h'], > command : [python, '@INPUT0@', '@INPUT1@', 'gconstructor_code', '@OUTPUT@']) > >-- >2.13.1
(In reply to Tim-Philipp Müller from comment #5) > I think someone else needs to look at the gdbus-codegen.py patch, and how/if > it affects other builds. Who could review that one?
This is not needed anymore.