[v2] vala: fix documentation build problem

Submitted by Thomas Petazzoni on Sept. 18, 2012, 9:52 a.m.


Message ID 1347961957-14841-1-git-send-email-thomas.petazzoni@free-electrons.com
State Accepted
Commit f9b0c1d293fa412919507771446b8efbcbc0efc2
Headers show

Commit Message

Thomas Petazzoni Sept. 18, 2012, 9:52 a.m.
On gcc10, the build of the host-vala documentation causes a
segmentation fault of xsltproc, for some unknown reason. See:


Therefore, this patch fixes the problem by:

 * Ensuring that host-libxslt is amongst the dependency of vala when
   documentation is enabled so that we build a well-known version of
   libxslt/xsltproc to build the documentation. If documentation is
   not enabled, then we force vala to *not* find xsltproc so that it
   doesn't try to use the xsltproc of the build machine distribution
   if available.

 * Disabling the build of the documentation for host-vala. Note that
   we need to pass ':' as the xsltproc binary name to teach the
   configure script that xsltproc is not available.

Signed-off-by: Thomas Petazzoni <thomas.petazzoni@free-electrons.com>
 package/vala/vala.mk |   15 +++++++++++++++
 1 file changed, 15 insertions(+)


Peter Korsgaard Sept. 18, 2012, 7:28 p.m.
>>>>> "Thomas" == Thomas Petazzoni <thomas.petazzoni@free-electrons.com> writes:

 Thomas> On gcc10, the build of the host-vala documentation causes a
 Thomas> segmentation fault of xsltproc, for some unknown reason. See:

Committed, thanks.

Patch hide | download patch | download mbox

diff --git a/package/vala/vala.mk b/package/vala/vala.mk
index 984043c..1cbbbc4 100644
--- a/package/vala/vala.mk
+++ b/package/vala/vala.mk
@@ -9,9 +9,24 @@  VALA_VERSION_MINOR = 6
 VALA_SITE = http://download.gnome.org/sources/vala/$(VALA_VERSION_MAJOR)
 VALA_SOURCE = vala-$(VALA_VERSION).tar.xz
 VALA_DEPENDENCIES = host-flex libglib2 \
 		$(if $(BR2_NEEDS_GETTEXT_IF_LOCALE),gettext libintl)
+# If we want the documentation, then xsltproc is needed. If we don't
+# want the documentation, force Vala to not use the host xsltproc even
+# if available, because it may or may not work with Vala documentation
+# (some versions of xsltproc segfault)
+VALA_DEPENDENCIES += host-libxslt
+VALA_CONF_ENV = ac_cv_path_XSLTPROC=:
 HOST_VALA_DEPENDENCIES = host-flex host-libglib2
+# Yes, the autoconf script understands ':' as "xsltproc is not
+# available".
 $(eval $(autotools-package))
 $(eval $(host-autotools-package))