Line 0
Link Here
|
0 |
- |
1 |
From 39f0ae6152acfb3d8ca086090a02e0aa8742f050 Mon Sep 17 00:00:00 2001 |
|
|
2 |
From: "Yann E. MORIN" <yann.morin.1998@free.fr> |
3 |
Date: Sun, 2 Dec 2018 16:26:07 +0100 |
4 |
Subject: [PATCH] buildsys: use variable for glib-genmarshal |
5 |
|
6 |
When doing cross-compilation, the path returned by pkg-config for |
7 |
glib-genmarshal can be incorrect (because it is the runtime path, |
8 |
not the build-time path). |
9 |
|
10 |
Change configure.ac to use pkg-config to get the variable. |
11 |
|
12 |
This allows overridig the path at configure time, by using the configure |
13 |
option GLIB_GENMARSHAL=/path/toglib-genmarshal |
14 |
|
15 |
Signed-off-by: "Yann E. MORIN" <yann.morin.1998@free.fr> |
16 |
[smcv: Fix typo in commit message] |
17 |
Reviewed-by: Simon McVittie <smcv@collabora.com> |
18 |
--- |
19 |
configure.ac | 4 ++-- |
20 |
dbus/Makefile.am | 4 ++-- |
21 |
dbus/examples/statemachine/Makefile.am | 4 ++-- |
22 |
3 files changed, 6 insertions(+), 6 deletions(-) |
23 |
|
24 |
diff --git a/configure.ac b/configure.ac |
25 |
index e00bc38..8be3147 100644 |
26 |
--- a/configure.ac |
27 |
+++ b/configure.ac |
28 |
@@ -267,8 +267,8 @@ AC_DEFINE([GLIB_VERSION_MIN_REQUIRED], [GLIB_VERSION_2_32], |
29 |
[Warn on use of APIs deprecated before GLib 2.32]) |
30 |
PKG_CHECK_MODULES([GLIB], [glib-2.0 >= 2.40, gobject-2.0 >= 2.40, gio-2.0 >= 2.40]) |
31 |
|
32 |
-GLIB_GENMARSHAL=`$PKG_CONFIG --variable=glib_genmarshal glib-2.0` |
33 |
-AC_SUBST(GLIB_GENMARSHAL) |
34 |
+PKG_CHECK_VAR([GLIB_GENMARSHAL], [glib-2.0], [glib_genmarshal],, |
35 |
+ [AC_MSG_ERROR([cannot find glib-genmarshal])]) |
36 |
|
37 |
dnl GLib flags |
38 |
AC_SUBST(GLIB_CFLAGS) |
39 |
diff --git a/dbus/Makefile.am b/dbus/Makefile.am |
40 |
index 37c6334..2e8cb27 100644 |
41 |
--- a/dbus/Makefile.am |
42 |
+++ b/dbus/Makefile.am |
43 |
@@ -87,10 +87,10 @@ dbus_binding_tool_LDADD= $(builddir)/libdbus-gtool.la $(builddir)/libdbus-glib-1 |
44 |
## we just rebuilt these manually and check them into cvs; easier than |
45 |
## convincing automake/make to do this properly |
46 |
regenerate-built-sources: |
47 |
- @GLIB_GENMARSHAL@ --prefix=_dbus_g_marshal dbus-gmarshal.list --header > dbus-gmarshal.h && \ |
48 |
+ $(GLIB_GENMARSHAL) --prefix=_dbus_g_marshal dbus-gmarshal.list --header > dbus-gmarshal.h && \ |
49 |
echo '#include <config.h>' > dbus-gmarshal.c && \ |
50 |
echo '#include "dbus-gmarshal.h"' >> dbus-gmarshal.c && \ |
51 |
- @GLIB_GENMARSHAL@ --prefix=_dbus_g_marshal dbus-gmarshal.list --body >> dbus-gmarshal.c |
52 |
+ $(GLIB_GENMARSHAL) --prefix=_dbus_g_marshal dbus-gmarshal.list --body >> dbus-gmarshal.c |
53 |
|
54 |
|
55 |
completiondir = $(sysconfdir)/bash_completion.d |
56 |
diff --git a/dbus/examples/statemachine/Makefile.am b/dbus/examples/statemachine/Makefile.am |
57 |
index 187b044..c8fe029 100644 |
58 |
--- a/dbus/examples/statemachine/Makefile.am |
59 |
+++ b/dbus/examples/statemachine/Makefile.am |
60 |
@@ -35,11 +35,11 @@ statemachine-glue.h: statemachine.xml |
61 |
|
62 |
sm-marshal.c: Makefile sm-marshal.list |
63 |
echo "#include <config.h>" > $@.tmp |
64 |
- @GLIB_GENMARSHAL@ --prefix=sm_marshal $(srcdir)/sm-marshal.list --header --body >> $@.tmp |
65 |
+ $(GLIB_GENMARSHAL) --prefix=sm_marshal $(srcdir)/sm-marshal.list --header --body >> $@.tmp |
66 |
mv $@.tmp $@ |
67 |
|
68 |
sm-marshal.h: Makefile sm-marshal.list |
69 |
- @GLIB_GENMARSHAL@ --prefix=sm_marshal $(srcdir)/sm-marshal.list --header > $@.tmp && mv $@.tmp $@ |
70 |
+ $(GLIB_GENMARSHAL) --prefix=sm_marshal $(srcdir)/sm-marshal.list --header > $@.tmp && mv $@.tmp $@ |
71 |
|
72 |
BUILT_SOURCES += sm-marshal.c sm-marshal.h |
73 |
|
74 |
-- |
75 |
GitLab |
76 |
|